We already have ENC_ASCII, ENC_UTF8, etc. Perhaps
proto_tree_add_string should just take an encoding value instead?
On Sat, Dec 7, 2013 at 7:20 AM, Jakub Zawadzki
<darkjames-ws@xxxxxxxxxxxx> wrote:
> Hi,
>
> I renamed base_display_e to field_display_e, and added new enums to field_display_e,
> but I wonder if it's correct approach.
>
> For FT_ABSOLUTE_TIME we're using seperate enum
> (absolute_time_display_e), so maybe FT_STRING* should also have own enum string_display_e ?
>
>
> Anyway, dissecotrs using proto_tree_add_unicode_string():
> packet-gsm_sms.c
> packet-etsi_card_app_toolkit.c
> packet-rrc.c
> packet-ptpip.c
> packet-lte-rrc.c
> packet-gmr1_rr.c
> packet-ansi_637.c
>
> Should probably change it back to proto_tree_add_item/ proto_tree_add_string and start using STR_UNICODE display.
> If you need some extra flag like: STR_UNICODE_DONT_ESCAPE than go on.
>
> I'll change packet-ip, packet-ipv6, packet-json.
>
>
> Kuba.
> ___________________________________________________________________________
> 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