From kde-multimedia Thu Aug 26 12:57:11 2004 From: Scott Wheeler Date: Thu, 26 Aug 2004 12:57:11 +0000 To: kde-multimedia Subject: Re: kfile theora plugin Message-Id: <200408261457.11279.wheeler () kde ! org> X-MARC-Message: https://marc.info/?l=kde-multimedia&m=109352504613320 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