On Thu, Feb 25, 2016 at 09:03:05AM -0500, John Ferlan wrote: > Rather than having it interspersed with other changes, do it once. > > Remove a couple ^L, 1 argument per line for functions, less than 80 chars > per line, use of spacing between logical groups of code, use of one line > if statements when doing fetch followed by comparison, use direct return > when no cleanup to be done. > > Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx> > --- > src/secret/secret_driver.c | 146 ++++++++++++++++++++++++--------------------- > 1 file changed, 77 insertions(+), 69 deletions(-) ACK Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list