search

ftylitak / qzxing

Qt/QML wrapper library for the ZXing library. 1D/2D barcode image processing library
Apache License 2.0
621 stars 340 forks source link

Fix some static analyzer warnings #156

Closed dseight closed 4 years ago

dseight commented 4 years ago

clazy and clang-tidy were used to check code. Some things are not fixed yet: the require thorough investigation.

okodron commented 4 years ago

LGTM

ftylitak commented 4 years ago

@dseight thank you for your contribution.

The only change that i have made on your code before merging the branch was to remove all c++11 specific code as the project is still C++98 compatible.

Well, in fact, I have found C++11 code already in the master branch from an older PR...and no one complained so i guess there is no point of keeping the C++98 support.

Thank you any way!