search

staruml / staruml-cpp

C++ extension for StarUML
MIT License
173 stars 66 forks source link

hpp extension not recognized #20

Open nealtanner opened 8 years ago

nealtanner commented 8 years ago

I am attempting to use this extension to reverse engineer an existing project that uses .cpp and .hpp file extensions and it acted like there was nothing in the directory being analyzed.

I modified line 1018 in CppReverseEngineer.js to add the third OR case below and now it successfully pulls in roughly half of the classes that are present in the directory being analyzed. Is the problem with an inadequate patch to the original hpp extension issue or is there some other reason that half of my classes are missing?

if (ext && ((ext.toLowerCase() === "cpp") || (ext.toLowerCase() === "h") || (ext.toLowerCase() === "hpp")))

Superlokkus commented 7 years ago

+1