Wireshark mailing list archives
Re: [Wireshark-commits] rev 40877: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dns.c
From: Alexis La Goutte <alexis.lagoutte () gmail com>
Date: Wed, 8 Feb 2012 16:37:23 +0100
On Wed, Feb 8, 2012 at 3:16 PM, Jeff Morriss <jeff.morriss.ws () gmail com>wrote:
Joerg Mayer wrote: [...] So more than half of all the stuff is added by using proto_tree_add_text.As long as the ratio is that way, people are likely to continue using it inside this dissector. Any volunteer(s) to get this down to some sane level by replacing it by proto_tree_add_item and adding hf_ entries where possible to make these Elements filterable? Should something like the above check be added to one of the check scripts to complain if the add_text percentage is above 10% or so?Done in r40930 though there's a lot of dissectors with the problem so I chose 50% as the warning level for now. And the code isn't super fast either. So it's commented out for now. Maybe another command line option is in order?
Nice ! About the speed of code, why not include in checkhf ? (it is only launch by dev when submitted new dissector !) It is possible to send some stats (Top 10 of worst dissector...)
______________________________**______________________________** _______________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/**lists/wireshark-dev<http://www.wireshark.org/lists/wireshark-dev> Unsubscribe: https://wireshark.org/mailman/**options/wireshark-dev<https://wireshark.org/mailman/options/wireshark-dev> mailto:wireshark-dev-request@**wireshark.org<wireshark-dev-request () wireshark org> ?subject=**unsubscribe
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: [Wireshark-commits] rev 40877: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dns.c Joerg Mayer (Feb 06)
- Re: [Wireshark-commits] rev 40877: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dns.c Alexis La Goutte (Feb 06)
- Re: [Wireshark-commits] rev 40877: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dns.c Jeff Morriss (Feb 08)
- Re: [Wireshark-commits] rev 40877: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dns.c Joerg Mayer (Feb 08)
- Re: [Wireshark-commits] rev 40877: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dns.c Alexis La Goutte (Feb 08)