On 05/03/2015 06:19 PM, Wireshark code review wrote:
URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c0978c46173992c87e18bfd635bef826835ad1af
Submitter: Guy Harris ([email protected])
Changed: branch: master-1.10
Repository: wireshark

Commits:

c0978c4 by Guy Harris ([email protected]):

     Don't dissect otherwise-undissected data twice.

     Also, set decoded to TRUE after we return from a dissector; that makes
     it a bit clearer when we're setting it.

This change (in master-1.10) is dependent on (the packet-atm.c portion of) SVN rev 54503. Without back porting 54503 the 'decoded' variable is set-but-not-used in this branch.

Any preferences on which way to fix it? My initial suggestion would be to revert this latest change.

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:[email protected]?subject=unsubscribe

Reply via email to