[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-multimedia
Subject:    Re: kfile theora plugin
From:       Scott Wheeler <wheeler () kde ! org>
Date:       2004-08-26 12:57:11
Message-ID: 200408261457.11279.wheeler () kde ! org
[Download RAW message or body]

On Thursday 26 August 2004 16:16, bj@altern.org wrote:
> Hello!
> 
> I recently wrote a kfile plugin for theora encoded video files that displays 
> meta data. (downloadable at 
> http://kde-apps.org/content/show.php?content=15553). 
> 
> I think it would be a good addition to kdemultimedia/kfile-plugins/
> What do you think ?

Sure -- I'd like to see it in CVS.  Please feel free to import it.  If you 
don't have a CVS account let me know and we can take care of such.

It looks like the code could use a bit of cleaning -- specifically I'd 
personally appreciate more verbose variable naming; it also seems like you 
switch the style of using spaces a few times in the code.  It'd be nice to 
see these things cleaned up either before or shortly after going in.

-Scott
_______________________________________________
kde-multimedia mailing list
kde-multimedia@kde.org
https://mail.kde.org/mailman/listinfo/kde-multimedia
[prev in list] [next in list] [prev in thread] [next in thread]