ENH: Improve Qt include path resolution macOS - #271
Conversation
Sorry, something went wrong.
Add specific framework path handling Refactor include directory parsing to support multiple candidate paths for headers and introduce macOS-specific logic for resolving framework-relative header paths. Enhance generator flexibility and robustness for cross-platform usage. Push upstream fix via: cppcheck-opensource/simplecpp#448
| for (const QString &includeDir: getIncludeDirectories(commandLineIncludes)) | ||
| { | ||
| QFileInfo fi(QDir(includeDir), "qtcoreversion.h"); | ||
| std::list<std::string> candiate_paths; |
There was a problem hiding this comment.
typo: candidate_paths
Sorry, something went wrong.
| std::string get_apple_framework_relative_path(const std::string& header); | ||
|
|
There was a problem hiding this comment.
Does this have to be public, or could you just put it into an anonymous namespace in the implementation?
Sorry, something went wrong.
| } | ||
|
|
||
| #ifdef __APPLE__ | ||
| static std::string get_apple_framework_relative_path(const std::string& header) |
There was a problem hiding this comment.
Please try to use the same naming style (e.g. camel case) as the rest of the code.
Sorry, something went wrong.
|
I am working with the upstream simplecpp developers to finalize the changes. cppcheck-opensource/simplecpp#448 I'll make fixes there. It will probably be a few months before I get back to this. |
Sorry, something went wrong.
|
Ah, I had missed that this is from an upstream change. In this case it is of course preferrable to leave the code compatible, though it would be even better if you could change this upstream, thanks! |
Sorry, something went wrong.
|
@hjmjohnson This pull request can now be closed as it is superseded by the following pull request: 🙏 For the initial draft & investigative work, this was instrumental to understand the issues. |
Sorry, something went wrong.
Add specific framework path handling
Refactor include directory parsing to support multiple candidate paths for headers and introduce macOS-specific logic for resolving framework-relative header paths. Enhance generator flexibility and robustness for cross-platform usage.
Push upstream fix via:
cppcheck-opensource/simplecpp#448