Ethereal-cvs: [Ethereal-cvs] cvs commit: ethereal/epan plugins.c proto.c proto.h

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Guy Harris <guy@xxxxxxxxxxxxxxxxxxx>
Date: Tue, 28 Aug 2001 03:28:18 -0500 (CDT)
guy         2001/08/28 03:28:18 CDT

  Modified files:
    epan                 plugins.c proto.c proto.h 
  Log:
  Get rid of "proto_tree_add_notext()" - if you create a subtree using it,
  but, before you set the text, you throw an exception while putting stuff
  under the subtree, you end up with an absolutely blank protocol tree
  item, which is really gross.  Instead of calling
  "proto_tree_add_notext()", call "proto_tree_add_text()" with at least a
  minimal label - yes, it does mean you do some work that will probably be
  unnecessary, but, absent a scheme to arrange to do that work if it *is*
  necessary (e.g., catching exceptions), the alternative is an ugly
  protocol tree display.
  
  Revision  Changes    Path
  1.31      +1 -2      ethereal/epan/plugins.c
  1.33      +7 -7      ethereal/epan/proto.c
  1.15      +1 -5      ethereal/epan/proto.h