fgerlits commented on a change in pull request #964:
URL: https://github.com/apache/nifi-minifi-cpp/pull/964#discussion_r551970890
##########
File path: libminifi/src/utils/StringUtils.cpp
##########
@@ -95,21 +95,23 @@ bool StringUtils::StringToFloat(std::string input, float
&output, FailurePolicy
}
std::string StringUtils::replaceEnvironmentVariables(std::string&
original_string) {
- int32_t beg_seq = 0;
- int32_t end_seq = 0;
+ size_t beg_seq = 0;
Review comment:
I think we can assume that `string::size_type` will always be `size_t`,
but you are right, `string::size_type` is more correct, so I changed it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
