Jason Tackaberry wrote:
> Dirk Meyer wrote:
>> There was a reason why the flac parser was at the end because it had
>> some false positive detections. We need some extra attribute about the
>> rating of a parser to sort them when checking all possible parser.
>>   
>
> If a parser has a problem with false positives (like the mp3 parser
> did), then we should fix the parser to not be so trigger happy (like I
> did with the mp3 parser) rather than kludge things by fiddling with the
> order.

Yes, in a perfect world we should do that :) But some parser /
standards are very vague and easy to match.

One other idea:
| register('video/avi', ('wav','avi'), 'video.riff.Riff')

What about naming all parser classes Parser() just like Freevo names
all plugins PluginInterface. In that case we would

| register('video/avi', ('wav','avi'), 'video.riff')

Which looks better IMHO.

> But in the meantime I will restore the order of register calls.

Thanks.


Dischi

-- 
(null cookie; hope that's ok)

Attachment: pgpJtg7DU4M16.pgp
Description: PGP signature

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Freevo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/freevo-devel

Reply via email to