You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey, I was wondering whether you are planning to change the loader in order to use proper c++ style code ( no c-like casting, more std library use etc.)?
I am dealing with many warnings on my project through the loader and some of them might become dangerous in special use cases I guess.
Unfortunately I have not enough knowledge about the code otherwise I would to that on my own.
The text was updated successfully, but these errors were encountered:
Hey, I was wondering whether you are planning to change the loader in order to use proper c++ style code ( no c-like casting, more std library use etc.)?
no. those would be purely cosmetic.
the only exception I see are alignment issues (enabled by -Wcast-align) on ARM architectures. (they work correctly on x86)
However those are deep in the bit-twiddling code where changes would require throughout testing.
Contributions are welcome here.
paroj
changed the title
use proper c++ style
alignment issues uncovered by -Wcast-align
Feb 23, 2018
Okay thank's for that answer! I use the library in my C++ project and I get a few warnings about the old c-style casts and as you already mentioned about possible alignment issues. I thought it would become handy to solve these.
But I did not find a bug so it's just a matter of style I guess.
Hey, I was wondering whether you are planning to change the loader in order to use proper c++ style code ( no c-like casting, more std library use etc.)?
I am dealing with many warnings on my project through the loader and some of them might become dangerous in special use cases I guess.
Unfortunately I have not enough knowledge about the code otherwise I would to that on my own.
The text was updated successfully, but these errors were encountered: