Well, mailman-2.1.5 mangled yet another of my messages. [ I know why, and have reported the bug. Maybe we'll get a fixed/newer version of mailman. ] But at least the patch got through. Here's the part that it removed: Here's a tiny patch to fix indentation, since I'm about to change the code in that area, I'm submitting this first, so we don't mix syntax-only changes with ones that change semantics. The last two times I tried to send an attachment, mailman removed them. I'll try one more time, with this one: Jim Meyering <meyering@xxxxxxxxxx> wrote: > Signed-off-by: Jim Meyering <meyering@xxxxxxxxxx> > --- > ChangeLog | 4 ++++ > src/virsh.c | 10 +++++----- > 2 files changed, 9 insertions(+), 5 deletions(-) > > diff --git a/ChangeLog b/ChangeLog > index ce68401..a6de485 100644 > --- a/ChangeLog > +++ b/ChangeLog > @@ -1,3 +1,7 @@ > +2007-11-08 Jim Meyering <meyering@xxxxxxxxxx> > + > + * src/virsh.c (vshCommandOptInt): Correct indentation > + > Wed Nov 7 14:38:43 CET 2007 Daniel Veillard <veillard@xxxxxxxxxx> > > * docs/libvir.html docs/site.xsl: small updates with references > diff --git a/src/virsh.c b/src/virsh.c > index 0615a7d..852dea2 100644 > --- a/src/virsh.c > +++ b/src/virsh.c > @@ -1,7 +1,7 @@ > /* > * virsh.c: a Xen shell used to exercise the libvirt API > * > - * Copyright (C) 2005 Red Hat, Inc. > + * Copyright (C) 2005, 2007 Red Hat, Inc. > * > * See COPYING.LIB for the License of this software > * > @@ -3956,10 +3956,10 @@ vshCommandOptInt(vshCmd * cmd, const char *name, int *found) > > if ((arg != NULL) && (arg->data != NULL)) { > res = strtol(arg->data, &end_p, 10); > - if ((arg->data == end_p) || (*end_p!= 0)) > - num_found = FALSE; > - else > - num_found = TRUE; > + if ((arg->data == end_p) || (*end_p!= 0)) > + num_found = FALSE; > + else > + num_found = TRUE; > } > if (found) > *found = num_found; -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list