Developer forums (C::B DEVELOPMENT STRICTLY!) > Compiler Framework Redesign
New compiler line type: cltInfo
MortenMacFly:
--- Quote from: rickg22 on June 19, 2007, 03:39:07 am ---the only problem would arise (I think) with people with custom compilers, [...]
--- End quote ---
I am such (I have quite some custom additional "native" and copy-based compilers) but I don't really see a big issue. Isn't the only thing that can happen that people still see warnings, where information maybe better... so what?! I guess I'm missing something...?!
With regards, Morten.
rickg22:
Let's see what Don Corleone says :P
mandrav:
--- Quote from: rickg22 on June 19, 2007, 07:56:38 am ---Let's see what Don Corleone says :P
--- End quote ---
--- Quote from: top of sdk/compiler.cpp ---// version of compiler settings
// when this is different from what is saved in the config, a message appears
// to the user saying that default settings have changed and asks him if he wants to
// use his own settings or the new defaults
const wxString CompilerSettingsVersion = _T("0.0.1");
--- End quote ---
Just make the change (if it doesn't break anything else) and pump that number a bit.
rickg22:
Thanks! That's what I was looking for. I'll commit asap.
Update: DONE! :-)
killerbot:
everybody take care, I think a serious issue might be introduced here !!
When you have a look at default.conf, you will see that the regexes for parsing errors/warnings all are of a certain type.
Example : Compiler warning of dmd, is of type int="1" (actually on the 2 systems I checked it was at 2, old bug ???).
Now this 1 resembles something :
--- Quote ---enum CompilerLineType
{
cltNormal,
cltWarning,
cltError
};
--- End quote ---
But because the enum has been INCORRECTLY changed to :
--- Quote ---enum CompilerLineType
{
cltNormal,
cltInfo,
cltWarning,
cltError
};
--- End quote ---
That means the warning regex is now threated as of type info, and so will the error be threated as warning ...
Never ever insert values in an enum that is exported as an int !!!!!!!!!
Gonna boot up my laptop and adjust this, the cltInfo should be the last one !!!
And even then I am not sure if all compilers will correctly adjust ???
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version