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.

Sign up for GitHub

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

Jump to bottom

Clang-Tidy: Issues with Windows paths with forward slashes are not supported #2245

Closed
guwirth opened this issue Sep 3, 2021 · 0 comments · Fixed by #2246
Closed

Clang-Tidy: Issues with Windows paths with forward slashes are not supported #2245

guwirth opened this issue Sep 3, 2021 · 0 comments · Fixed by #2246
Assignees
@guwirth
Labels
bug
Milestone
2.0.5

Comments

@guwirth
Copy link
Collaborator

guwirth commented Sep 3, 2021

The Clang-Tidy sensor is using the regex below:

((?>[a-zA-Z]:\\)??[^:]++):(\d{1,5}):(\d{1,5}): ([^:]++): (.+)

Issues with Windows paths with forward slashes are not supported:

D:/Path/Sample.cpp:78:23: warning: use std::make_unique instead [modernize-make-unique]
@guwirth guwirth added the bug label Sep 3, 2021
guwirth added a commit to guwirth/sonar-cxx that referenced this issue Sep 3, 2021
@guwirth
support Windows paths with forward slashes
4db6a88 
- Clang-Tidy sensor
- DrMemory sensor
- close SonarOpenCommunity#2245
@guwirth guwirth mentioned this issue Sep 3, 2021
Merged
@guwirth guwirth added this to the 2.0.5 milestone Sep 3, 2021
@guwirth guwirth self-assigned this Sep 3, 2021
guwirth added a commit that referenced this issue Sep 3, 2021
@guwirth
Merge pull request #2246 from guwirth/fix-2245
cb3afc3 
Fix #2245
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@guwirth guwirth

Labels
bug
Milestone
2.0.5
Development

Successfully merging a pull request may close this issue.

Fix 2245 guwirth/sonar-cxx
1 participant
@guwirth