News:

The new Release 25.03 is out! You can download binaries for Windows and many major Linux distros here .

Main Menu

wxWidgets 3.1.2 Released

Started by ollydbg, December 11, 2018, 03:32:23 PM

Previous topic - Next topic

gd_on

Thanks M. Gimenez. No more problem (until the next one  ;D) with your temporary patch  ;).
gd_on
Windows 11 64 bits (25H2), svn C::B (last version or almost!), wxWidgets 3.3.2, Msys2 Compilers 16.1.0, 64 bits (seh, posix : gcc, g++ and gfortran in C:\msys64\mingw64) or 32 bits (dwarf2, posix  in C:\msys64\mingw32).

Miguel Gimenez

#46
Quote from: oBFusCATed on January 15, 2019, 09:09:38 PM
First we wait for the changes to enter wx-master. Then we'll decide what course of action we'll take for night/release binaries. I don't think waiting for wx3.1.3 is viable option. wx releases are very infrequent.

New Pagodi's changes to wxPopupWindow have been commited today:

https://github.com/wxWidgets/wxWidgets/commit/07f64c3b75c79b79b46a7e8ac8db585a618c57da

EDIT: I have just compiled C::B revision 11558 with this commit in MSW, the issues with Code Completion are gone.

oBFusCATed

Would it be possible to create a patch only containing this change against 3.1.2 and see if everything works fine.

@killerbot: Are you willing to use a patched version of 3.1.2 for the night builds?
(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!]

Miguel Gimenez

I have patched my wx3.1.2 library with New Pagodi“s changes and Code Completion works fine.

I have generated a diff file, but I don't know if it can be applied without some tweaking (indexes probably are invalid).

Just in case I am attaching also the affected files (including the interface one, although it is not needed).

gd_on

Does this new wxPopupWindow patch is cumulative with the precedent one (declared as temporary on january 16) or should we start from the original version ?
Windows 11 64 bits (25H2), svn C::B (last version or almost!), wxWidgets 3.3.2, Msys2 Compilers 16.1.0, 64 bits (seh, posix : gcc, g++ and gfortran in C:\msys64\mingw64) or 32 bits (dwarf2, posix  in C:\msys64\mingw32).

Miguel Gimenez

If wxWidgets is patched then the temporary patch should be removed, because it hides the library's wxPopupWindow.

ollydbg

Quote from: Miguel Gimenez on January 28, 2019, 08:44:58 PM
I have patched my wx3.1.2 library with New Pagodi“s changes and Code Completion works fine.

I have generated a diff file, but I don't know if it can be applied without some tweaking (indexes probably are invalid).

Just in case I am attaching also the affected files (including the interface one, although it is not needed).
Nice work, I just tested it, and it works fine!
If some piece of memory should be reused, turn them to variables (or const variables).
If some piece of operations should be reused, turn them to functions.
If they happened together, then turn them to classes.

Miguel Gimenez

If devs choose patching wxWidgets 3.1.2 to solve CC crashes I suggest adding this code (or similar) somewhere so people compiling from SVN in MSW with 3.1.2 know what to do.


#if wxUSE_POPUPWIN && defined(__WXMSW__) && wxVERSION_NUMBER == 3102
    #include "wx/popupwin.h"
    #ifndef wxPU_CONTAINS_CONTROLS
        #error wxWidgets 3.1.2 must be patched or CC will crash
    #endif
#endif