<p>On 16/05/17 19:47, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
#define IF_EXTRACT_FMT(txt,var,fmt,target)<br />
- IF_EXTRACT(txt,var)<br />
- if( fmt && !strncasecmp(psz_comment, txt, strlen(txt)) )<br />
+ if( !strncasecmp(psz_comment, txt, strlen(txt)) )<br />
+ {<br />
+ IF_EXTRACT(txt,var)<br />
+ if( fmt )<br />
{<br />
if ( fmt->target ) free( fmt->target );<br />

</blockquote>
<p>Remove the if() while you´re at it.</p>
</blockquote>
<p>Well-spotted (I was focusing on fixing the immediate issue), thanks!</p>
<p>See the attached patch on this email (which also removes the redundant check to see whether <code>fmt->target</code> is not null).</p>