added FileWithDetails to store resolved input path in#6285
added FileWithDetails to store resolved input path in#6285firewave merged 5 commits intocppcheck-opensource:mainfrom
FileWithDetails to store resolved input path in#6285Conversation
firewave
changed the title
PathWithDetails to store input path inPathWithDetails to store resolved input path in
firewave
force-pushed
the
path-details
branch
2 times, most recently
from
38e4f73 to
d7fe55b
Compare
|
This is the initial step so we start out with the (currently) basic wrapper. We need to adjust the code to pass that instead of of a simple string and we can add details and more logic along the way. The aim is to have merged |
| return err; | ||
|
|
||
| // add *.cpp files to the "files" vector.. | ||
| for (const std::pair<const std::string&, size_t> file : filelist) { |
There was a problem hiding this comment.
The previous type was wrong and there was also an unnecessary copy which was not detected by clang-tidy.
|
I think it would be better to call it |
firewave
force-pushed
the
path-details
branch
3 times, most recently
from
bc20feb to
132f60e
Compare
firewave
changed the title
PathWithDetails to store resolved input path inFileWithDetails to store resolved input path in
|
This is quite a large refactoring. Please do not merge before the release. |
firewave
added
the
merge-after-next-release
As this doesn't change any logic at all it is not really that big of a change. I am fine with delaying this nevertheless. |
firewave
removed
the
merge-after-next-release
2 participants
No description provided.