Re: [PATCH 2/5] connectivity: Add libxml2 as a dependency



On Mon, 2013-02-11 at 17:10 +0100, Bastien Nocera wrote:
> On Mon, 2013-02-11 at 10:06 -0600, Dan Williams wrote:
> > On Mon, 2013-02-11 at 12:09 -0200, Jonh Wendell wrote:
> > > From: Jonh Wendell <jonh wendell oiwifi com br>
> > > 
> > > libsoup already depends on libxml2 but we need to explicitly link
> > > to it.
> > 
> > At least we already theoretically required it; though is it possible to
> > use GMarkup here instead of libxml2?  GMarkup would be somewhat simpler,
> > though it's only a subset.
> 
> Given how it's used, there's probably little reason this couldn't be a
> regexp. Both GMarkup and libxml2 would choke on broken, slightly broken,
> and very very broken HTML files.

Yeah, and I've heard that for example, some hotspots literally just
append raw XML to the end of the HTTP request outside the XML.  I think
we need to be somewhat more robust here and XML parsing may not be the
way to get there?  Also, we may need to add special cases for various
hotspots, which might require regex and not just XML parsing.

Dan



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]