[Scummvm-tracker] [ScummVM :: Bugs] #14652: SCI: Additional crash context when ScummVM errors

[ScummVM :: Bugs]

#14652: SCI: Additional crash context when ScummVM errors
--------------------------------------+--------------------------
Reporter:  deckarep                   |       Owner:  deckarep
    Type:  feature request            |      Status:  closed
Priority:  normal                     |   Component:  Engine: SCI
 Version:                             |  Resolution:  fixed
Keywords:  Bug reporting and crashes  |        Game:
--------------------------------------+--------------------------
Changes (by sluicebox):

 * owner:  (none) => deckarep
 * status:  new => closed
 * resolution:   => fixed

Comment:

 Please do!

 I can't believe that only two other engines use `errorString`! (SCUMM and
 NANCY.) It's been there since at least 2006!! I have begun a campaign to
 raise awareness.

 This would not have gotten done anytime soon if it was just "someone
 please implement `errorString`", so thank you for getting the ball rolling
 and showing how it's done. This was good timing too, because things are
 kind of quiet in SCI at the moment, and I've been sniping code cleanups
 when I can, so I'm growing more intolerant of unreadable C-isms and bad
 comments. Looking at the core VM/engine code again to figure out the
 queries has added targets to my hit list...

 Feature Request Complete! Closing with gusto!
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/14652#comment:7>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM