Magus
Magus copied to clipboard
Overqualification of symbols in generated header
Hi, I'm that guy that is annoying fixing linux related stuff you in the HLMSEditor repo ;-)
I'm only looking at code generated by Magus in the HLMSEditor project, it seems that there's a bug in Magus that makes it generate non standard code, and the compiler you are using (MSVC?) seems to accept it.
It's regarding class headers. It tend to over qualify the headers by reapeating the name of the symbol twice, and GCC will not accept that as it sees it as an error.
It may be simpler to see with an example :
class MyClassName
{
//This is good:
someMethod();
int someAttribute;
//This doesn't compile:
MyClassName::someOtherMethod();
int MyClassName::someOtherAttribute;
}
So, I did fix some of these in the past in the HLMSEditor code, but if it's generated code by some tool, we shold fix the tool, not the output, right? ^^"
I'm going to look into this more closely when I will have some time (if you're not a linux guy, or using mingw, you'll not find this problem anyway)
Tell me what you think of it, where should I look at in Magus code to gain some time? ^^"
PS: this tool looks great actually. I've started working on a Qt5+Ogre level editor for an open source game engine project and even if I got around to make a quite flexible "3D viewport" widget for Ogre2.1, Magus seems to have the potential to greatly reduce the pain of mixing GUI and 3D. Could you indicate a proper open source licence in the readme or a LICENCE file at the root of the repo.
If so, I may suggest the MIT one, as used by HLMSEditor and Ogre itself (and my game engine project ^^")
I will scan through the code and remove all this issues. It is just a copy/paste error and they are not supposed to be in there