[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] bye to close(), welcome to VIR_(FORCE_)CLOSE()

On 10/22/2010 05:19 AM, Stefan Berger wrote:
Using automated replacement with sed and editing I have now replaced all
occurrences of close() with VIR_(FORCE_)CLOSE() except for one, of
course. Some replacements were straight forward, others I needed to pay
attention. I hope I payed attention in all the right places... Please
have a look. This should have at least solved one more double-close

Signed-off-by: Stefan Berger<stefanb us ibm com>

Pre-review comment:
One more thing to do, which is adding something like this to cfg.mk:

        @prohibit='\<close *\('                                        \
        in_vc_files='\.c$$'                                            \
        halt='use VIR_(FORCE_)CLOSE instead of close'                  \

along with adding .x-sc_avoid_close with contents of src/util/file.c to exempt the one valid use of close().

Would you mind testing 'make syntax-check' after folding that in, and addressing any other fallout that it detects, while I proceed with the rest of my review?

Eric Blake   eblake redhat com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]