[prev in list] [next in list] [prev in thread] [next in thread] 

List:       xerces-c-dev
Subject:    Re: Fatal error not resetting parseInProgress
From:       "Jason Stewart" <jason.e.stewart () gmail ! com>
Date:       2006-10-26 10:42:01
Message-ID: 41c1ade50610260330t116c2900m1ed35a859c485702 () mail ! gmail ! com
[Download RAW message or body]

Hey David,

Done - XERCESC-1643 is all yours ;-)

Cheers, jas.

On 9/25/06, David Bertoni <dbertoni@apache.org> wrote:
> Jason Stewart wrote:
> > Hey,
> >
> > In the test suite code I've noticed that certain errors leave the
> > parser in a useless state, i.e. it will issue a ParseInProgress
> > exception if you call parse() again. It doesn't seem possible to reset
> > this either.
> >
> > XercesDOMParser has a resetParse() method - but it is private, and so
> > not available to the test-suite.
> >
> > SAXParser uses the Janitor to do cleanup - but I don't really
> > understand how it works - and it seems that exceptions bypass the
> > cleanup.
> >
> > I haven't yet tried the DOMLSParser or the SAX2XMLReader - but I
> > imagine there are similar issues.
> >
> > What should be done with parser objects after exceptions? Are they
> > unusable, and they must be re-allocated from scratch? Or is their a
> > public way to reset them?
> >
>
> I think this is a bug, and that SAXParser et al. should reset anything they
> need to in their cleanup() member functions.
>
> Can you file a bug and assign it to me?  It would be great if you could
> figure out which classes are broken, but only if you have the time.
>
> Thanks!
>
> Dave
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: c-dev-unsubscribe@xerces.apache.org
> For additional commands, e-mail: c-dev-help@xerces.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscribe@xerces.apache.org
For additional commands, e-mail: c-dev-help@xerces.apache.org

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic