<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<meta http-equiv="Content-Style-Type" content="text/css" />
<meta name="generator" content="pandoc" />
<title></title>
<style type="text/css">code{white-space: pre;}</style>
</head>
<body>
<p>Hi Francois,</p>
<p>On 2016-10-25 17:59, Francois Cartegnie wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le 25/10/2016 à 17:31, Filip Roséen a écrit :</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Hi Francois,
On 2016-10-25 17:18, Francois Cartegnie wrote:</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le 25/10/2016 à 06:56, Tristan Matthews a écrit :</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> + /* Probe for XOR'd ID3 tag. */
+ if( memcmp( peek, "\x6B\x66\x11", 3 ) )
+ return VLC_EGENERIC;</code></pre>
</blockquote>
<pre><code> Being before any demuxer and packetizer,
expect high rate of false positives.</code></pre>
</blockquote>
<pre><code> - Why would this cause a *"high rate of false positives"?
It is equivalent to a *demuxer* probing for the same data, and as far
as I could see when I looked up the *magic-bytes* used, there is no
*file-format* that is publically documented that uses that marker.</code></pre>
</blockquote>
<pre><code> Probing a single sequence is *weak*.
and data has no alignment guarantee.
Hence the demuxers priorities and the packetizer based demuxers and
multiple frames probing for most audio codecs. (and libav has even more
intensive probing)
Putting this as stream filter gives it higher priority than any demuxer.
Will break any non aligned, sync code based, if the stream starts with
"\x6B\x66\x11" (raw video, raw audio, ts, ps...), where demux with low
score would not.</code></pre>
</blockquote>
<p>That is a very valid point, thank you for the elaboration!</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Francois
_______________________________________________
vlc-devel mailing list
To unsubscribe or modify your subscription options:
https://mailman.videolan.org/listinfo/vlc-devel</code></pre>
</blockquote>
</body>
</html>