Wireshark-commits: [Wireshark-commits] master 4d4190f: Have wmem conform to checkAPIs.pl
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 16:01:16 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4d4190f12791ca04599c616800b6f7803b1d5f54
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

4d4190f by Michael Mann (mmann78@xxxxxxxxxxxx):

    Have wmem conform to checkAPIs.pl
    
    Yes, the rename of structure members is a bit hacky.
    Yes, catering to Windows since "GLib's v*printf routines are
    surprisingly slow on Windows".
    But it does pass checkAPIs.pl
    
    Change-Id: I5b1552472c83aa2e159f17b5b7eb70b37d03eff9
    Reviewed-on: https://code.wireshark.org/review/15404
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  f3e1208   Qt: add some cleanup function calls missing when leaving application
    adds  4d4190f   Have wmem conform to checkAPIs.pl


Summary of changes:
 doc/README.wmem                       |   12 ++++++------
 epan/wmem/wmem_allocator.h            |    6 +++---
 epan/wmem/wmem_allocator_block.c      |    6 +++---
 epan/wmem/wmem_allocator_block_fast.c |    6 +++---
 epan/wmem/wmem_allocator_simple.c     |    6 +++---
 epan/wmem/wmem_allocator_strict.c     |    6 +++---
 epan/wmem/wmem_core.c                 |    6 +++---
 epan/wmem/wmem_strutl.c               |    3 ++-
 8 files changed, 26 insertions(+), 25 deletions(-)