<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Oct 15, 2015 at 1:18 PM, Roland McGrath <span dir="ltr">&lt;<a href="mailto:roland@hack.frob.com" target="_blank">roland@hack.frob.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">&gt; +inline static int<br>
<br>
static inline int</blockquote><div><br></div><div>Fixed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
&gt; +file_matches (Dwarf_Files * files, size_t idx,<br>
<br>
</span>Gratuitous space after * there.<br>
<span class=""><br></span></blockquote><div><br></div><div>I found most other places do not have space after &quot;*&quot;.</div><div>So I removed the extra space here to be consistent. :-)</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
&gt; +              const char *lastfile, size_t match_file_len,<br>
&gt; +              const char *match_file, bool *lastfile_matches)<br>
<br>
</span>When what should be a local function has several parameters like this to<br>
replace what should be the closed-over variables, I think it&#39;s clearer to<br>
put all those parameters first, before the real parameters to the function.<br>
<br></blockquote><div><br></div><div>Changed, to have input read-only parameter first, followed by old parameters and then output parameters.</div><div><br></div><div>Please review the new attached 0002*patch file again.</div><div>Thanks.</div><div><br></div></div><br></div></div>