[Scummvm-tracker] [ScummVM :: Bugs] #16112: NANCY: Nancy Drew Stay Tuned For Danger Crashing

[ScummVM :: Bugs]

#16112: NANCY: Nancy Drew Stay Tuned For Danger Crashing
-------------------------+------------------------------------------------
Reporter:  bananapeel11  |       Owner:  (none)
    Type:  defect        |      Status:  new
Priority:  normal        |   Component:  Engine: Nancy
 Version:                |  Resolution:
Keywords:                |        Game:  Nancy Drew: Stay Tuned for Danger
-------------------------+------------------------------------------------
Comment (by antoniou79):

 I can confirm that there's an assertion failure there. The code attempts
 to call pop_front() of our queue implementation and fails the assertion
 that the queue cannot be empty.

 After some debugging, it looks like this is triggered from the nancy
 engine's hypertext.cpp file here:
 https://github.com/scummvm/scummvm/blob/e347e81fcea7d804e30919b00fe96a11060d3fe7/engines/nancy/misc/hypertext.cpp#L248

 I have issued a PR with a potential fix for this, whereby I basically add
 a warning and an extra guard in the if clause for entering that part of
 the code. A developer more familiar with the engine should review the PR
 (or implement a more efficient solution).
 https://github.com/scummvm/scummvm/pull/6840
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/16112#comment:11>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM