wxlua-users Mailing List for wxLua

On Jan 7, 2008 2:27 PM, John Labenski <jla...@gm...> wrote:

> These are common in wxWidgets in GTK even with C++ code. All it means
> in this case is that the wxSizer is trying to size the wxFrame or
> wxButton before it had a chance to get a valid size so it's using
> wx.wxDefaultSize = (-1, -1).


OK. How come the application just doesn't work?

> Here is a bit more information. It displays the frame as a very small
> frame
> > or as a VERY long titlebar only. It displays the errors in the terminal
> when
> > I run it. If I comment out the SetSizeHints() I get the frame to show up
> > correctly, but it only has the menu. I don't see the controls. This
> works on
> > Windows, but not on Linux. I hope I am doing something wrong.
>
> Not really, but you might want to provide some positive initial size
> for the wxFrame to get rid of the warnings.


I already tried this and it didn't work either. I send another email about
wxSizer::Add() in Windows not functioning correctly when wxLua is used as a
Lua module. I wonder if this has anything to do with that.


> Also and I hope this is all it takes (can't test now) is that you
> create a single wxPanel child of the wxFrame. A single wxPanel child
> of a wxFrame takes the client size of the wxFrame and then you use a
> wxSizer to hold the children of the panel. Note how the sizer is for
> the panel, but we set the size hints for the wxFrame.
>
> frame = wx.wxFrame(wx.NULL, -1, "wxLua module sample")
> panel = wx.wxPanel(frame, -1)
>
> -- Layout all the buttons using wxSizers
> local mainSizer = wx.wxBoxSizer( wx.wxVERTICAL )
> local ID_BUTTON = 1001
> local buildButton = wx.wxButton( panel, ID_BUTTON, "Button1" )
> mainSizer:Add( buildButton, 0, wx.wxALL + wx.wxEXPAND, 5 )
> panel:SetSizer( mainSizer )
> mainSizer:SetSizeHints( frame )
>

In the real application that I am trying to write I do use a wxPanel. It
still doesn't work. I can send the source for the application if that would
help? It is just a bit larger. It does function when you double-click on the
lua file. (I used the installer to register the OPEN command with wxLua) ANy
other ideas?

I will check to make sure that I am I am SetSizeHints() on the frame.
-- 
Regards,
Ryan

On Jan 7, 2008 5:27 AM, Klaas Holwerda <db...@nl...> wrote:

>
> Euuh, no it crashes exactly as the stack above shows, luaO_rawequalObj, put i
> think the damage is already done at/before that moment.
> For some reason i think what is done in wxluaT_pushuserdatatype is ruining the rest.
>

I think I see the problem, I don't pop an extra value we pushed.

bool LUACALL wxluaO_istrackedweakobject(lua_State *L, void *obj_ptr,
int wxl_type, bool push_on_stack)
...
    if (lua_istable(L, -1))
    {
        lua_pushnumber(L, wxl_type); // push key
        lua_rawget(L, -2);           // get t[key] = value; pops key

        // check if they've dynamic casted the object or if it was casted in C++
        if (wxlua_iswxuserdata(L, -1) && (wxl_type == wxluaT_type(L, -1)))
        {
            if (push_on_stack)
            {
                lua_remove(L, -3); // remove the obj table, leave
value on the stack
                lua_remove(L, -2); // remove table of userdata, leave
value on the stack
            }
            else
                lua_pop(L, 3);

            return true;
        }
        else
            lua_pop(L, 1); // pop the userdata that is not the right type
    }


------------------------
Add these two lines of code.

        else
            lua_pop(L, 1); // pop the userdata that is not the right type

-------------------------

I can't commit it right now since making static member variables work
is not finished yet.

Thanks,
    John

On Jan 5, 2008 2:25 PM, Ryan Pusztai <rpu...@gm...> wrote:
> I am getting all kinds of errors when using the wxLua Lua module in Linux.
> In the Terminal I see errors for sizing upon creation. Here are the errors I
> get when running:
>
> > $ lua luamodule.wx.lua
> >
> > (<unknown>:30818): Gtk-CRITICAL **: gtk_window_resize: assertion `height >
> 0' failed

These are common in wxWidgets in GTK even with C++ code. All it means
in this case is that the wxSizer is trying to size the wxFrame or
wxButton before it had a chance to get a valid size so it's using
wx.wxDefaultSize = (-1, -1).

> Here is a bit more information. It displays the frame as a very small frame
> or as a VERY long titlebar only. It displays the errors in the terminal when
> I run it. If I comment out the SetSizeHints() I get the frame to show up
> correctly, but it only has the menu. I don't see the controls. This works on
> Windows, but not on Linux. I hope I am doing something wrong.

Not really, but you might want to provide some positive initial size
for the wxFrame to get rid of the warnings.

Also and I hope this is all it takes (can't test now) is that you
create a single wxPanel child of the wxFrame. A single wxPanel child
of a wxFrame takes the client size of the wxFrame and then you use a
wxSizer to hold the children of the panel. Note how the sizer is for
the panel, but we set the size hints for the wxFrame.

frame = wx.wxFrame(wx.NULL, -1, "wxLua module sample")
panel = wx.wxPanel(frame, -1)

-- Layout all the buttons using wxSizers
local mainSizer = wx.wxBoxSizer( wx.wxVERTICAL )
local ID_BUTTON = 1001
local buildButton = wx.wxButton( panel, ID_BUTTON, "Button1" )
mainSizer:Add( buildButton, 0, wx.wxALL + wx.wxEXPAND, 5 )
panel:SetSizer( mainSizer )
mainSizer:SetSizeHints( frame )

Hope this helps,
    John

On Jan 7, 2008 6:10 AM, Klaas Holwerda <db...@nl...> wrote:
> Hi,
>
> Don't know if its bad, but i get this:
>
> C:\data\art2d\wxluacvs\wxLua\bindings>genwxbind.bat
> Generating wxWidgets Binding
> ..\bin\vc_lib\lua.exe: genwxbind.lua:4882: attempt to get length of global 'inte
> rface_fileTable' (a nil value)

Sorry, I didn't update the paths for the the genwxbind.bat file. I've
been using the Makefile with cygwin if I'm in MS Windows. I've fixed
it in CVS so it works now.

Thanks,
    John

Hi John,

I managed to change the wxLuaCan sample to create the same error.
If you unpack the source code in the attached file, to apps/wxluacan/src,
and run the sample, you get it.

The two pointers are:

wxlLuaCanCmd* wxlLuaCanCmd1;
wxlLuaCanCmd* wxlLuaCanCmd2;

And in wxluacan.i they are wrapped like here:

%class %delete %noclassinfo wxlLuaCanCmd, wxCommandProcessor

     %define_pointer wxlLuaCanCmd1

     wxlLuaCanCmd( wxlCan* canvas, int maxCommands = -1 )
     void MoveObject( int index, double x, double y )
%endclass

%class %delete %noclassinfo wxlLuaCanCmdDer, wxlLuaCanCmd

     %define_pointer wxlLuaCanCmd2

     wxlLuaCanCmdDer( wxlCan* canvas, int maxCommands = -1 )
%endclass

Stack at crash time is:

luaO_rawequalObj(const lua_TValue * 0x4c6d97c7, const lua_TValue * 0x00f77f48) 
line 73 + 6 bytes
luaH_get(Table * 0x011d4538, const lua_TValue * 0x00f77f48) line 484 + 16 bytes
luaH_set(lua_State * 0x00f77c58, Table * 0x011d4538, const lua_TValue * 
0x00f77f48) line 495 + 13 bytes
lua_rawset(lua_State * 0x00f77c58, int -3) line 673 + 37 bytes
wxLuaBinding::DoRegisterBinding(const wxLuaState & {...}, int 1) line 1192 + 11 
bytes
wxLuaBinding::RegisterBinding(const wxLuaState & {...}) line 986
wxLuaState::RegisterBindings() line 2729
wxLuaState::Create(lua_State * 0x00f77c58, int 2) line 2262
wxLuaState::Create(wxEvtHandler * 0x00f63c40, int -1) line 2157 + 14 bytes
wxLuaState::wxLuaState(wxEvtHandler * 0x00f63c40, int -1) line 657 + 75 bytes
MyFrame::MyFrame(wxWindow * 0x00000000, int -1, const wxString & {...}, const 
wxPoint & {...}, const wxSize & {...}, long 541072960) line 94 + 17 bytes
MyApp::OnInit() line 320 + 107 bytes
wxAppConsole::CallOnInit() line 76 + 28 bytes
wxEntryReal(int & 1, char * * 0x00f649b8) line 440 + 25 bytes
wxEntry(int & 1, char * * 0x00f649b8) line 209 + 13 bytes
wxEntry(HINSTANCE__ * 0x00400000, HINSTANCE__ * 0x00000000, HINSTANCE__ * 
0x00000000, int 1) line 386 + 13 bytes
WinMain(HINSTANCE__ * 0x00400000, HINSTANCE__ * 0x00000000, char * 0x00151f16, 
int 1) line 45 + 24 bytes
WinMainCRTStartup() line 330 + 54 bytes
KERNEL32! 77e735dd()


I think this makes it much easier to find for you,

Klaas



-- 
Unclassified

Hi,

Don't know if its bad, but i get this:

C:\data\art2d\wxluacvs\wxLua\bindings>genwxbind.bat
Generating wxWidgets Binding
..\bin\vc_lib\lua.exe: genwxbind.lua:4882: attempt to get length of global 'inte
rface_fileTable' (a nil value)
stack traceback:
         genwxbind.lua:4882: in function 'main'
         genwxbind.lua:4903: in main chunk
         [C]: ?
Generating wxStyledTextCtrl Binding
..\bin\vc_lib\lua.exe: genwxbind.lua:4882: attempt to get length of global 'inte
rface_fileTable' (a nil value)
stack traceback:
         genwxbind.lua:4882: in function 'main'
         genwxbind.lua:4903: in main chunk
         [C]: ?
Generating wxLua Binding
Generating wxLuaDebugger Binding
..\bin\vc_lib\lua.exe: genwxbind.lua:3073: bad argument #1 to 'find' (string exp
ected, got nil)
stack traceback:
         [C]: in function 'find'
         genwxbind.lua:3073: in function 'GenerateLuaLanguageBinding'
         genwxbind.lua:1465: in function 'WriteWrapperFiles'
         genwxbind.lua:4884: in function 'main'
         genwxbind.lua:4903: in main chunk
         [C]: ?
Generating wxLuaCan app Binding
Done.
C:\data\art2d\wxluacvs\wxLua\bindings>

-- 
Unclassified

Hi John,

Latest CVS, but still not :-(

John Labenski wrote:

> It is normal that it can't find the object when you're registering the
> bindings since it's being push for the first time. The whole idea
> behind the check for wxluaO_istrackedweakobject() is that we don't
> want to create multiple Lua userdata wrapping our C++ object, but push
> the same one again. However, just like in C++ the object can be cast
> to different types and wxLua must create new userdata for each type to
> distinguish them (for wxObject::DynamicCast()).

Oke, i thought it would be something like that ;-)

> 
> Exactly, the second time through it finds that the object was already
> pushed, but with a different type so as far as wxLua is concerned,
> it's not what we want and returns false.

oke.

> 
>> ===============Stack ================
>>
>> luaO_rawequalObj(const lua_TValue * 0x4d6584b1, const lua_TValue * 0x02006608)
>> line 73 + 6 bytes
>> luaH_get(Table * 0x02174b60, const lua_TValue * 0x02006608) line 484 + 16 bytes
>> luaH_set(lua_State * 0x02006318, Table * 0x02174b60, const lua_TValue *
>> 0x02006608) line 495 + 13 bytes
>> lua_rawset(lua_State * 0x02006318, int -3) line 673 + 37 bytes
> 
> ??? What happens here?

LUA_API void lua_rawset (lua_State *L, int idx) {

.......... In the next liene it goes to luaH_set, where you can see above it 
goes a bit further and generates the exception. I examined L and t.

   setobj2t(L, luaH_set(L, hvalue(t), L->top-2), L->top-1);
..........
}

But in luaH_set calls luaH_get

const TValue *luaH_get (Table *t, const TValue *key) {

These input parameters here e.g parameter t is weird, so i think  L->top-2 is 
not oke. It contains a lots of "expression  cannot be evaluated" when i 
QuickWatch it.

}

So i think the lua stack is already not oke before calling lua_rawset.

> 
> wxLuaBinding::DoRegisterBinding
> ...
>     if (wxlObject->objPtr != 0)
>         wxluaT_pushuserdatatype(L, wxlObject->objPtr,
> *wxlObject->wxluatype, true);

NO it goes into the else of the if, because its a pointer i think.

         if (wxlObject->objPtr != 0)
             wxluaT_pushuserdatatype(L, wxlObject->objPtr, 
*wxlObject->wxluatype, true);
         else //HERE
             wxluaT_pushuserdatatype(L, *wxlObject->pObjPtr, 
*wxlObject->wxluatype, true);

AND IN THE NEXT LINE ALL IS WRONG
         lua_rawset(L, -3);
> 
> then
> 
> bool LUACALL wxluaT_pushuserdatatype
> ...
>             if (wxluaR_getref(L, wxl_type, &wxlua_lreg_types_key))
>             {
>                 // pop the table and set it as the metatable for the newuserdata
>                 lua_setmetatable(L, -2);
> 
>                 if (track)
>                     wxluaO_trackweakobject(L, -1, (void*)obj_ptr, wxl_type);

It goes in here.

> 
>                 return true; // leave value on the stack

And returns here.

>             }
> 
> Does it crash in wxluaO_trackweakobject() or wxluaR_getref()?

Euuh, no it crashes exactly as the stack above shows, luaO_rawequalObj, put i 
think the damage is already done at/before that moment.
For some reason i think what is done in wxluaT_pushuserdatatype is ruining the rest.

Thanks,

Klaas

-- 
Unclassified

Ryan Pusztai wrote:

> 
>     Check to see if you have these two libs
>     libwx_gtk2ud_gl-2.8.so.0.3.0
>     libwx_gtk2ud_media-2.8.so.0.3.0

oke.

You should check the configuration wxLua tries to find/uses.

On the command line:

wx-config --libs
wx-config --selected_config

Try also wx-config --help.
Is it indeed unicode version that is found?
This is how you know exactly which libs will be linked.

If you do use this the next, be aware that there is some caching of previous 
arguments to configure, i suggest, remove all start from scratch.

./configure --with-opengl

I also noticed in your compile line, that it does not contain the 
-libwx_gtk2ud_gl-2.8.so.0.3.0.

It only has these wx libs linked in:
-lwx_gtk2u_stc-2.8 -pthread   -lwx_gtk2u_aui-2.8 -lwx_gtk2u_xrc-2.8 
-lwx_gtk2u_html- 2.8 -lwx_gtk2u_adv-2.8 -lwx_baseu_net-2.8 -lwx_baseu_xml-2.8 
-lwx_gtk2u_core-2.8 -lwx_baseu-2.8

So i think it is not found by configure of wxLua.

Can't get to Linux right now, but hope it helps.

Klaas




-- 
Unclassified

On Jan 6, 2008 4:45 PM, Ryan Pusztai <rpu...@gm...> wrote:

> The wxSizer::Add() function is not working right when wxLua is used as a
> Lua module.
>
> Example script:
>
> require( "wx" )
>
> frame = wx.wxFrame(wx.NULL, wx.wxID_ANY,  "wxLua sizer test frame")
>
> -- Create two controls (note that their parents are the _frame_ (not the
> sizer))
> textEntry = wx.wxTextCtrl(frame, 1001, "Enter URL");
> button = wx.wxButton(frame, 1002, "test")
>
> -- Put them in a vertical sizer, with ratio 3 units for the text entry, 5
> for button
> -- and padding of 6 pixels.
> sizerTop = wx.wxBoxSizer(wx.wxVERTICAL)
> sizerTop:Add(textEntry, 1, wx.wxGROW + wx.wxALL, 6)
> sizerTop:Add(button, 1, wx.wxGROW + wx.wxALL, 6)
>
> -- Set up the frame to use that sizer to move/resize its children controls
> frame:SetAutoLayout(true)
> frame:SetSizer(sizerTop)
>
> -- Optional - these will set an initial minimal size, just enough to hold
> the
> -- controls (more useful for dialogs than a frame)
> sizerTop:SetSizeHints(frame)
> sizerTop:Fit(frame)
>
> -- Start the application
> wx.wxGetApp():SetTopWindow(frame)
> frame:Show(true)
>
> -- ALWAYS call wx.wxGetApp():MainLoop() last to keep the program active
> -- otherwise the lua program will exit immediately
> wx.wxGetApp():MainLoop()
>
>
> Error output:
>
> > $ ./lua test.wx.lua
> > d:\devel\wxLua\wxBuilder\lua.exe: wxLua: Expected a number for parameter
> > 2, but got 'userdata'.
> > stack traceback:
> >         [C]: in function 'Add'
> >         test.wx.lua:21: in main chunk
> >         [C]: ?
> >
>
> What is the problem? Shouldn't this work no matter how I am using wxLua?
>

I forgot to mention that this is using the dll release of wxLua v2.8.4.2 dll
version. I am also using Windows XP.
-- 
Regards,
Ryan