This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[ob] fix varobj.type comment


Daniel has noted that the comment for the type member of
struct varobj say the type is never NULL.  This is plain wrong,
fixed thusly, and checked in as obvious.

- Volodya

? 1.diff
? 2.diff
? 3.diff
? fix_comment.diff
? unbreak_mi.diff
Index: ChangeLog
===================================================================
RCS file: /cvs/src/src/gdb/ChangeLog,v
retrieving revision 1.8661
diff -u -p -r1.8661 ChangeLog
--- ChangeLog	31 Aug 2007 18:52:04 -0000	1.8661
+++ ChangeLog	31 Aug 2007 19:00:50 -0000
@@ -1,5 +1,11 @@
 2007-08-31  Vladimir Prus  <vladimir@codesourcery.com>
 
+	* varobj.c (struct varobj): Fix comment
+	for the type member not to lie when it can be 
+	NULL.
+	
+2007-08-31  Vladimir Prus  <vladimir@codesourcery.com>
+
 	Implement -var-info-path-expression.
 
 	* mi/mi-cmds.h (mi_cmd_var_info_path_expression):
Index: varobj.c
===================================================================
RCS file: /cvs/src/src/gdb/varobj.c,v
retrieving revision 1.94
diff -u -p -r1.94 varobj.c
--- varobj.c	31 Aug 2007 18:52:04 -0000	1.94
+++ varobj.c	31 Aug 2007 19:00:51 -0000
@@ -110,7 +110,9 @@ struct varobj
   /* Index of this variable in its parent or -1 */
   int index;
 
-  /* The type of this variable. This may NEVER be NULL. */
+  /* The type of this variable.  This can be NULL
+     for artifial variable objects -- currently, the "accessibility" 
+     variable objects in C++.  */
   struct type *type;
 
   /* The value of this expression or subexpression.  A NULL value

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]