"H. Langos" <henrik-gnu...@prak.org> wrote on 01.06.2009 22:00:21:

> Hi David,
> 
> I have some suggestions:
> 
> 1. Add a hexdump either for the complete chunk or for the decoded 
fields.
>    That helps to verify the information we already know (or belief 
> to know) and
>    also shows how wide the decoded field was.
> 
> Example:  [header_length]               => 64 (40 00 00 00)

Done.

> 2. Separate bytes in the hexdump by a single space and groups of 8 
> (or 4) bytes 
>    by two spaces and add line break after 16 bytes.
> Example: [unknown3]    => 40 02 00 00 9c 0c 00 00  24 d8 b7 00 00 00 00 
00 
>                           38 c1 69 00 00 00 00 00  81 00 00 00

Done.
 
> 3. When decoded output is shown in hex, add the "0x" prefix
> 
> Example:  [playlist_header_chunk_offset]=> 0x0000b964 (64 b9 00 00)

Done.
 
> 4. When allowing users to upload files, create a separate directory
> [...]

Will come this week. 

Looks like we will create a social network web application for sharing 
iTunes databases ;-)

David


_______________________________________________
Bug-gnupod mailing list
Bug-gnupod@nongnu.org
http://lists.nongnu.org/mailman/listinfo/bug-gnupod

Reply via email to