Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for cppcheck 1.87 #1687

Merged
merged 2 commits into from
Feb 10, 2019
Merged

Conversation

ivangalkin
Copy link
Contributor

@ivangalkin ivangalkin commented Feb 10, 2019

Enable 7 new rules

* constArgument                         style   -> MINOR CODE_SMELL
* danglingReference                     error   -> MAJOR BUG
* duplicateCondition                    style   -> MINOR CODE_SMELL
* invalidScanfFormatWidth_smaller       warning -> MINOR BUG
* iterators2                            error   -> MAJOR BUG
* nullPointerArithmeticRedundantCheck   warning -> MINOR BUG
* wxBitmap::SetDepthCalled              style   -> MINOR CODE_SMELL

Also fix descriptions of some existing ones


This change is Reviewable

Enable 7 new rules

```
* constArgument                         style   -> MINOR CODE_SMELL
* danglingReference                     error   -> MAJOR BUG
* duplicateCondition                    style   -> MINOR CODE_SMELL
* invalidScanfFormatWidth_smaller       warning -> MINOR BUG
* iterators2                            error   -> MAJOR BUG
* nullPointerArithmeticRedundantCheck   warning -> MINOR BUG
* wxBitmap::SetDepthCalled              style   -> MINOR CODE_SMELL
```

+ fix descriptions of some existing ones
@ivangalkin ivangalkin requested a review from guwirth February 10, 2019 16:09
@ivangalkin ivangalkin self-assigned this Feb 10, 2019
@ivangalkin ivangalkin added this to the 1.2.2 milestone Feb 10, 2019
Copy link
Collaborator

@guwirth guwirth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guwirth guwirth merged commit d1519ba into SonarOpenCommunity:master Feb 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants