Developer forums (C::B DEVELOPMENT STRICTLY!) > Plugins development

wxSmith improvement

<< < (11/12) > >>

Jenna:
I attach a patch that seems to fix the issue for wxsColourProperty, it's not deeply tested (and not tested at all on windows and mac), but seems to work.

Please other devs (and users of course) please test and give feedback.

I will look into the other issues (wxListbox and others) later.

The patch is against trunk (r6595), but I think it works with all other branches too.

EDIT:
attachment removed, actual patch see http://forums.codeblocks.org/index.php/topic,12362.msg89758.html#msg89758

Jenna:
After some testing, I found that the patch is just a starting point not a solution, it works for existing elements, but crashes if a new one with colourproperty is added.

Jenna:

--- Quote from: jens on September 17, 2010, 09:02:23 pm ---After some testing, I found that the patch is just a starting point not a solution, it works for existing elements, but crashes if a new one with colourproperty is added.

--- End quote ---
New version (a little larger), but this time it seems to work correctly.

It's against trunk and not (yet) tested on windows.

Any feedback is welcome.

EDIT:
attachment removed, actual patch see http://forums.codeblocks.org/index.php/topic,12362.msg89758.html#msg89758

MortenMacFly:

--- Quote from: jens on September 19, 2010, 12:50:11 pm ---Any feedback is welcome.

--- End quote ---
I had applied the first one, too - no serious issues so far. I'll try this one, too. If you hear nothing all is fine. :-)

Jenna:
Small update to my previous patch (build/link-fix for windows):

either use the patch attached here or remove WXDLLIMPEXP_PG from class-declaration of wxsMyColourPropertyClass in wxscolourproperty.cpp:124.

I remove the older patches.

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version