Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] master 14824e6: Revert "Fixup: tvb_* ->
From: Evan Huus <[email protected]>
Date: Thu, 19 Jun 2014 12:18:50 -0700
On Thu, Jun 19, 2014 at 12:16 PM, Guy Harris <[email protected]> wrote:

On Jun 19, 2014, at 11:26 AM, Wireshark code review <[email protected]> wrote:

> URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=14824e6adf07a1c3cfb6537ce8723d873ecd0d0f
> Submitter: Michael Mann ([email protected])
> Changed: branch: master
> Repository: wireshark
>
> Commits:
>
> 14824e6 by Michael Mann ([email protected]):
>
>    Revert "Fixup: tvb_* -> tvb_captured"
>
>    https://www.wireshark.org/lists/wireshark-dev/201406/msg00131.html
>
>    This reverts commit 246fe2ca4c67d8c98caa84e2f57694f6322e2f96.

Why did this change need to be reverted?  Did it do anything other than changing calls to tvb_length() and tvb_length_remaining() to calls to tvb_captured_length() and tvb_captured_length_remaining()?

We were not bulk-converting them so that we could catch the ones that really should have been tvb_reported_length and tvb_reported_length_remaining from the beginning.
 
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe