diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2017-09-29 20:55:23 +0200 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-10-01 15:11:10 +0000 |
commit | 9ab17810bcc334ca76e4b470f7302013dc0bd261 (patch) | |
tree | c891ae2749a28afde623009f4d1f21da60a7cf1b /capchild | |
parent | d8903ce3975b57c3f4cbd24d58994a1db8c572c8 (diff) |
stats_tree: fix the comparison routine for COL_AVERAGE
Generally, the average is calculated as node->total / node->count.
The curent code does not handle the case where we compare two nodes
and both have count == 0. It defines one of the nodes to be bigger.
This triggers (at least on Windows) an assertion about invalid operator<.
To fix this, we define average = 0 for a node with count == 0.
We can then simply compare the two averages.
Change-Id: Ie7d9cd590deddcdb9214c4a2693c2eb47c66b287
Reviewed-on: https://code.wireshark.org/review/23799
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Petri-Dish: Martin Kaiser <wireshark@kaiser.cx>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'capchild')
0 files changed, 0 insertions, 0 deletions