Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

Patches to compile and link C::B against wxWidgets 2.8.0

<< < (4/25) > >>

stahta01:

--- Quote from: sethjackson on November 24, 2006, 03:11:59 am ---Why do we need WXWIN_COMPATIBILITY_2_4? C::B won't even compile with that version of wx........

--- End quote ---

By default wxWidgets sets a flag for compatibility to the last version in setup.h.

wx26 has WXWIN_COMPATIBILITY_2_4 set to 1

wx28 has WXWIN_COMPATIBILITY_2_4 set to 0 and WXWIN_COMPATIBILITY_2_6 set to 1

So code::blocks current dll is compiled with WXWIN_COMPATIBILITY_2_4 set to 1.

Tim S

sethjackson:
Oh ok. Thanks. :)

afb:

--- Quote from: stahta01 on November 24, 2006, 03:08:57 am ---We can just turn it on for just 2.6; 2.8 has it off by default. I think it will require more work if we don't turn it off for 2.6 but the amount of work won't be that much. Just some #ifdef blocks in codeblocks.

From a user point of view having it on for 2.6 and off for 2.8 will be the simplest way. I will work on a patch to fix code::blocks so it works both with WXWIN_COMPATIBILITY_2_4 on or off.
--- End quote ---

This would the best solution, since it would mean that Code::Blocks would work with both default wxWidgets 2.6 and default wxWidgets 2.8 without any modifications.

Of course, if we have any of our own code that still uses wxWidgets 2.4 methods then we should strive to update those to 2.6. We should survive 2.4 compat, but not require it...


--- Quote from: killerbot on November 23, 2006, 11:06:42 pm ---first of all, let's see what wx 2.8 brings . Will it fix some of the bugs we are suffering from the 2.6, if yes, then it light be interesting to use 2.8. If not, then might be best to wait.

--- End quote ---

wxWidgets 2.6 already has some showstopper bugs, but those can be fixed with patches (or with wxWidgets 2.6.4) so that in itself is not reason enough I think.

stahta01:
FYI:

The only thing that seems to need WXWIN_COMPATIBILITY_2_4 set to 1 is the use of wxHIDE_READONLY. I have submitted a patch to guard all the locations that it is used in.
Tim S

Patches to core Code::Blocks project and contrib Code::Blocks workspace:
 [ Patch #1655 ] wxHIDE_READONLY patch for wxWidgets 2.7 Submitted By: stahta01
   https://developer.berlios.de/patch/?func=detailpatch&patch_id=1655&group_id=5358

mandrav:
Just a note: discussions to switch C::B development to wx2.8 are pointless. wx2.8 will take quite a while to enter the major linux distros repositories. Until then, wx2.6 will be used for C::B development.
Of course, patches that allow building with wx2.8 too are fine (assuming they don't break existing functionality).

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version