Skip to content

Fixed #12504 (dumpfile: usage of relative paths is inconsistent)#6275

Merged
danmar merged 2 commits intocppcheck-opensource:mainfrom
cppchecksolutions:fix-12504
Apr 11, 2024
Merged

Fixed #12504 (dumpfile: usage of relative paths is inconsistent)#6275
danmar merged 2 commits intocppcheck-opensource:mainfrom
cppchecksolutions:fix-12504

Conversation

@danmar
Copy link
Copy Markdown
Collaborator

@danmar danmar commented Apr 11, 2024

No description provided.

@danmar danmar force-pushed the fix-12504 branch 2 times, most recently from 81b18a1 to e7e558a Compare April 11, 2024 17:16
@firewave
Copy link
Copy Markdown
Collaborator

I encountered similar issues with #6182.

I am considering adding a Path class so we have consistent paths being used throughout the code. We keep converting and normalizing all over the place and multiple times. That could also be merged with/into FileSettings which would solve a few more problems. That needs some refactoring/cleanups first though.

@danmar
Copy link
Copy Markdown
Collaborator Author

danmar commented Apr 11, 2024

I am considering adding a Path class so we have consistent paths being used throughout the code.

That sounds like a good idea imho. Sounds like it could be more type safe also.

@firewave
Copy link
Copy Markdown
Collaborator

That sounds like a good idea imho. Sounds like it could be more type safe also.

FileSettings would essentially become PathWithDetails and you pass that around instead. A light version will be introduced when I post the PR for https://trac.cppcheck.net/ticket/10692 so we do not try to identify an input file more than once.

@danmar danmar merged commit a41ef74 into cppcheck-opensource:main Apr 11, 2024
@danmar danmar deleted the fix-12504 branch April 12, 2024 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants