Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] FW: [Wireshark-commits] rev 46479: /trunk/epan/dissectors/ /

From: Anders Broman <anders.broman@xxxxxxxxxxxx>
Date: Sun, 9 Dec 2012 19:18:50 +0000
Hi,
Could you try this patch?
Regards
Anders


From: wireshark-dev-bounces@xxxxxxxxxxxxx [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Anders Broman
Sent: den 9 december 2012 17:53
To: Developer support list for Wireshark
Subject: Re: [Wireshark-dev] FW: [Wireshark-commits] rev 46479: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-per.c

Yes that' was what I was thinking, Perhaps a similar aproach to what you did in your patch would be possible?
Use a buffer initiated with "."?
Unfortunatly I can't profile a solution.
Regards
Anders


From: wireshark-dev-bounces@xxxxxxxxxxxxx [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Martin Mathieson
Sent: den 9 december 2012 17:21
To: Developer support list for Wireshark
Subject: Re: [Wireshark-dev] FW: [Wireshark-commits] rev 46479: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-per.c

Maybe.  I'm guessing we'd probably want to improve decode_bits_in_field() to avoid those g_strlcat() calls by keeping track of the offset.
Martin

On Sun, Dec 9, 2012 at 11:07 AM, Anders Broman <anders.broman@xxxxxxxxxxxx> wrote:
Hi,
Would using proto_tree_add_bits_item() be less efficient? Possibly that can be improved which would give
A better over all improvement I think.
Regards
Anders

-----Original Message-----
From: wireshark-commits-bounces@xxxxxxxxxxxxx [mailto:wireshark-commits-bounces@xxxxxxxxxxxxx] On Behalf Of martinm@xxxxxxxxxxxxx
Sent: den 9 december 2012 05:50
To: wireshark-commits@xxxxxxxxxxxxx
Subject: [Wireshark-commits] rev 46479: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-per.c

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=46479

User: martinm
Date: 2012/12/08 08:49 PM

Log:
 Similarly avoiding calling g_snprintf() in dissect_per_constrained_integer().

Directory: /trunk/epan/dissectors/
  Changes    Path            Action
  +7 -1      packet-per.c    Modified

___________________________________________________________________________
Sent via:    Wireshark-commits mailing list <wireshark-commits@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
             mailto:wireshark-commits-request@xxxxxxxxxxxxx?subject=unsubscribe
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe

Attachment: to_str.patch
Description: to_str.patch