On 01/29/2016 05:43 PM, Christoph Grüninger wrote: > please find attached a minimal patch to fix a GCC 6 pedantic warning > concerning a superfluous semicolon after a method.
Applied, thanks: cmAlgorithms.h: remove superfluous semicolon after method https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=ccb2d5c0 > 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. There is no reason to have negative values for that, so please update both cmLinkInterface::Multiplicity and cmGeneratorTarget::Multiplicity to use `unsigned int` and follow through with any further changes needed for that (I don't think there are any but have not checked). Thanks, -Brad -- 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
