Wireshark-bugs: [Wireshark-bugs] [Bug 4700] WSLUA improvement (ProtoField.bool extension)

Date: Thu, 28 Oct 2010 13:24:13 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4700

Beth <beth.tridium@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
                 CC|                            |beth.tridium@xxxxxxxxx
         Resolution|FIXED                       |

--- Comment #20 from Beth <beth.tridium@xxxxxxxxx> 2010-10-28 13:24:07 PDT ---
The change in wslua_tree.c seems to be problematic for me.  I define a proto
field as follows:

  local F_isHop1  = ProtoField.bool("myproto.isHop1", "1 if 1st hop of route")

Later, I add the value to the display tree as follows:

  local oh = subtreeitem:add(F_isHop1, tvbuffer(), gotrec.isHop1)

The latter line causes this error message to appear in the display pane:

    Lua error: ...\foo.lua:1003: calling 'add' on bad self (number expected,
got boolean)


So it seems that Lua no longer wants a boolean as the third argument to
treeitem:add()?  


The same Lua script works fine under 1.2.11, so I'm wondering if it's the
change from luaL_checkboolean to luaL_checknumber that is causing the problem.


If this isn't a bug, and I simply need to update my syntax, then let me know.
Thanks.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.