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

["Zeeshan Ali (Khattak)" <zeeshanak gnome org>]

On Mon, Jul 18, 2011 at 10:37 AM, Sven Neumann <s neumann raumfeld com> wrote:
> On Mon, 2011-07-18 at 01:23 +0300, Zeeshan Ali (Khattak) wrote:
>>   However, I wonder if those percentages map to real scenarios. Any
>> decent client will never parse that many DIDLs in a short amount of
>> time. i-e it will only browse objects that are actually needed and
>> even then, it will use the right filter to get only the properties
>> that it needs. Usually that translates to what is visible to user at a
>> particular time and neighbouring objects. Right?
>
> Well, we are not looking at absolute numbers here. The relative numbers
> given by Jens should be more or less representative for a real-world
> usage scenario. It doesn't really matter if the client parses one DIDL
> object or thousands.

  IMHO It does. If you browse 1000 objects at a time and use a filter
of '*', there will be two issues:

1. User will most probably have to wait for a few seconds to be able
to see the results. i-e bad user experience.
2. You will chew a lot more of the CPU (and memory too) than you
really need. That would be a disaster for embedded systems.

> I may find time later this week to benchmark our code. I'll let you know
> about the results then.

  That would be very much appreciated. Thanks!

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124