commit c3d3f07c2551caaf283e8c2edb4012c5a61b7793
author Jamie Madill <jmadill@chromium.org> Wed Dec 14 17:21:43 2016 -0500
committer Mark Young <marky@lunarg.com> Thu Dec 15 13:42:01 2016 -0700
tree 999ef4d346e3dbf339e82ea44112141a02b82bf8
parent 6a27de584e5ca503509bbbd02caf5db020afb8ce

loader: Fix Clang warnings.

error: equality comparison with extraneous parentheses
error: comparison of array equal to a null pointer is always false
error: using namespace directive in global context in header

Change-Id: I52b5d9062653b9429b9c42b40eca54e71b6d794b

loader/loader.c

diff --git a/loader/loader.c b/loader/loader.c
index 85b5f48..2ed52c4 100644
--- a/loader/loader.c
+++ b/loader/loader.c
@@ -3799,8 +3799,7 @@
                 continue;
             if ((fpGIPA = layer_prop->functions.get_instance_proc_addr) ==
                 NULL) {
-                if (layer_prop->functions.str_gipa == NULL ||
-                    strlen(layer_prop->functions.str_gipa) == 0) {
+                if (strlen(layer_prop->functions.str_gipa) == 0) {
                     fpGIPA = (PFN_vkGetInstanceProcAddr)
                         loader_platform_get_proc_address(
                             lib_handle, "vkGetInstanceProcAddr");
@@ -3914,8 +3913,7 @@
                 continue;
             if ((fpGIPA = layer_prop->functions.get_instance_proc_addr) ==
                 NULL) {
-                if (layer_prop->functions.str_gipa == NULL ||
-                    strlen(layer_prop->functions.str_gipa) == 0) {
+                if (strlen(layer_prop->functions.str_gipa) == 0) {
                     fpGIPA = (PFN_vkGetInstanceProcAddr)
                         loader_platform_get_proc_address(
                             lib_handle, "vkGetInstanceProcAddr");
@@ -3933,8 +3931,7 @@
                 }
             }
             if ((fpGDPA = layer_prop->functions.get_device_proc_addr) == NULL) {
-                if (layer_prop->functions.str_gdpa == NULL ||
-                    strlen(layer_prop->functions.str_gdpa) == 0) {
+                if (strlen(layer_prop->functions.str_gdpa) == 0) {
                     fpGDPA = (PFN_vkGetDeviceProcAddr)
                         loader_platform_get_proc_address(lib_handle,
                                                          "vkGetDeviceProcAddr");
@@ -4464,7 +4461,7 @@
         }
         res = icd_term->EnumeratePhysicalDevices(
             icd_term->instance, &(phys_devs[i].count), phys_devs[i].phys_devs);
-        if ((res == VK_SUCCESS)) {
+        if (res == VK_SUCCESS) {
             inst->total_gpu_count += phys_devs[i].count;
         } else {
             return res;