Merge pull request #351 from gazebosim/scpeters/merge_9_main

Merge gz-msgs9 ➡️ main
gazebosim · Jun 21, 2023 · 3c62ef7 · 3c62ef7
2 parents df0cd36 + ccfbba9
commit 3c62ef7
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -66,9 +66,7 @@ message(STATUS "\n\n-- ====== Finding Dependencies ======")
 
 #--------------------------------------
 # Find Protobuf
-set(REQ_PROTOBUF_VER 3)
 gz_find_package(GzProtobuf
-                 VERSION ${REQ_PROTOBUF_VER}
                  REQUIRED
                  COMPONENTS all
                  PRETTY Protobuf)

diff --git a/src/Generator.cc b/src/Generator.cc
@@ -149,7 +149,7 @@ bool Generator::Generate(const FileDescriptor *_file,
 
     auto ns = getNamespaces(_file->package());
 
-    for (const auto name : ns)
+    for (const auto &name : ns)
         printer.PrintRaw("namespace " + name + " {\n");
 
     for (auto i = 0; i < _file->message_type_count(); ++i)