[Scummvm-tracker] [ScummVM :: Bugs] #12012: COMMON: Rewrote of encoder causes assertion on typing ALT+? key combos (AmigaOS4) (was: COMMON: Rewrote of encoder causes assertion when using ALT+x in LA titles (AmigaOS4))

[ScummVM :: Bugs]

#12012: COMMON: Rewrote of encoder causes assertion on typing ALT+? key combos
(AmigaOS4)
----------------------+----------------------------------------
  Reporter:  raziel-  |      Owner:  (none)
      Type:  defect   |     Status:  new
  Priority:  normal   |  Component:  Common
Resolution:           |   Keywords:  regression, encoder, iconv
      Game:           |
----------------------+----------------------------------------
Changes (by raziel-):

 * summary:
     COMMON: Rewrote of encoder causes assertion when using ALT+x in LA
     titles (AmigaOS4)
     =>
     COMMON: Rewrote of encoder causes assertion on typing ALT+? key combos
     (AmigaOS4)


Old description:

> Latest sources built locally
>
> Lately i was testing the RTL/QTL feature and stumbled over an assertion
> in LA titles when trying to quit with ALT+x.
>
> Since no one else seems to have this problem, i bisected it myself
> manually and found the culprit in this commit:
> https://github.com/scummvm/scummvm/commit/68a9136e4d43b424243e4ad33821138a7b659aa0
> which was this FR: https://github.com/scummvm/scummvm/pull/2586
>
> Unfortunately i don't know anything about the stuff in there, but would
> love to have this fixed on AmigaOS.
>
> @phcoder
> Could you take a look, please?
>
> @BeWorld2018
> Could you test on MorphOS as well, please?
>
> Thanks a lot in advance

New description:

 Latest sources built locally

 Since no one else seems to have this problem, i bisected it myself
 manually and found the culprit in this commit:
 https://github.com/scummvm/scummvm/commit/68a9136e4d43b424243e4ad33821138a7b659aa0
 which was this FR: https://github.com/scummvm/scummvm/pull/2586

 Unfortunately i don't know anything about the stuff in there, but would
 love to have this fixed on AmigaOS.

 @phcoder
 Could you take a look, please?

 @BeWorld2018
 Could you test on MorphOS as well, please?

 Thanks a lot in advance

--
Comment:

 Nevermind the games i tested.
 I can make it crash as basic as the launcher.
 When launcher is up, i just type a key combo (ALT+x and ALT+w confirmed)
 to make ScummVM assert.

 It seems as if the new encoder is trying to do something that brings down
 my system.
 Not sure if those key combos are part of a language
 Alt+x is × on my system.
 Alt+w is å on my system.

 Help?
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/12012#comment:4>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM