[PATCH] virterror: supply missing spaces

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* 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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]