Author Topic: Anyone want a patch to remove the defines of WXMAKINGDLL_PROPGRID etc.  (Read 3889 times)

Offline stahta01

  • Lives here!
  • ****
  • Posts: 6894
    • My Best Post
I just created an untested patch to remove the defines of WXMAKINGDLL_PROPGRID, WXMAKINGLIB_PROPGRID, and wxPG_USE_WXMODULE from the CB source and build files.

I replaced the WXMAKINGDLL_PROPGRID and WXMAKINGLIB_PROPGRID with the CB define of EXPORT_LIB.

I will post the patch here if anybody wants to test it.

Patched many *.am and *.cbp files and about 4 cpp/h files.

I also patched the sdk.doxy file; but, I am not sure what that file does.

Tim S.
C Programmer working to learn more about C++ and Git.
On Windows 7 64 bit and Windows 10 32 bit.
On Debian Stretch, compiling CB Trunk against wxWidgets 3.0.
--
When in doubt, read the CB WiKi FAQ. http://wiki.codeblocks.org

Offline oBFusCATed

  • Developer
  • Lives here!
  • *****
  • Posts: 12595
    • Travis build status
Re: Anyone want a patch to remove the defines of WXMAKINGDLL_PROPGRID etc.
« Reply #1 on: February 11, 2012, 01:41:45 am »
What is the problem/bug solved by the patch?
(most of the time I ignore long posts)
[strangers don't send me private messages, I'll ignore them; post a topic in the forum, but first read the rules!]

Offline stahta01

  • Lives here!
  • ****
  • Posts: 6894
    • My Best Post
Re: Anyone want a patch to remove the defines of WXMAKINGDLL_PROPGRID etc.
« Reply #2 on: February 11, 2012, 01:47:42 am »
What is the problem/bug solved by the patch?

Several of the Plugin Managers fails to define the WXMAKINGDLL_PROPGRID Macros this causes warning and results in the build NOT using the precompiled headers.

The binary output after this change should be an exact match of what it was before the change.
But, the build speed should be slightly faster and the warning should be reduced.
The main gain is it it easier for new developers to make a Plugin the correct way.
Because they do NOT have to add a single define statement for the PCH to be properly used in respect to the wxPropertyGrid library. Patch created against the propgrid/debugger branch.

Tim S.    
« Last Edit: July 08, 2012, 08:15:19 am by stahta01 »
C Programmer working to learn more about C++ and Git.
On Windows 7 64 bit and Windows 10 32 bit.
On Debian Stretch, compiling CB Trunk against wxWidgets 3.0.
--
When in doubt, read the CB WiKi FAQ. http://wiki.codeblocks.org

Offline stahta01

  • Lives here!
  • ****
  • Posts: 6894
    • My Best Post
Noticed that someone applied my patch or a related one to SVN.

I removed my attached patch from prior post.

Note: I found a missed spot. The file Cscope.cbp still have the old macros in it.

Tim S.

Code: [Select]
Index: src/plugins/contrib/Cscope/Cscope.cbp
===================================================================
--- src/plugins/contrib/Cscope/Cscope.cbp (revision 8095)
+++ src/plugins/contrib/Cscope/Cscope.cbp (working copy)
@@ -37,8 +38,6 @@
  <Add option="-DWXUSINGDLL" />
  <Add option="-DcbDEBUG" />
  <Add option="-DwxUSE_UNICODE" />
- <Add option="-DWXMAKINGDLL_PROPGRID" />
- <Add option="-DwxPG_USE_WXMODULE=0" />
  <Add directory="$(#WX.include)" />
  <Add directory="$(#WX.lib)\gcc_dll$(WX_CFG)\msw$(WX_SUFFIX)" />
  <Add directory="$(#WX)\contrib\include" />
C Programmer working to learn more about C++ and Git.
On Windows 7 64 bit and Windows 10 32 bit.
On Debian Stretch, compiling CB Trunk against wxWidgets 3.0.
--
When in doubt, read the CB WiKi FAQ. http://wiki.codeblocks.org

Offline MortenMacFly

  • Administrator
  • Lives here!
  • *****
  • Posts: 9594
Note: I found a missed spot. The file Cscope.cbp still have the old macros in it.
Done. Thanks!
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: http://www.codeblocks.org/docs/main_codeblocks_en.html
C::B FAQ: http://wiki.codeblocks.org/index.php?title=FAQ