Wireshark-commits: [Wireshark-commits] master 9dc3d67: wmem: get rid of the allocator timing test
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 3 Aug 2014 07:36:07 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9dc3d67672e27bd185ea8139444d7642d2b91d56
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

9dc3d67 by Evan Huus (eapache@xxxxxxxxx):

    wmem: get rid of the allocator timing test
    
    Some of the allocators are close enough that a little bit of system load at the
    time of the test was enough to throw them off, and it was causing unnecessary
    buildbot failures. Benchmarking on real-world cases at the time of
    implementation (as well as reliable "average" results on the buildbots) has left
    me confident that block_fast is, in fact, faster than block, which is faster
    than simple/strict.
    
    Additionally, the allocators are effectively "done" and unlikely to change, so
    the possibility of uncaught performance regressions is low.
    
    Change-Id: I144ea4cd9d6000edc4075ed682ce6a4aeaf5e28a
    Reviewed-on: https://code.wireshark.org/review/3356
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  fb68a0c   Have ConversationDialog retain data between captures.
    adds  9dc3d67   wmem: get rid of the allocator timing test


Summary of changes:
 epan/wmem/wmem_test.c |   59 -------------------------------------------------
 1 file changed, 59 deletions(-)