Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.
Wireshark logo

Wireshark-dev: [Wireshark-dev] Remove check_col() references from doc/README.developer?

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Stephen Fisher <steve@xxxxxxxxxxxxxxxxxx>
Date: Thu, 10 Sep 2009 20:11:24 -0600
Should we remove the references to check_col() calls in doc/ README.developer? At least the ones that handle the COL_INFO and COL_PROTOCOL columns? I glanced at epan/column-utils.c and noticed that the normal column setting functions check for us now, but others such as col_set_abs_date_time() don't check for us. Should these check for the column's existence at the beginning of the function also? 


Steve
Wireshark logo footer

© Wireshark Foundation · Privacy Policy

Facebook·Mastodon·Twitter·YouTube