Comment # 20
on bug 8313
from Evan Huus
(In reply to comment #19)
> proto_tree_add_bytes_format (insert_tree, hf_sml_globalSignature, tvb,
> *offset, *length + *data, FALSE, "global Signature %s", (*data == 0)? ": NOT
> SET" : "");
>
> is this not allowed?
>
> It's simply FALSE instead a pointer and the field is still searchable if its
> not set. Otherwise i have to use a simple proto_tree_add_text or
> proto_tree_add_item + append text if it's not set.
In that case explicitly specifying NULL instead of FALSE would be better.
I feel like there ought to be a nicer way of doing this kind of thing in
general, but I'm not sure what it is.
You are receiving this mail because:
- You are watching all bug changes.