Merge pull request #68 from ggtakec/fix/cppcheck

Changed cppcheck options and fixed the code

Author: ggtakec

.github/workflows/nodejs_addon_helper.sh

@@ -1370,7 +1370,7 @@ PRNSUCCESS "Start to check options and environments"
 # Default command parameters for each phase
 #
 CPPCHECK_TARGET="."
-CPPCHECK_BASE_OPT="--quiet --error-exitcode=1 --inline-suppr -j 4 --std=c++03 --xml --enable=warning,style,information,missingInclude"
+CPPCHECK_BASE_OPT="--quiet --error-exitcode=1 --inline-suppr -j 8 --std=c++14 --xml --enable=warning,style,information,missingInclude"
 CPPCHECK_ENABLE_VALUES="warning style information missingInclude"
 CPPCHECK_IGNORE_VALUES="unmatchedSuppression missingIncludeSystem normalCheckLevelMaxBranches"
 CPPCHECK_BUILD_DIR="/tmp/cppcheck"

src/k2h_shm_async.h

@@ -301,12 +301,7 @@ class GetValueAsyncWorker : public Napi::AsyncWorker
 				if(sk){
 					strarr_t	strarr;
 					sk->StringArray(strarr);
-					for(const auto &_str : strarr){
-						if(_str == _strsubkey){
-							found = true;
-							break;
-						}
-					}
+					found = std::any_of(strarr.begin(), strarr.end(), [&](const auto &_str){ return _str == _strsubkey; });
 					delete sk;
 				}
 				if(!found){