* breakpoint.c (breakpoint_cond_eval): Fix and enhance comment.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 15722499bc3cd0cfbcf0c30ebda9f480d850a1cb..c86f25de2320d5e73c5609ac0b2eec1d1b05aad7 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2013-11-13  Doug Evans  <xdje42@gmail.com>
+
+       * breakpoint.c (breakpoint_cond_eval): Fix and enhance comment.
+
 2013-11-13  Joel Brobecker  <brobecker@adacore.com>
 
        * mi/mi-main.c (mi_cmd_list_features): Replace "info-ada-exceptions"

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index 36252ee542302e66f7459acbeece981f8ad07922..f0b496d7080aff09a7a7e036e5b01db1389381b4 100644 (file)
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -4649,10 +4649,12 @@ bpstat_print (bpstat bs, int kind)
   return PRINT_UNKNOWN;
 }
 
-/* Evaluate the expression EXP and return 1 if value is zero.  This is
-   used inside a catch_errors to evaluate the breakpoint condition.
+/* Evaluate the expression EXP and return 1 if value is zero.
+   This returns the inverse of the condition because it is called
+   from catch_errors which returns 0 if an exception happened, and if an
+   exception happens we want execution to stop.
    The argument is a "struct expression *" that has been cast to a
-   "char *" to make it pass through catch_errors.  */
+   "void *" to make it pass through catch_errors.  */
 
 static int
 breakpoint_cond_eval (void *exp)