Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-61946

Replace LineRange with Location

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Major Major
    • analysis-model
    • None

      Currently, issues refer to a line, a line range, or multiple line ranges in a single file. It would make sense to extend this concept so that issues refer to multiple locations, that may span multiple files.

      One parser that already supports that concept is the CPPCheck. Multiple files are currently ignored. The Gcc parser could be extended as well.

          [JENKINS-61946] Replace LineRange with Location

          Kalle Niemitalo added a comment -

          Static Analysis Results Interchange Format (SARIF) Version 2.1.0 also supports multiple locations. I don't think analysis-model has a parser for SARIF yet, but it might be useful as a source of ideas.

          Kalle Niemitalo added a comment - Static Analysis Results Interchange Format (SARIF) Version 2.1.0 also supports multiple locations. I don't think analysis-model has a parser for SARIF yet, but it might be useful as a source of ideas.

          Ulli Hafner added a comment -

          Yes, I hoped that someone will volunteer in writing a SARIF parser since PMD and SpotBugs can write that format now (and several Microsoft scanners use it as major output forma)t. Hopefully this will be a matter of time only.

          Adding support for multiple files in the model shouldn't be too hard. However, it would make sense to wait for an actual implementation that uses it.

          Ulli Hafner added a comment - Yes, I hoped that someone will volunteer in writing a SARIF parser since PMD and SpotBugs can write that format now (and several Microsoft scanners use it as major output forma)t. Hopefully this will be a matter of time only. Adding support for multiple files in the model shouldn't be too hard. However, it would make sense to wait for an actual implementation that uses it.

            drulli Ulli Hafner
            drulli Ulli Hafner
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: