iox-eclipse-iceoryx#1196 More readable statement for valid paths with ending dots check

elfenpiff · elfenpiff · commit a78037f71c97 · 2022-07-07T09:19:18.000+02:00
Signed-off-by: Christian Eltzschig &lt;christian.eltzschig@apex.ai&gt;
diff --git a/iceoryx_hoofs/include/iceoryx_hoofs/internal/cxx/helplets.inl b/iceoryx_hoofs/include/iceoryx_hoofs/internal/cxx/helplets.inl
@@ -52,8 +52,13 @@ inline bool isValidPathEntry(const string<StringCapacity>& name,
         }
     }
 
+    if (nameSize == 0)
+    {
+        return true;
+    }
+
     // dot at the end is invalid to be compatible with windows api
-    return !(nameSize != 0 && name[nameSize - 1] == '.');
+    return !(name[nameSize - 1] == '.');
 }
 
 template <uint64_t StringCapacity>

Original file line number	Diff line number	Diff line change
`@@ -52,8 +52,13 @@ inline bool isValidPathEntry(const string<StringCapacity>& name,`
`52`	`52`	`}`
`53`	`53`	`}`
`54`	`54`
	`55`	`+ if (nameSize == 0)`
	`56`	`+ {`
	`57`	`+ return true;`
	`58`	`+ }`
	`59`	`+`
`55`	`60`	`// dot at the end is invalid to be compatible with windows api`
`56`		`- return !(nameSize != 0 && name[nameSize - 1] == '.');`
	`61`	`+ return !(name[nameSize - 1] == '.');`
`57`	`62`	`}`
`58`	`63`
`59`	`64`	`template <uint64_t StringCapacity>`