On 06/18/14 10:14, Wireshark code review wrote:
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=246fe2ca4c67d8c98caa84e2f57694f6322e2f96 Submitter: Anders Broman (a.broma...@gmail.com) Changed: branch: master Repository: wiresharkCommits: 246fe2c by Dario Lombardo (lom...@gmail.com): Fixup: tvb_* -> tvb_captured
[...]
epan/dissectors/packet-cast.c | 6 +- ... 544 lines suppressed ...
Hmm, as Evan said[1], I thought the reason these hadn't all been mass-converted was because they were supposed to be manually reviewed to see which of the new APIs were supposed to be used; this is especially true with the tvb_length() API because it has been a very common mistake to use tvb_length() when what was intended was tvb_reported_length().
[1] https://www.wireshark.org/lists/wireshark-dev/201406/msg00131.html ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe
