Amended merging upstream/master into subcommand

Amended #c66b20247858
catchorg · Jan 29, 2024 · 699a6f1 · 699a6f1
1 parent c66b202
commit 699a6f1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/include/clara.hpp b/include/clara.hpp
@@ -116,7 +116,9 @@ namespace detail {
         void loadBuffer() {
             m_tokenBuffer.resize( 0 );
 
-            // Note: not skiping empty strings
+            // Note: not skipping empty strings
+            while (it != itEnd && it->empty())
+                ++it;
 
             if( it != itEnd ) {
                 auto const &next = *it;
@@ -979,9 +981,6 @@ namespace detail {
                 }
             }
 
-            struct ParserInfo {
-                ParserBase const* parser = nullptr;
-            };
             const size_t totalParsers = m_options.size() + m_args.size();
             assert( totalParsers < 512 );
             // ParserInfo parseInfos[totalParsers]; // <-- this is what we really want to do
@@ -1007,7 +1006,8 @@ namespace detail {
             while( result.value().remainingTokens() ) {
                 bool tokenParsed = false;
 
-                for (auto& parser : parsers) {
+                for (size_t i = 0; i < totalParsers; ++i) {
+                    ParserBase const* parser = parsers[i];
                     if( parser->canParse() ) {
                         result = parser->internalParse(exeName, result.value().remainingTokens());
                         if (!result)

diff --git a/single_include/clara.hpp b/single_include/clara.hpp
@@ -456,7 +456,9 @@ namespace detail {
         void loadBuffer() {
             m_tokenBuffer.resize( 0 );
 
-            // Note: not skiping empty strings
+            // Note: not skipping empty strings
+            while (it != itEnd && it->empty())
+                ++it;
 
             if( it != itEnd ) {
                 auto const &next = *it;
@@ -1319,9 +1321,6 @@ namespace detail {
                 }
             }
 
-            struct ParserInfo {
-                ParserBase const* parser = nullptr;
-            };
             const size_t totalParsers = m_options.size() + m_args.size();
             assert( totalParsers < 512 );
             // ParserInfo parseInfos[totalParsers]; // <-- this is what we really want to do
@@ -1347,7 +1346,8 @@ namespace detail {
             while( result.value().remainingTokens() ) {
                 bool tokenParsed = false;
 
-                for (auto& parser : parsers) {
+                for (size_t i = 0; i < totalParsers; ++i) {
+                    ParserBase const* parser = parsers[i];
                     if( parser->canParse() ) {
                         result = parser->internalParse(exeName, result.value().remainingTokens());
                         if (!result)