search

microsoft / sarif-sdk

.NET code and supporting files for working with the 'Static Analysis Results Interchange Format' (SARIF, see https://github.com/oasis-tcs/sarif-spec)
Other
191 stars 88 forks source link

Adding support in rolling hash for unicode separators #2723

Closed suvamM closed 9 months ago

suvamM commented 9 months ago

This PR makes the following changes:

  1. Rolling hash computations now support Unicode line separator \u2028 and Unicode paragraph separator \u2029.
  2. Unit tests checking this new logic.
  3. Refactoring to conform to coding guidelines: (for example, if explicit type declared in RHS, use var in LHS).