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.

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

header file changes #19

Open
Trass3r opened this issue Feb 4, 2019 · 5 comments
Open

header file changes #19

Trass3r opened this issue Feb 4, 2019 · 5 comments

Comments

@Trass3r
Copy link

Trass3r commented Feb 4, 2019

How does it handle header file changes?
I guess it's not possible figure out which source files are affected.

@crosire
Copy link
Owner

crosire commented Feb 4, 2019

It doesn't currently. It would be possible to figure out all the source files affected in theory (that information is stored in the PDB and blink already reads that, it just doesn't do anything with it right now).

@crosire
Copy link
Owner

crosire commented Feb 4, 2019

See the following (this list also contains header files, but blink currently filters it for *.cpp files only):

pdb.read_source_files(_source_files);

@Trass3r
Copy link
Author

Trass3r commented Feb 5, 2019

Maybe it should output a warning at least that the change was detected and the project is now in a floating state.

It would be possible to figure out all the source files affected in theory (that information is stored in the PDB

It is? Isn't that an awful lot of data? Or does it filter system includes out?

@crosire
Copy link
Owner

crosire commented Feb 5, 2019

Nope, they are all in. It is a lot of data.

@crosire
Copy link
Owner

crosire commented Mar 18, 2019

Proof of concept is in the headers branch. It will compile all source files which include the modified header file.

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

No branches or pull requests

2 participants