Re: [Gtkpod-devel] [PATCH] Update video support

[Jorg S. <Jor...@gm...>]

Hi Mario,

thanks for your patch -- looks quite good I think. I've uploaded your 
patch to CVS.

> With a fresh installation, "General" is the one selected by default,
> and yes, it remembers the last one used (but that works without me
> doing anything).

That's good. I've probably implemented this when I first wrote the Edit 
Details dialog and subsequently forgot about it...

 > 2) added columns for the Video fields in the main display (MEDIA_TYPE
 > not editable as Hex, since only some values are valid)
 > 3) removed function to convert a string to media type, since in is not
 > safe (the same as 2)

I agree with you. I'm looking forward to your patches once you find out 
more about the MEDIATYPE definitions :-)

Cheers,



JCS.