Opened 9 years ago

Closed 9 years ago

Last modified 10 months ago

#9192 closed patch (wontfix)

BRA: don't display null strings

Reported by: fuzzie Owned by: peres
Priority: normal Component: Engine: Parallaction
Keywords: Cc:


This 'short-circuits' the dialogue loop for when an answer is 'null' in order to jump to the next answer immediately, to stop scummvm from displaying 'null' inside balloons. You can see this in a lot of places in BRA, for example examine the money in your inventory.

Patch seems to apply fine to trunk. I don't know if this might cause problems for NS or there's something I missed, though.

Ticket imported from: #3021742. Ticket imported from: patches/1297.

Attachments (1)

0004-always-execute-next-dialog-state-immediately-if-prev.txt (1.3 KB) - added by fuzzie 9 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 9 years ago by fuzzie

Owner: set to peres

comment:2 Changed 9 years ago by peres

I committed an alternative solution in r55603 - inspired by your patch #3021740 - changing code in only one spot.

comment:3 Changed 9 years ago by peres

Resolution: wontfix
Status: newclosed

comment:4 Changed 10 months ago by digitall

Component: Engine: Parallaction
Note: See TracTickets for help on using tickets.