[PATCH] D65206: [analyzer] Fix text range end columns in SARIF to be exclusive

Joe Ranieri via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 24 06:23:45 PDT 2019


jranieri-grammatech created this revision.
jranieri-grammatech added reviewers: george.karpenkov, NoQ, aaron.ballman.
Herald added subscribers: cfe-commits, Charusso, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun.
Herald added a project: clang.

According to the SARIF specification, "a text region does not include the character specified by endColumn".


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D65206

Files:
  clang/lib/StaticAnalyzer/Core/SarifDiagnostics.cpp
  clang/test/Analysis/diagnostics/Inputs/expected-sarif/sarif-diagnostics-taint-test.c.sarif
  clang/test/Analysis/diagnostics/Inputs/expected-sarif/sarif-multi-diagnostic-test.c.sarif

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65206.211484.patch
Type: text/x-patch
Size: 6271 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190724/498e564d/attachment.bin>


More information about the cfe-commits mailing list