search

danmar / simplecpp

C++ preprocessor
BSD Zero Clause License
204 stars 80 forks source link

do not add empty filename to `files` in `preprocess()` #350

Closed firewave closed 5 months ago

firewave commented 5 months ago

Required for https://github.com/danmar/cppcheck/pull/6326.

Not sure if a test can be written for this.

firewave commented 5 months ago

Made a draft because the downstream PR which is built upon these changes fails.

firewave commented 5 months ago

Made a draft because the downstream PR which is built upon these changes fails.

Turns out that it was a fluke.

firewave commented 5 months ago

I will still add a unit test for this.

firewave commented 5 months ago

I will still add a unit test for this.

Done.