[LTP] [PATCH] cpuset: Work around a clang compiler warning

Alistair Strachan astrachan@google.com
Tue Jun 26 18:45:52 CEST 2018


The clang compiler will warn about pasting a C string literal and an
int, suggesting that array indexing should be used instead:

testcases/kernel/controllers/cpuset/cpuset_lib/cpuinfo.c:143:3: error:
 adding 'int' to a string does not append to the string
 [-Werror,-Wstring-plus-int]
  while_each_childdir(SYS_CPU_DIR, "/", c_relpath,
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
testcases/kernel/controllers/cpuset/cpuset_lib/common.h:54:13: note:
 expanded from macro 'while_each_childdir'
  p_relpath + start); \
  ~~~~~~~~~~^~~~~~~
testcases/kernel/controllers/cpuset/cpuset_lib/cpuinfo.c:143:3: note:
 use array indexing to silence this warning

This seems like a strange warning, but the suggestion has no effect on
code generation or readability, so change it to keep clang happy.

Signed-off-by: Alistair Strachan <astrachan@google.com>
---
 testcases/kernel/controllers/cpuset/cpuset_lib/common.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testcases/kernel/controllers/cpuset/cpuset_lib/common.h b/testcases/kernel/controllers/cpuset/cpuset_lib/common.h
index bd347413c..28e948e63 100644
--- a/testcases/kernel/controllers/cpuset/cpuset_lib/common.h
+++ b/testcases/kernel/controllers/cpuset/cpuset_lib/common.h
@@ -51,7 +51,7 @@
 		start = 1;						\
 									\
 	snprintf(fullpath, sizeof(fullpath), "%s%s", basepath,		\
-		p_relpath + start);					\
+		 &p_relpath[start]);					\
 	pathlen = strlen(fullpath);					\
 									\
 	if ((dp = opendir(fullpath)) == NULL)				\


More information about the ltp mailing list