Re: [Informa-developer] FeedParser and NoOpEntityResolver

[Aleksey G. <spy...@no...>]

Niko,

Done. I'll wait for another day before checking in the changes. :)

Thanks,
Aleksey

Niko Schmuck wrote:
>>Please tell me why the line below was commented? It is in
>>FeedParser's parse() method.
>>
>>     // turn off DTD loading
>>     // saxBuilder.setEntityResolver(new NoOpEntityResolver());
>>
>>The reason I'm asking is that before this the feeds with <!DOCTYPE
>>...> pointing to non-existing DTD were parsed fine and now they
>>cannot be parsed because of missing DTD.
> 
> 
> I'm not sure either, why this has gone. At least I found out that
> it has gone since a commit made by me on April 20, 2004 :-)
> [RSSParser/rev 1.7]. Could be related to feeds using character
> entities which were not declared.
> 
> If nobody else crys (all others are invited now to cry out loud,
> now), please add a short test case that lightens the problem and
> then comment it out again.
> 
> Thanks and sorry for the inconvenience,
> Niko