Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash when run lua.exe wxluasudoku.wx.lua. #38

Closed
ValeryKV opened this issue Jul 22, 2019 · 5 comments
Closed

Crash when run lua.exe wxluasudoku.wx.lua. #38

ValeryKV opened this issue Jul 22, 2019 · 5 comments
Assignees

Comments

@ValeryKV
Copy link

I built wxLua 3.0 with Lua 5.2, wxWidgets 2.9.5 under Windows XP. I ran samples with lua.exe. veryminimal.wx.lua and minimal.wx.lua work great. When I ran wxluasudoku.wx.lua, it crashes with the error:
wxluasudoku.wx.lua:4901: attempt to call method 'CopyFromBitmap' (a nil value)
wxluasudoku.wx.lua is started OK with wxLua 2.8.

wxcore_gdi.i contains the declaration of wxIcon:
%wxchkver_3_0_0 void CopyFromBitmap(const wxBitmap& bmp);

Most probably that means that I have to use wxWidgets version 3.0 or above.

wxLua 2.8 was release on 2012, but last commit to wxLua 3.0 was done several days ago. That is why I chose wxLua 3.0 when I wanted to write my first app with wxWidgets and lua. I also want to use Lua 5.3 that wxLua 3.0 supports.

Does wxLua 3.0 compatible with wxWidgets 2.9?

@pkulchenko pkulchenko self-assigned this Jul 22, 2019
@pkulchenko
Copy link
Owner

@ValeryKV, this should be fixed in the commit I just pushed, but 2.9.5 version is really old, so you should probably be using 3.0 or 3.1 versions.

@ValeryKV
Copy link
Author

Now lua.exe wxluasudoku.wx.lua crashes at bmp:delete() in MItem function. I tried to comment some code, but I always got a crash somewhere. May be you should add a note that this wxLua version is not compatible with wxWidgets 2.x. But maybe the issue is in VC 6.0.
Thank you for fixed bugs.

@pkulchenko
Copy link
Owner

I don't get any errors in the wxluasudoku example compiling with 2.9.5 on Windows (however I'm using mingw, as I don't have MSVC). It would be unusual to crash on "delete", as this function is wxlua-specific (not wxwidgets) and hasn't changed since wxlua 2.8.12.

wxlua is compatible with wxWidgets 2.x, but it doesn't implement all the functions exposed in the API and while I try to preserve backward compatibility, 2.9.5 was released 6 years ago and I didn't feel the need to retest on such an old release.

I've confirmed the current master compiling against wxwidgets 2.9.5, 3.0 and 3.1 versions.

@ValeryKV
Copy link
Author

ValeryKV commented Jul 25, 2019

Yes, the issue is in VC 6.0 compiler in wxWidgets code. I filed a bug https://trac.wxwidgets.org/ticket/18453. Maybe they will decide to fix it.

@pkulchenko
Copy link
Owner

ok; thank you for the update. Nice analysis in the wxwidgets ticket!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants