Dear CMake devs,
please find attached a minimal patch to fix a GCC 6 pedantic warning
concerning a superfluous semicolon after a method.

Additional I got the following warning:
> CMake/Source/cmGeneratorTarget.cxx:4835:52: warning: format ‘%u’ expects 
> argument of type ‘unsigned int*’, but argument 3 has type ‘int*’ [-Wformat=]
>        sscanf(config_reps, "%u", &iface.Multiplicity);

Same on 4840:45, 5543:51, 5548:44 in the same file. It is not obvious to
me whether you want to change the types in cmOptionalLinkInterface,
change the type in sscanf or cast the types.

Bye
Christoph
>From be4d8226d4f34d0cbf18b1a08b0e187e577edc38 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christoph=20Gr=C3=BCninger?= <f...@grueninger.de>
Date: Fri, 29 Jan 2016 22:16:03 +0100
Subject: [PATCH] cmAlgorithms.h: remove superfluous semicolon after method

---
 Source/cmAlgorithms.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Source/cmAlgorithms.h b/Source/cmAlgorithms.h
index 54617f3..bb65ea5 100644
--- a/Source/cmAlgorithms.h
+++ b/Source/cmAlgorithms.h
@@ -230,7 +230,7 @@ template<typename Range>
 std::string cmJoin(Range const& r, std::string delimiter)
 {
   return cmJoin(r, delimiter.c_str());
-};
+}
 
 template<typename Range>
 typename Range::const_iterator cmRemoveN(Range& r, size_t n)
-- 
2.6.2

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Reply via email to