Wireshark-commits: [Wireshark-commits] master 7f2a838: lua: Allow proto:register_heuristic to be us
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 13 Nov 2016 17:48:56 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=7f2a83892204821145768b76bbdd0719b57787f8
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

7f2a838 by Franklin "Snaipe" Mathieu (snaipe@xxxxxxxxxxxx):

    lua: Allow proto:register_heuristic to be used on multiple list names
    
    In the C API, one can register a heuristic for the same protocol on different
    lists by specifying another unique short_name. This is impossible in the
    lua API, as the protocol name is used as the short name itself.
    
    This change fixes that by creating an unique shortname composed of the
    protocol name and the target list name.
    
    Change-Id: I2c30ce6e4f7a3b38879180c64cf8564f779163b4
    Signed-off-by: Franklin "Snaipe" Mathieu <snaipe@xxxxxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/18711
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  67e1ed5   [Automatic update for 2016-11-13]
    adds  7f2a838   lua: Allow proto:register_heuristic to be used on multiple list names


Summary of changes:
 epan/wslua/wslua_proto.c |   13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)