On 20/03/06, Phil Cazzola <[EMAIL PROTECTED]> wrote:

> Just more info....
>
> The core dump is from easyxml.cxx:267.
>
> It is easy enough to understand.
>
> If there is an error  readXML() calls XML_ParserFree  twice.
> After calling XML_ParserFree there should probably  be a return statement (3 
> cases).

I write more Java than C++, so I might be importing my prejudices, but
wouldn't a return be redundant after throwing an exception?  Each of
the instances looks like this (in the SimGear version, at least):

    XML_ParserFree(parser);
    throw ex;

There's no way I see that the control flow could reach the second
XML_ParserFree at the end of the method call.


All the best,


David

--
http://www.megginson.com/


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid0944&bid$1720&dat1642
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel

Reply via email to