#1538 closed defect (fixed)
GUI: BrowserDialog return value
| Reported by: | wjp | Owned by: | fingolfin |
|---|---|---|---|
| Priority: | normal | Component: | GUI |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
Pressing escape in the BrowserDialog sets the return value to -1 (Dialog::handleKeyDown)
LauncherDialog::addGame() interprets any non-zero return value as a successful selection.
Result: crash when attempting to get/use the selected directory.
(Latest scummvm anon-CVS)
Ticket imported from: #917532. Ticket imported from: bugs/1538.
Change History (5)
comment:1 by , 21 years ago
| Owner: | set to |
|---|---|
| Resolution: | → fixed |
comment:2 by , 21 years ago
comment:3 by , 21 years ago
I'll test the 'real' CVS version when it filters through to anonymous CVS.
In the meantime I added the '> 0' check in launcher.cpp (the location I reported) manually, and that indeed fixes that crash, as expected.
comment:4 by , 21 years ago
| Status: | new → closed |
|---|
comment:5 by , 6 years ago
| Component: | --Unset-- → GUI |
|---|
Note:
See TracTickets
for help on using tickets.
Should be fixed in CVS; please test.