<!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 2017-03-20 14:37, Francois Cartegnie wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le 20/03/2017 à 14:26, 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> A switch should be easier to maintain, while also improving
readability.</code></pre>
</blockquote>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> + case 0x01: /* closed caption */
+ case 0x02: /* XDS */
+ DemuxRecCc( p_demux, p_rec, p_block_in );
+ break;</code></pre>
</blockquote>
<pre><code> Misleading incorrect attribution of comments to single values.</code></pre>
</blockquote>
<p>Thank you for pointing that out, I misread the previous comments and thought they applied to each of the two values seperately (which I know understand is of course not the case).</p>
<p>See attached patch for a fixup (since the patch got applied while you were replying).</p>
<p>All the best,<br />
Filip</p>
</body>
</html>