virsh: Add helper to avoid the strcase check for virsh-*.c

* tools/virsh.c: New macro vshStrcasecmp
* tools/virsh-domain-monitor.c: Use vshStrcasecmp instead of
                                strcasecmp
* tools/virsh-snapshot.c: Likewise
* cfg.mk: Only avoid doing strcase checking for virsh.c

diff --git a/cfg.mk b/cfg.mk
index c0457e7fb8dc8a450b440a224818dcbcd736f246..1318593b511312c27973eae8e821d129affbb47f 100644 (file)
--- a/cfg.mk
+++ b/cfg.mk
@@ -727,7 +727,7 @@ $(srcdir)/src/remote/remote_client_bodies.h: $(srcdir)/src/remote/remote_protoco
        $(MAKE) -C src remote/remote_client_bodies.h
 
 # List all syntax-check exemptions:
-exclude_file_name_regexp--sc_avoid_strcase = ^tools/virsh(-domain-monitor|-snapshot)?\.c$$
+exclude_file_name_regexp--sc_avoid_strcase = ^tools/virsh\.c$$
 
 _src1=libvirt|fdstream|qemu/qemu_monitor|util/(command|util)|xen/xend_internal|rpc/virnetsocket|lxc/lxc_controller
 exclude_file_name_regexp--sc_avoid_write = \

diff --git a/tools/virsh-domain-monitor.c b/tools/virsh-domain-monitor.c
index b57c39074c0217adf13390ac52b2dc4c911d0afe..1272a0575c1561fafa49815f30d5a16769ecb3a1 100644 (file)
--- a/tools/virsh-domain-monitor.c
+++ b/tools/virsh-domain-monitor.c
@@ -1273,7 +1273,7 @@ vshDomainSorter(const void *a, const void *b)
     idb = virDomainGetID(*db);
 
     if (ida == inactive && idb == inactive)
-        return strcasecmp(virDomainGetName(*da), virDomainGetName(*db));
+        return vshStrcasecmp(virDomainGetName(*da), virDomainGetName(*db));
 
     if (ida != inactive && idb != inactive) {
         if (ida > idb)

diff --git a/tools/virsh-snapshot.c b/tools/virsh-snapshot.c
index d6de3da57c6d92de30e3ef7e8687f6227af879a5..24e44b026ccd36e6eecb5b8cea22a60f5b69fe7c 100644 (file)
--- a/tools/virsh-snapshot.c
+++ b/tools/virsh-snapshot.c
@@ -816,9 +816,8 @@ vshSnapSorter(const void *a, const void *b)
     if (!sa->snap)
         return sb->snap != NULL;
 
-    /* User visible sort, so we want locale-specific case comparison.  */
-    return strcasecmp(virDomainSnapshotGetName(sa->snap),
-                      virDomainSnapshotGetName(sb->snap));
+    return vshStrcasecmp(virDomainSnapshotGetName(sa->snap),
+                         virDomainSnapshotGetName(sb->snap));
 }
 
 /* Compute a list of snapshots from DOM.  If FROM is provided, the

diff --git a/tools/virsh.c b/tools/virsh.c
index 303bdd4930bc3d72849d3efb4bafd5033d53986e..4dff02ea658d90e714e38fe41e94330dd2cc864e 100644 (file)
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -357,6 +357,9 @@ static void vshDebug(vshControl *ctl, int level, const char *format, ...)
 /* XXX: add batch support */
 #define vshPrint(_ctl, ...)   vshPrintExtra(NULL, __VA_ARGS__)
 
+/* User visible sort, so we want locale-specific case comparison.  */
+#define vshStrcasecmp(S1, S2) strcasecmp(S1, S2)
+
 static int vshDomainState(vshControl *ctl, virDomainPtr dom, int *reason);
 static const char *vshDomainStateToString(int state);
 static const char *vshDomainStateReasonToString(int state, int reason);
@@ -462,8 +465,7 @@ vshNameSorter(const void *a, const void *b)
     const char **sa = (const char**)a;
     const char **sb = (const char**)b;
 
-    /* User visible sort, so we want locale-specific case comparison. */
-    return strcasecmp(*sa, *sb);
+    return vshStrcasecmp(*sa, *sb);
 }
 
 static double