Wireshark-commits: [Wireshark-commits] master-1.10 31f3669: Lua: FieldInfo '==' check isn't really
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 4 Jan 2015 03:32:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=31f3669335d335d58b2bc119bced034f41cccf72
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

31f3669 by Hadriel Kaplan (hadrielk@xxxxxxxxx):

    Lua: FieldInfo '==' check isn't really equality
    
    The FieldInfo metamethod for equality (letting you use the '==' operator)
    doesn't check for equality, but rather if the left-hand side is within
    the right-hand side. It should be equality instead. Also, all of the
    FieldInfo operate overloads should push a boolean even if they're false
    result.
    
    Ping-Bug: 10820
    Change-Id: Ibddaab29713f26d22ddb4d5804b9edb15e93fd79
    Reviewed-on: https://code.wireshark.org/review/6186
    Petri-Dish: Hadriel Kaplan <hadrielk@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    (cherry picked from commit 445ddc833669e8cff7e66b268d4b4b77faa504cc)
    Reviewed-on: https://code.wireshark.org/review/6295
    Reviewed-by: Hadriel Kaplan <hadrielk@xxxxxxxxx>
    

Actions performed:

    from  17268c4   IPv6: fix compilation when building with GeoIP support
    adds  31f3669   Lua: FieldInfo '==' check isn't really equality


Summary of changes:
 epan/wslua/wslua_field.c |   20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)