Add comments to currently_stepping and target_resume
authorPedro Alves <palves@redhat.com>
Fri, 7 Aug 2015 16:23:59 +0000 (17:23 +0100)
committerPedro Alves <palves@redhat.com>
Fri, 7 Aug 2015 16:23:59 +0000 (17:23 +0100)
Clarify that currently_stepping works at a higher level than
target_resume.

gdb/ChangeLog:
2015-08-07  Pedro Alves  <palves@redhat.com>

* infrun.c (currently_stepping): Extend intro comment.
* target.h (target_resume): Extend intro comment.

gdb/ChangeLog
gdb/infrun.c
gdb/target.h

index bd9f96905e9855010aed0a7fc800c5858a5c7dac..3e4cc04c5c414b2cdb8f6e3651a52a3e1546830a 100644 (file)
@@ -1,3 +1,8 @@
+2015-08-07  Pedro Alves  <palves@redhat.com>
+
+       * infrun.c (currently_stepping): Extend intro comment.
+       * target.h (target_resume): Extend intro comment.
+
 2015-08-07  Pedro Alves  <palves@redhat.com>
 
        * infrun.c (switch_back_to_stepped_thread): Use ecs->ptid instead
index 455fc2d9609245dd93f4315992dea07dba3689ec..32eeca8b426d6717b39794bddbe1f1dbfd90515d 100644 (file)
@@ -5854,7 +5854,9 @@ switch_back_to_stepped_thread (struct execution_control_state *ecs)
   return 0;
 }
 
-/* Is thread TP in the middle of single-stepping?  */
+/* Is thread TP in the middle of (software or hardware)
+   single-stepping?  (Note the result of this function must never be
+   passed directly as target_resume's STEP parameter.)  */
 
 static int
 currently_stepping (struct thread_info *tp)
index 462c0dee562d1b5aea8fa30de27c4649d7a2544c..d6167980ac8049877336836a2bf9f5d18cc9c9b0 100644 (file)
@@ -1292,8 +1292,8 @@ extern void target_detach (const char *, int);
 extern void target_disconnect (const char *, int);
 
 /* Resume execution of the target process PTID (or a group of
-   threads).  STEP says whether to single-step or to run free; SIGGNAL
-   is the signal to be given to the target, or GDB_SIGNAL_0 for no
+   threads).  STEP says whether to hardware single-step or to run free;
+   SIGGNAL is the signal to be given to the target, or GDB_SIGNAL_0 for no
    signal.  The caller may not pass GDB_SIGNAL_DEFAULT.  A specific
    PTID means `step/resume only this process id'.  A wildcard PTID
    (all threads, or all threads of process) means `step/resume