Wireshark-commits: [Wireshark-commits] master 4ec84a3: Fix (spurious) checkAPIs issue
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: Fri, 13 May 2016 19:57:36 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4ec84a3e2b1023bb6d27fc4fdb3d69cd2385f2e2
Submitter: Graham Bloice (graham.bloice@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

4ec84a3 by Graham Bloice (graham.bloice@xxxxxxxxxxxxx):

    Fix (spurious) checkAPIs issue
    
    checkAPIs complains that the system member of the kingfisher_packet_t
    structure shadows a system variable.
    
    Althoguh this is a false positive, rename the variable to keep
    checkAPIs happy.
    
    Change-Id: Ia356dea5abb3ed93d10c5057f55786e2f28cec11
    Reviewed-on: https://code.wireshark.org/review/15414
    Petri-Dish: Graham Bloice <graham.bloice@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Graham Bloice <graham.bloice@xxxxxxxxxxxxx>
    

Actions performed:

    from  0766f78   packet-nfs: Add flex file flag: FF_FLAGS_NO_READ_IO
    adds  4ec84a3   Fix (spurious) checkAPIs issue


Summary of changes:
 epan/dissectors/packet-kingfisher.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)