inconsistency between wxs and cpp/h files in code::blocks itself

[frithjofh]

Hi everybody,

I just found that somebody has changed code in wxssettings.h and .cpp without going through wxSmith. As a result the files do not concur anymore. Opening the wxs file in wxSmith changes the source files and results in a non compiling scenario. In this particular case it is a checkbox someone added to allow for unique IDs only.

I found half a dozen of wxs files out of sync with the respective source files.

Regards

frithjofh

[oBFusCATed]

Patches welcome

[frithjofh]

woa, I figure I do a patch now and tomorrow somebody does some changes again manually. So my question is: what is the policy to follow here? Is it worth while or is wxSmith dead for use inside code::blocks?

regards

[MortenMacFly]

or is wxSmith dead for use inside code::blocks?

Of course not. What's designed with wxSmith (inside C::B) has to be modified using wxSmith. If this got out of sync its a bug, maybe due to the merge of the wxSmith branch.

[frithjofh]

OK. I'll make some patches then...

regards

PS: should I post the patches here as attachment, once I'm done?

[MortenMacFly]

PS: should I post the patches here as attachment, once I'm done?

Yes, if you get no response, put it in the patch tracker and post a link.

[frithjofh]

patch for according files attached. These were all I found to be faulty. The others are checked to be alright, that is, they compile alright. Checked only contrib workspace.

And: the ConfigPanel in the DoxyBlocks plugin has a minor quirk: everytime the wxs file is opened it makes changes to the sources that conflict with cbStyleTextCtrl. wxSmith seems to not be able to figure out the correct number of parameters for the widget, see comment on line 42 in ConfigPanel.wxs

Please tell me if they worked and were alright.

[MortenMacFly]

Please tell me if they worked and were alright.

They do not work, what patch format did you choose?

Please do as following:
In your C::B SVN source tree, apply the patch, then type in the root of C::B's SVN folder:
svn diff > my.patch
(...assuming you have the subversion command line tool installed.)

There are many diff format flavours around - we need the one compatible to SVN which is produced that way.

[frithjofh]

OK. next try. as attached...

hope this one works

[MortenMacFly]

hope this one works

Yes, and applied in trunk. Thank you!