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

fix CppcheckParserV2; allow multiple NewIssueLocations #1436

Closed
wants to merge 6 commits into from

Commits on Mar 22, 2018

  1. fix CppcheckParserV2; allow multiple NewIssueLocations

    Ivan Galkin authored and Ivan Galkin committed Mar 22, 2018
    Configuration menu
    Copy the full SHA
    d0934a8 View commit details
    Browse the repository at this point in the history
  2. fix CppcheckParserV2; allow multiple NewIssueLocations

    Ivan Galkin authored and Ivan Galkin committed Mar 22, 2018
    Configuration menu
    Copy the full SHA
    452f640 View commit details
    Browse the repository at this point in the history
  3. fix CppcheckParserV2; allow multiple NewIssueLocations [fix 2]

    Ivan Galkin authored and Ivan Galkin committed Mar 22, 2018
    Configuration menu
    Copy the full SHA
    c886539 View commit details
    Browse the repository at this point in the history
  4. fix CppcheckParserV2; allow multiple NewIssueLocations [formatting]

    Ivan Galkin authored and Ivan Galkin committed Mar 22, 2018
    Configuration menu
    Copy the full SHA
    4a640d4 View commit details
    Browse the repository at this point in the history
  5. fix CppcheckParserV2; allow multiple NewIssueLocations [fix 3]

    Ivan Galkin authored and Ivan Galkin committed Mar 22, 2018
    Configuration menu
    Copy the full SHA
    e04cb27 View commit details
    Browse the repository at this point in the history

Commits on Mar 29, 2018

  1. fix CppcheckParserV2; allow multiple NewIssueLocations [fix 4]

    * create first-class citizen entities CxxReportIssue and CxxReportLocation (1:n)
      (CxxReportIssue::equals() allows clear distinguishing if issue was processed already)
    
    * CppcheckParserV2: better implementation for primary and secondary locations
    
    * CxxReportSensor: increase issues-per-module counter only once per CxxReportIssue,
                       increase issues-per-file counter only once per affected file
    
    TODO (?): tests for CxxReportIssue and CxxReportLocation
    TODO (?): use the new function CxxReportSensor::saveUniqueViolation(SensorContext, CxxReportIssue)
    Ivan Galkin authored and Ivan Galkin committed Mar 29, 2018
    Configuration menu
    Copy the full SHA
    6a9347e View commit details
    Browse the repository at this point in the history