Opened 11 years ago

Closed 11 years ago

Last modified 12 months ago

#3954 closed defect (fixed)

GUI:Clicking "Load" when no games are added triggers a crash

Reported by: SF/espiox Owned by: lordhoto
Priority: normal Component: GUI
Keywords: Cc:
Game:

Description

(Lastest SVN build, Win32) If no games have been added to ScummVM's games list, clicking the "Load" button on the launcher crashes the program. Presumably, the button should be inactive if no game is selected, like the "Start" button.

Ticket imported from: #2100790. Ticket imported from: bugs/3954.

Attachments (1)

load_fix.patch (1.4 KB ) - added by bluegr 11 years ago.
Load button fix

Download all attachments as: .zip

Change History (8)

comment:1 by SF/espiox, 11 years ago

Summary: Clicking "Load" when no games are added triggers a crashGUI:Clicking "Load" when no games are added triggers a crash

comment:2 by fingolfin, 11 years ago

Hi there,
indeed, if no game is selected, the "Load" button should be disabled. Chris, want to try fixing this? :)

comment:3 by fingolfin, 11 years ago

Owner: set to SF/cpage88

by bluegr, 11 years ago

Attachment: load_fix.patch added

Load button fix

comment:4 by bluegr, 11 years ago

I went ahead and fiddled around with this, as it seemed to be pretty straightforward...

I've attached a patch that seems to be working correctly and fixes the problem

Please, revew :)
File Added: load_fix.patch

comment:5 by lordhoto, 11 years ago

Committed md5's patch. Thanks for reporting.

comment:6 by lordhoto, 11 years ago

Owner: changed from SF/cpage88 to lordhoto
Resolution: fixed
Status: newclosed

comment:7 by digitall, 12 months ago

Component: --Unset--GUI
Note: See TracTickets for help on using tickets.