Fix: Remove use of bsl::next for Solaris compatibility (#1183)

pniedzielski · web-flow · commit 896a80ee8b0e · 2026-03-06T16:42:21.000-05:00
Signed-off-by: Patrick M. Niedzielski &lt;pniedzielski@bloomberg.net&gt;
diff --git a/src/groups/mqb/mqbauthn/mqbauthn_authenticationcontroller.cpp b/src/groups/mqb/mqbauthn/mqbauthn_authenticationcontroller.cpp
@@ -152,10 +152,10 @@ int AuthenticationController::initializeAuthenticators(
         for (AuthenticatorMap::const_iterator it = d_authenticators.cbegin();
              it != d_authenticators.cend();
              ++it) {
-            BALL_LOG_OUTPUT_STREAM << it->first;
-            if (bsl::next(it) != d_authenticators.cend()) {
+            if (it != d_authenticators.cbegin()) {
                 BALL_LOG_OUTPUT_STREAM << ", ";
             }
+            BALL_LOG_OUTPUT_STREAM << it->first;
         }
         BALL_LOG_OUTPUT_STREAM << "]";
     }

Original file line number	Diff line number	Diff line change
`@@ -152,10 +152,10 @@ int AuthenticationController::initializeAuthenticators(`
`152`	`152`	`for (AuthenticatorMap::const_iterator it = d_authenticators.cbegin();`
`153`	`153`	`it != d_authenticators.cend();`
`154`	`154`	`++it) {`
`155`		`- BALL_LOG_OUTPUT_STREAM << it->first;`
`156`		`- if (bsl::next(it) != d_authenticators.cend()) {`
	`155`	`+ if (it != d_authenticators.cbegin()) {`
`157`	`156`	`BALL_LOG_OUTPUT_STREAM << ", ";`
`158`	`157`	`}`
	`158`	`+ BALL_LOG_OUTPUT_STREAM << it->first;`
`159`	`159`	`}`
`160`	`160`	`BALL_LOG_OUTPUT_STREAM << "]";`
`161`	`161`	`}`