Re: [PATCH] Add a basic date format check to DIDL-Lite parser



On Mo, 2011-07-11 at 10:48 +0200, Sven Neumann wrote:
> On Mon, 2011-07-11 at 09:58 +0200, Jens Georg wrote:
> > > Could we try to keep the DIDL-Lite parser as efficient as possible? It
> > > is already rather slow the way it is implemented right now. This is a
> > > real problem on embedded devices.
> > 
> > Can you point out the other issues you're seeing? Maybe we can fix them
> > this cycle?
> 
> I can't point at a particular issue. All I can say is that parsing
> DIDL-Lite takes a considerable amount of CPU cycles. It might help to
> use a SAX based parser instead of building the DOM, but that would be a
> major rewrite of gupnp-av and I am not sure if it's worth the effort.

Hm, yes. true. Nevertheless I filed an enhancement bug so this isn't
lost:

https://bugzilla.gnome.org/show_bug.cgi?id=654530





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