search

pcolby / bipolar

Free your data from Polar FlowSync
GNU General Public License v3.0
115 stars 16 forks source link

cppcheck fails on OSX only #79

Closed pcolby closed 10 months ago

pcolby commented 7 years ago

Restore cppcheck for OSX when defect #7792 is fixed, and available on Travis CI (ie via Homebrew).

Note, this is not an OSX-specific problem. OSX just happens to be getting a more-recent (and thus affected) version of cppcheck currently.

aj664 commented 7 years ago

slightly unrelated... but cppcheck has issue with /test/protobuf/testmessage.cpp line 38.. syntax error. i have a strange feeling it has issue with foreach ( .. ) is that supposed to be for_each as in std::for_each() ?

pcolby commented 7 years ago

Thanks @aj664, which version of cppcheck are you seeing this on?

The foreach sholdn't be a problem (usually isn't) for cppcheck, because its a preprocessor macro (defined by Qt).

Thanks.

aj664 commented 7 years ago

v1.80
ok, very likely that the Qt macro was not available during the test. non-issue.

pcolby commented 6 years ago

Note to self: easy to just restart Job #831.1 to test.