<!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-01-10 14:58, Francois Cartegnie wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le 10/01/2017 à 13:52, 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> There's no need for a temporary local variable and the do-while
 wrapping to limit its scope, instead these changes uses a
 compound-literal.</code></pre>
</blockquote>
<pre><code> Unless you enable the #ifdef section above.</code></pre>
</blockquote>
<p>Could you elaborate on that? I might be a little tired, but I cannot see how the conditional definition of <code>EPG_DEBUG</code> changes anything regarding the patch in question.</p>
<p>Best Regards, Filip</p>
</body>
</html>