Developer forums (C::B DEVELOPMENT STRICTLY!) > Development
Proposed Rules and suggestions for using PCH properly under Windows
stahta01:
I added patch to the CB Plugin Wizard that needs tested in my scriptedwizard branch before it is applied to SVN trunk.
It should be enough just to verify that the CB Plugin Wizard results in code that compilers and links under Linux.
Thank you
Tim S.
--- Quote from: stahta01 on June 05, 2014, 07:44:23 pm ---Pushed my local changes to https://github.com/stahta01/codeblocks_sf_tims/tree/scriptedwizard
I tested them under Windows 32-bit using wxWidgets 2.8 and they tested well.
The only things changed are wxWidgets templates so rebuilding the ScriptedWizard Plugin is likely NOT needed.
The list of things changed are the same.
PCH file (wx_pch.h) use in Wizard (Misc. Settings: Create and use precompiled Header)
Preferred GUI Builder NONE and wxFormBuilder code changes (No changes to wxSmith GUI Builder code)
Fixed Bug in CB Project creation when Misc. Settings: "Create and use precompiled Header" and "Create Empty Project" both checked.
The most important fix from a PCH build view is moving wx/wx.h from the main.h to app.cpp/main.cpp changes.
The upgrading of the wxFormBuilder to version 3.0 is the other important PCH build change.
Please verify the wxWidget wizard runs and results in projects that build without error on Linux.
Then, I would hope these changes are able to be commited to SVN trunk.
Thank you, again.
Tim S.
--- End quote ---
stahta01:
Created another branch to do other changes/patches https://github.com/stahta01/codeblocks_sf_tims/tree/for_upstream
I committed a change/patch to add PCH building back to CB under wxWidgets 3.0 Windows 32 bit (also added it to work under wxWidgets 2.8 Windows 32 bit).
The Contrib plugins I added it to is wxSmith Aui. plugin; please test to verify it works under Linux.
Please re-build just the wxSmith Aui. plugin for both wxWidgets 2.8 and 3.0 to verify that the build time is reduced by the patch under Linux.
Under Windows, the build time is reduced by 20 to 60 percent of what it is without the patch.
The patch is safe to be applied to trunk if there is no build issues under Linux.
But, if it does NOT speed up the Linux build it is NOT likely worth applying to SVN trunk.
If it works (speeds up the Linux Build); I plan to work on the wxSmith Plugin next.
Tim S.
oBFusCATed:
Can you add an upper limit to the version check?
This way newer compilers will fail and they can be added to the check.
Otherwise it is quite possible that the wxpch won't be used for newer compilers forever until someone digs into this again and stumbles upon this check.
stahta01:
--- Quote from: oBFusCATed on June 17, 2014, 02:18:27 am ---Can you add an upper limit to the version check?
This way newer compilers will fail and they can be added to the check.
Otherwise it is quite possible that the wxpch won't be used for newer compilers forever until someone digs into this again and stumbles upon this check.
--- End quote ---
What version number do you wish me to think works? 4.9 or what?
Tim S.
stahta01:
--- Quote from: stahta01 on June 17, 2014, 02:26:57 am ---
--- Quote from: oBFusCATed on June 17, 2014, 02:18:27 am ---Can you add an upper limit to the version check?
This way newer compilers will fail and they can be added to the check.
Otherwise it is quite possible that the wxpch won't be used for newer compilers forever until someone digs into this again and stumbles upon this check.
--- End quote ---
What version number do you wish me to think works? 4.9 or what?
Tim S.
--- End quote ---
I will change it to a range of GCC versions stopping at either 4.9.0 or 4.9.1 for now.
Will also start at 4.7.2 (the web implies it might have started with that version.)
Will post when I get it done.
Tim S.
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version