[libvirt] [PATCH] virterror: supply missing spaces
* src/util/virterror.c (virErrorDomainName): Always supply
trailing space if category was present.
---
> > > +++ b/src/util/virterror.c
> > > @@ -190,6 +190,9 @@ static const char *virErrorDomainName(virErrorDomain
domain) {
> > > case VIR_FROM_AUDIT:
> > > dom = "Audit";
> > > break;
> > > + case VIR_FROM_STREAMS:
> > > + dom = "Streams ";
>
> In just this context, I wondered why the trailing space? Then looking
> at the entire file, I instead wonder: why are VIR_FROM_NWFILTER and
> VIR_FROM_AUDIT the only ones that lack trailing space?
Looks like a bug to me.
Then I'm pushing this under the trivial rule (okay, it's a bit of a
stretch, since the whitespace is significant as part of a string literal
rather than insignificant as in indentation, but I didn't think
anyone would mind :)
src/util/virterror.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/util/virterror.c b/src/util/virterror.c
index 70749a7..dc08aaa 100644
--- a/src/util/virterror.c
+++ b/src/util/virterror.c
@@ -179,7 +179,7 @@ static const char *virErrorDomainName(virErrorDomain domain) {
dom = "CPU ";
break;
case VIR_FROM_NWFILTER:
- dom = "Network Filter";
+ dom = "Network Filter ";
break;
case VIR_FROM_HOOK:
dom = "Sync Hook ";
@@ -188,7 +188,7 @@ static const char *virErrorDomainName(virErrorDomain domain) {
dom = "Domain Snapshot ";
break;
case VIR_FROM_AUDIT:
- dom = "Audit";
+ dom = "Audit ";
break;
}
return(dom);
--
1.7.2.3