On Mon, Dec 04, 2006 at 03:59:09PM -0000, Douglas Pratley wrote: > Corrected patch; epan/column.c and epan/column_utils.c were not > included. This one has now been properly tested against a clean > checkout of today's code. > > - New menu option available under view\time display format > - New sub-option (e) to -t switch for both wireshark and tshark > - Extended recent settings code to handle new value > - Did NOT add new explicit epoch time column
Looks good! I have committed your changes as SVN revision 20040 and added you to the authors list - thanks taking the time to implement this :). I made a few minor changes to formatting to line up tabs, spacing, etc. I removed the comment about being linked against the GNU C library as that's only the case on (Linux?) and not other systems we support, such as the BSDs. Anyway, time_t is defined so that 0 = epoch (Jan 1 1970 00:00:00) on any standard system so it is ok in your code. I added the new e option to -t to the wireshark.pod / tshark.pod (for man pages and HTML help pages) and the User's Guide (docbook/wsug_src/WSUG_chapter_customize.xml). Finally, I removed two unused variables from col_set_epoch_time in column-utils.c to squelch compiler warnings. Steve _______________________________________________ Wireshark-dev mailing list [email protected] http://www.wireshark.org/mailman/listinfo/wireshark-dev
