[libvirt] indentation (was: [PATCH 11/12] vcpu: complete vcpu support in qemu driver)

Daniel Veillard veillard at redhat.com
Wed Oct 6 16:26:17 UTC 2010


On Fri, Oct 01, 2010 at 02:21:05PM -0600, Eric Blake wrote:
> On 10/01/2010 09:31 AM, Daniel Veillard wrote:
> >>-    ret = qemudDomainHotplugVcpus(vm, nvcpus);
> >>+    switch (flags) {
> >>+    case VIR_DOMAIN_VCPU_MAXIMUM | VIR_DOMAIN_VCPU_PERSISTENT:
> >
> >  hum I usually indent the cases 4 spaces from the switch, minor nit
> 
> I just used what emacs gave by default, after applying the suggested
> ~/.emacs settings from HACKING.  In that mode, emacs puts the labels
> flush with the switch.  The libvirt code base has a decent mix of
> both styles, but favors my style over yours (don't ask me how much
> time I wasted trying to figure out that sed script):
> 
> $ git grep -A1 switch src \
>  | sed -n 's/:/-/;/switch/{N;/^\(.*\)switch.*\n\1case/p}' | wc -l
> 350
> $ git grep -A1 switch src \
>  | sed -n 's/:/-/;/switch/{N;/^\(.*\)switch.*\n\1  *case/p}' | wc -l
> 96
> 
> Speaking of indentation nits, are we any closer to Jim's idea of
> using an automated indent-like tool for setting on a consistent
> formatting style?

  Well if the editor/typing habits and the SCM checking rules start to
conflict it can get really messy I'm afraid, but maybe I don't remember
the proposal well.

  What I usuall do when I want to reformat a piece of code is run it
through indent:

indent -bad -bap -bbb -bli4 -br -ce -brs -cs -i4 -l75 -lc75 -nut -sbi4
-psl -saf -sai -saw -sbi4 -ss -sc -cdw -cli4 -npcs -nbc

and in general it does what I'm used to, but I often have to clean the
thing up a bit still.

  Automating it may be a bit painful, what you commit is not what you
have in the patch, I'm a bit vary of that...

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list