<br><tt><font size=2>libvir-list-bounces@redhat.com wrote on 10/22/2010
12:33:48 PM:<br>
<br>
</font></tt>
<br><tt><font size=2>> <br>
> On 10/22/2010 05:19 AM, Stefan Berger wrote:<br>
> > Using automated replacement with sed and editing I have now replaced
all<br>
> > occurrences of close() with VIR_(FORCE_)CLOSE() except for one,
of<br>
> > course. Some replacements were straight forward, others I needed
to pay<br>
> > attention. I hope I payed attention in all the right places...
Please<br>
> > have a look. This should have at least solved one more double-close<br>
> > error.<br>
> ><br>
> > Signed-off-by: Stefan Berger<stefanb@us.ibm.com><br>
> <br>
> Pre-review comment:<br>
> One more thing to do, which is adding something like this to cfg.mk:<br>
> <br>
> sc_avoid_close:<br>
>          @prohibit='\<close *\('  
                     
               \<br>
>          in_vc_files='\.c$$'    
                     
                 \<br>
>          halt='use VIR_(FORCE_)CLOSE instead
of close'                  \<br>
>            $(_sc_search_regexp)<br>
> <br>
> along with adding .x-sc_avoid_close with contents of src/util/file.c
to <br>
> exempt the one valid use of close().<br>
> <br>
> Would you mind testing 'make syntax-check' after folding that in,
and <br>
> addressing any other fallout that it detects, while I proceed with
the <br>
> rest of my review?</font></tt>
<br>
<br><tt><font size=2>Yes, I can do that... runs fine.</font></tt>
<br>
<br><tt><font size=2>The patch will need to be rebased to tip...</font></tt>
<br>
<br><tt><font size=2>   Stefan</font></tt>
<br>