search

wxWidgets / wxWidgets

Cross-Platform C++ GUI Library
https://www.wxwidgets.org/
5.77k stars 1.7k forks source link

Fix for bind with rtti disabled (fixes #14850) #24499

Closed MapleLeaf-X closed 3 weeks ago

MapleLeaf-X commented 3 weeks ago

This commit fixes the long standing bug report #14850 in a sane way without any additional allocations and it fully supports lambdas with disabled rtti.

MaartenBent commented 3 weeks ago

The commit also changes the file permissions: include/wx/typeinfo.h 100644 → 100755

MapleLeaf-X commented 3 weeks ago

Thanks for letting me know. I'm not actually sure how it even happened but its fixed now.

PS: I'm considering perhaps to make further changes down the line to fix #16431 which I've already tested and it seem to work when rtti settings are mixed between wxWidgets and a test project. please let me know if such pr is desired.

vadz commented 3 weeks ago

Thanks for finally fixing this!