Github user phrocker commented on a diff in the pull request:

    https://github.com/apache/nifi-minifi-cpp/pull/293#discussion_r179582611
  
    --- Diff: libminifi/include/utils/StringUtils.h ---
    @@ -88,7 +91,10 @@ class StringUtils {
        */
       static inline bool equalsIgnoreCase(const std::string &left, const 
std::string right) {
         if (left.length() == right.length()) {
    -      return std::equal(right.begin(), right.end(), left.begin(), 
[](unsigned char lc, unsigned char rc) {return std::tolower(lc) == 
std::tolower(rc);});
    --- End diff --
    
    I noticed you changed a lot of files outside of core. Can you keep these at 
200 characters so we avoid unnecessary change later?


---

Reply via email to