Opened 12 years ago

Closed 12 years ago

Last modified 12 months ago

#3348 closed defect (fixed)

AMIGAOS4: Amiga key doesn't fire on AmigaOne

Reported by: peres Owned by: peres
Priority: normal Component: Port: AmigaOS4
Keywords: Cc:
Game:

Description

Key combinations using the special Amiga key don't work on AmigaOne. This notably prevents user from quitting.

Ticket imported from: #1763018. Ticket imported from: bugs/3348.

Change History (10)

comment:1 by fingolfin, 12 years ago

Priority: highnormal
Status: newpending

comment:2 by fingolfin, 12 years ago

To process your bug report appropriately, we need you to
provide the following additional information:

* ScummVM version (PLEASE test the latest CVS/Daily build)
* Bug details, including instructions on reproducing it
* Language of game (English, German, ...)
* Version of game (talkie, floppy, ...)
* Platform and Compiler (Win32, Linux, MacOS, ...)
* Attach a save game if possible
* If this bug only occurred recently, please note the last
version without the bug, and the first version including
the bug. That way we can fix it quicker by looking at the
changes made.

This should only take you a little time but will make it much easier for
us to process your bug report in a way that satisfies both you and us.

Thank you for your support!

comment:3 by peres, 12 years ago

* ScummVM version: latest SVN
* Bug details, including instructions on reproducing it : just start a game and try known quit combination
* Language of game (English, German, ...) : any language
* Version of game (talkie, floppy, ...) : any version
* Platform and Compiler (Win32, Linux, MacOS, ...) : AmigaOS4, as stated

comment:4 by peres, 12 years ago

Status: pendingnew

comment:5 by peres, 12 years ago

After some testing performed by Raziel, we discovered that the Amiga key is not recognized as a key modifier by SDL on AmigaOS4 (and maybe even on older AmigaOS'es).

If nobody objects, I would change the #ifdef at line 235 in backends/platform/sdl/events.cpp like below:

- #if defined(MACOSX) || defined(__AMIGA__)
+ #if defined(MACOSX)

This way, Amiga users will be able to quit using 'ctrl-z' or 'alt-x' like we do on most other platforms.

comment:6 by fingolfin, 12 years ago

Sounds reasonable!

comment:7 by peres, 12 years ago

Resolution: fixed
Status: newclosed

comment:8 by peres, 12 years ago

Applied code in revision 29826. Now closing this item.

comment:9 by Kirben, 10 years ago

Owner: set to peres

comment:10 by digitall, 12 months ago

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