bugQtLua script engine for Qt - Bugs: bug #36608, Improvement suggestion

 
 

bug #36608: Improvement suggestion

Submitter:  Sergey <z80>
Submitted:  Fri 08 Jun 2012 07:41:12 AM UTC
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Open
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       No canned response available

 

Sat 14 Jul 2012 05:59:56 PM UTC, comment #1: 

I just added the State::get_lua_state function.

Alexandre Becoulet <diaxen>
Project Administrator
Fri 08 Jun 2012 07:41:12 AM UTC, original submission:  

I'm sorry in advance if my suggestion sounds stupid!

Using QtLua I met the following problem. As far as QtLua works in the same thread as GUI for long time running scrits software stalls.

For Lua versions 5.1.x I've found the only one ability to overcome that. I register lua_hook function where I call qApp->processEvents() and tracing a code.

The problem is I failed to find any rational way to get lua_State pointer from QtLua::State. So I get it using global variable and global mutex. See lines 133-138 of attached binder.cpp file. I find this way to be not very elegant.

So my suggestion is to implement either an ability to execute long time running scripts or to provide an ability to get lua_State pointer.

I do believe I could add both issues and provide them as a patch or something like that.

Sergey <z80>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #25993:  binder.cpp added by z80 (11KiB - text/x-c++src)
file #25994:  binder.h added by z80 (2KiB - text/x-chdr)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by diaxen (Posted a comment)
  • -email is unavailable- added by z80 (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

     

     

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-07-14 diaxen StatusNone Fixed
    2012-06-08 z80 Attached File- Added binder.cpp, #25993
        Attached File- Added binder.h, #25994

    Back to the top


    Powered by Savane 3.9