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

[trunk] OsX Catalina: crash on init due to wxTheClipboard issue

(1/1)

perazz:
Hi,

I'm a longtime C::B user but new to developing and contributing to this forum - sorry if I'm posting in the wrong place.
I've just managed to build C::B trunk on Mac OSX Catalina (arm64). I would like to help debuggin/bugfixing on Mac.

The app does not open because of an assert issue with the clipboard.

I think it's related to this mac-specific bug: https://github.com/OSGeo/grass/pull/819. It's fixed if the call to Flush() is only made if the clipboard IsOpened():
applies to both OnInit (src/app.cpp, line 581) and OnExit (src/app.cpp, line 829)
and I solved it by wrapping:


--- Code: ---   if (wxTheClipboard->IsOpened()) {
        wxTheClipboard->Flush();
      }

--- End code ---

Is this the right way to suggest a bug fix?
Thanks for the hard work on C::B,

Federico

BlueHazzard:
Hi,

--- Quote ---I would like to help debuggin/bugfixing on Mac.
--- End quote ---
Highly appreciated!!! Thank you!


--- Quote ---Is this the right way to suggest a bug fix?
--- End quote ---
The best way for bug fixes and patches is to provide them on sourceforge.
Here they will get lost in the noise of normal forum posts

AndrewCot:
Posted in the right forum and good to see a Mac dev trying to help out.

The following wiki page has info on patches and info on the C::B SF ticket process:
   https://wiki.codeblocks.org/index.php/Creating_a_patch_to_submit_(Patch_Tracker)
Unfortunately the page is for Windows/Linux, so not all of the info may be applicable or may work on OSX, but a large chunk of it will. The page info on submitting a patch is not right, create a ticket and attach the patch.

Miguel Gimenez:
@perazz or @Xaviou, would this be acceptable?


--- Code: ---#ifdef __WXOSX_MAC__
if (wxTheClipboard->IsOpened())
#endif
        wxTheClipboard->Flush();

--- End code ---

or, is there a method to check specifically for Catalina?

Miguel Gimenez:
Fixed in r12901.

Navigation

[0] Message Index