search

Viladoman / CompileScore

Tools for profiling and visualizing C++ build times.
MIT License
460 stars 19 forks source link

Show absolute path for includes #44

Closed redorav closed 1 year ago

redorav commented 1 year ago

I've been using CompileScore for some time now to profile compile times in my personal projects, excellent tool and amazing job developing it.

One thing I've found is that headers with the same name but different path are hard to tell apart, e.g. a header called string.h that lives in one path and another in another path will get bundled together. However, if string.h is included via the <> then they seem to appear as separate includes.

The other issue is that even if they appeared as different in the image below, it's hard to tell which one is which, so another suggestion is to (optionally?) show absolute paths in the columns to be able to properly tell which one is which.

image

Thanks again!

Viladoman commented 1 year ago

I agree with you here. There might be some bad behavior when matching paths that are not obvious due to this. I definitely need to improve this!

As a workaround at the moment I would suggest to use the contextual menu with right click on the element and paste to a notepad or similar to see the actual full path. image

Not ideal, but at least we can get the information.

Viladoman commented 1 year ago

I just submitted a change to display the full path to includes and units.

If you want to try it before I bundle a new release, you can grab it from the action artifact directly.

redorav commented 1 year ago

@Viladoman Thank you so much for addressing this quickly. I've tried it and it works great, the only improvement I would make is to make the column sortable like the others, that would make it perfect :)

Viladoman commented 1 year ago

The full path and sorting should be up and running on Version 1.8.4.

redorav commented 1 year ago

Amazing, moltes gràcies