[Scummvm-devel] New way of handling GUIO
Tarek Soliman
tarek-spam at zeusmail.bounceme.net
Mon Oct 24 23:40:43 CEST 2011
On Mon, Oct 24, 2011 at 09:33:29PM +0300, Filippos Karapetis wrote:
> On Mon, Oct 24, 2011 at 8:06 AM, Tarek Soliman <
> tarek-spam at zeusmail.bounceme.net> wrote:
>
> > https://github.com/tsoliman/scummvm/commits/scumm-speech-detection2
> > This sets a null pointer on gui options and works only because of the
> > current detection table entries not containing anything critical.
> > It works fine because the guioptions are being used only after being
> > wrapped into a Common::String which is well guarded against null pointers:
> > it treats it as an empty string.
> > This works _now_ but won't work if the detection entries change which is
> > going to happen.
> >
>
> Well, you could use "" or GUIO1(GUIO_NONE) instead, which should work just
> fine. In fact, I think this patch (with the suggested change) looks like the
> cleanest approach (and it's not a hack)
Thanks Filippos and Matt for helping with this.
--
Tarek
More information about the Scummvm-devel
mailing list