Fix JPluginLoader warnings

JeffersonLab · Aug 16, 2024 · 93b91ba · 93b91ba
1 parent cdf6f6c
commit 93b91ba
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/libraries/JANA/Services/JPluginLoader.cc b/src/libraries/JANA/Services/JPluginLoader.cc
@@ -116,7 +116,7 @@ void JPluginLoader::attach_plugins(JComponentManager* jcm) {
 
     // Loop over all requested plugins as per the `plugins` parameter. Note that this vector may grow as 
     // plugins themselves request additional plugins. These get appended to the back of `m_plugins_to_include`.
-    for (int i=0; i<m_plugins_to_include.size(); ++i) {
+    for (size_t i=0; i<m_plugins_to_include.size(); ++i) {
         std::ostringstream paths_checked;
         const std::string& user_plugin = m_plugins_to_include[i];
 
@@ -263,7 +263,7 @@ JPlugin::~JPlugin() {
 
 
 bool JPluginLoader::is_path(const std::string user_plugin) {
-    return user_plugin.find('/') != -1;
+    return user_plugin.find('/') != std::string::npos;
 }
 
 std::string JPluginLoader::normalize_name(const std::string user_plugin) {