-
Notifications
You must be signed in to change notification settings - Fork 258
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
Bundle resource bug fixes #1026
Bundle resource bug fixes #1026
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
@@ -159,20 +159,20 @@ namespace cppmicroservices | |||
return; | |||
} | |||
|
|||
for (++iter; iter != m_SortedEntries.end(); ++iter) | |||
for (iter=iter; iter != m_SortedEntries.end(); iter++) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: explicitly assigning value of variable of type 'iterator' (aka '_Rb_tree_const_iterator<std::pair<std::basic_string, int>>') to itself [clang-diagnostic-self-assign-overloaded]
for (iter=iter; iter != m_SortedEntries.end(); iter++)
^
These changes appear to be trying to work around a problem with a malformed or incorrectly built bundle. |
The resource compile creates a default folder named resources-plugin and puts in it the manifest.json file. which is not made by me. I found it weird that the manifest.json is not in the root path of the zip file. And because of that the whole code with normalization is broken due to the path never found inside of the zip file |
|
If you use my project : https://github.com/al-sabr/bismika The loading of resources which are embedded in the shared library are recognized by the framework but if you replace the PR version by the original version from your repo it will not work at this line : https://github.com/al-sabr/bismika/blob/114b63a38d19f0bef3af008525209643db14bba8/cmake/cppmicroservices.cmake#L7 |
Due to this thread I corrected the code so that it works.
#1025