r2914 broke metadata brute force run

Hi Duncan,

I like the new magic stuff in r2914, but unfortunately it
broke the old code. It will now never proceed done to the
brute force code path, so lots and lots of files stopped
getting metadata info. I think you need to restructure
things a bit or add some boolean so that you are sure you
got at least one hit before deciding to abort early.

Rgds
-- 
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org

------------------------------------------------------------
-------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Freevo-devel mailing list
Freevo-devellists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-dev
el

Pierre Ossman wrote:
> Hi Duncan,
>
> I like the new magic stuff in r2914, but unfortunately
it broke the old code. It will now never proceed done to the
brute force code path, so lots and lots of files stopped
getting metadata info. I think you need to restructure
things a bit or add some boolean so that you are sure you
got at least one hit before deciding to abort early.
>   

Can you explain this a bit more, you are referring to
kaa.metadata but I
don't quite see what you mean in the last sentence.

Duncan


------------------------------------------------------------
-------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Freevo-devel mailing list
Freevo-devellists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-dev
el

Pierre Ossman wrote:
> Hi Duncan,
>
> I like the new magic stuff in r2914, but unfortunately
it broke the
> old code. It will now never proceed done to the brute
force code
> path, so lots and lots of files stopped getting
metadata info. I
> think you need to restructure things a bit or add some
boolean so
> that you are sure you got at least one hit before
deciding to abort
> early.

Oops, the indent was wrong. Can you update kaa.metadata
again and let
me know if it works now. If it does, we need a metadata
0.7.1
release.


Dischi

=2D-=20
A 14.4 modem makes you want to get out and push!

------------------------------------------------------------
-------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Freevo-devel mailing list
Freevo-devellists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-dev
el

Hi Dirk,

I've turned of delivery of freevo-devel since I don't have
time to actively participate, so a cc for any replies is
appreciated. 

I've tested your update and it solves the problem here. So a
new kaa release is probably in order. Without the fix it
doesn't parse .vob files and there doesn't enable
deinterlace filters.

Rgds
-- 
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org

------------------------------------------------------------
-------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Freevo-devel mailing list
Freevo-devellists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-dev
el

Pierre Ossman wrote:
> Hi Dirk,
>
> I've tested your update and it solves the problem here.
So a new kaa
> release is probably in order. Without the fix it
doesn't parse .vob
> files and there doesn't enable deinterlace filters.

And 0.7.1 is out!


Dischi

=2D-=20
I pretend to work. They pretend to pay me.

------------------------------------------------------------
-------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Freevo-devel mailing list
Freevo-devellists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-dev
el