Insert optional data visitor file regions cache (#2544)

* Allow per-instance file regions cache to be set on InsertOptionalDataVisitor. * Provide context region construction helper in 'FileRegionsCache'.
microsoft · Sep 12, 2022 · af69a18 · af69a18
1 parent eb24e66
commit af69a18
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/ReleaseHistory.md b/src/ReleaseHistory.md
@@ -1,5 +1,6 @@
 # SARIF Package Release History (SDK, Driver, Converters, and Multitool)
 
+## Unreleased
 * FEATURE: Allow initialization of file regions cache in `InsertOptionalDataVisitor` (previously initialized exclusively from `FileRegionsCache.Instance`).
 * BUGFIX: Resolve issue where `match-results-forward` command fails to generate VersionControlDetails data. [#2487](https://github.com/microsoft/sarif-sdk/pull/2487)
 * BUGFIX: Remove duplicated rule definitions when executing `match-results-forward` commands for results with sub-rule ids. [#2486](https://github.com/microsoft/sarif-sdk/pull/2486)

diff --git a/src/Sarif/FileRegionsCache.cs b/src/Sarif/FileRegionsCache.cs
@@ -142,7 +142,7 @@ private Region PopulateTextRegionProperties(NewLineIndex lineIndex, Region input
             return region;
         }
 
-        internal Region ConstructMultilineContextSnippet(Region inputRegion, Uri uri)
+        public Region ConstructMultilineContextSnippet(Region inputRegion, Uri uri)
         {
             if (inputRegion?.IsBinaryRegion != false)
             {