There were several occurrences of an extra space inserted between a function name and the ( opening the argument list in datatypes.c. This is not consistent with the coding style used in the rest of this file so removing this extra space makes the code slightly more readable. --- src/datatypes.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/datatypes.c b/src/datatypes.c index 7cc37c1..1b68f6a 100644 --- a/src/datatypes.c +++ b/src/datatypes.c @@ -243,19 +243,19 @@ virReleaseConnect(virConnectPtr conn) { virMutexUnlock(&conn->lock); if (conn->networkDriver) - conn->networkDriver->close (conn); + conn->networkDriver->close(conn); if (conn->interfaceDriver) - conn->interfaceDriver->close (conn); + conn->interfaceDriver->close(conn); if (conn->storageDriver) - conn->storageDriver->close (conn); + conn->storageDriver->close(conn); if (conn->deviceMonitor) - conn->deviceMonitor->close (conn); + conn->deviceMonitor->close(conn); if (conn->secretDriver) - conn->secretDriver->close (conn); + conn->secretDriver->close(conn); if (conn->nwfilterDriver) - conn->nwfilterDriver->close (conn); + conn->nwfilterDriver->close(conn); if (conn->driver) - conn->driver->close (conn); + conn->driver->close(conn); virMutexLock(&conn->lock); -- 1.7.4

2011/2/18 Eric Blake <eblake(a)redhat.com&gt;: I tried to use uncrustify [1]. It has tones of options to configure the style. I tweaked it's default config file to match libvirt's style. Works quite well, but there are still some issues left with how it breaks certain long functions calls and if statements. I attached my config so you can give it a try if you like :) [1] http://uncrustify.sourceforge.net/

On 02/17/2011 02:18 PM, Christophe Fergeau wrote: ACK and pushed. -- Eric Blake eblake(a)redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org