[Lldb-commits] [lldb] 394e52a - [lldb] NFC reflow comments in WatchpointLocations
Jason Molenda via lldb-commits
lldb-commits at lists.llvm.org
Wed Aug 30 17:59:49 PDT 2023
Author: Jason Molenda
Date: 2023-08-30T17:58:44-07:00
New Revision: 394e52a0bb576508e0fed59ab267e75cf1350fca
URL: https://github.com/llvm/llvm-project/commit/394e52a0bb576508e0fed59ab267e75cf1350fca
DIFF: https://github.com/llvm/llvm-project/commit/394e52a0bb576508e0fed59ab267e75cf1350fca.diff
LOG: [lldb] NFC reflow comments in WatchpointLocations
Reading through this header, many of the comment
line breaks are a mess after various code reformats.
Added:
Modified:
lldb/include/lldb/Breakpoint/WatchpointOptions.h
Removed:
################################################################################
diff --git a/lldb/include/lldb/Breakpoint/WatchpointOptions.h b/lldb/include/lldb/Breakpoint/WatchpointOptions.h
index 369120e985342b..cedd1b186d659a 100644
--- a/lldb/include/lldb/Breakpoint/WatchpointOptions.h
+++ b/lldb/include/lldb/Breakpoint/WatchpointOptions.h
@@ -59,24 +59,20 @@ class WatchpointOptions {
// synchronous callbacks: 1) They should NOT resume the target themselves.
// Just return false if you want the target to restart. 2) Watchpoints with
// synchronous callbacks can't have conditions (or rather, they can have
- // them, but they
- // won't do anything. Ditto with ignore counts, etc... You are supposed
- // to control that all through the
- // callback.
+ // them, but they won't do anything. Ditto with ignore counts, etc...
+ // You are supposed to control that all through the callback.
// Asynchronous callbacks get run as part of the "ShouldStop" logic in the
// thread plan. The logic there is:
// a) If the watchpoint is thread specific and not for this thread, continue
- // w/o running the callback.
+ // w/o running the callback.
// b) If the ignore count says we shouldn't stop, then ditto.
// c) If the condition says we shouldn't stop, then ditto.
// d) Otherwise, the callback will get run, and if it returns true we will
- // stop, and if false we won't.
+ // stop, and if false we won't.
// The asynchronous callback can run the target itself, but at present that
- // should be the last action the
- // callback does. We will relax this condition at some point, but it will
- // take a bit of plumbing to get
+ // should be the last action the callback does. We will relax this
+ // condition at some point, but it will take a bit of plumbing to get
// that to work.
- //
/// Adds a callback to the watchpoint option set.
///
@@ -87,8 +83,8 @@ class WatchpointOptions {
/// A baton which will get passed back to the callback when it is invoked.
///
/// \param[in] synchronous
- /// Whether this is a synchronous or asynchronous callback. See discussion
- /// above.
+ /// Whether this is a synchronous or asynchronous callback.
+ /// See discussion above.
void SetCallback(WatchpointHitCallback callback,
const lldb::BatonSP &baton_sp, bool synchronous = false);
@@ -102,11 +98,9 @@ class WatchpointOptions {
///
/// \param[in] context
/// The context in which the callback is to be invoked. This includes the
- /// stop event, the
- /// execution context of the stop (since you might hit the same watchpoint
- /// on multiple threads) and
- /// whether we are currently executing synchronous or asynchronous
- /// callbacks.
+ /// stop event, the execution context of the stop (since you might hit
+ /// the same watchpoint on multiple threads) and whether we are currently
+ /// executing synchronous or asynchronous callbacks.
///
/// \param[in] watch_id
/// The watchpoint ID that owns this option set.
@@ -129,23 +123,24 @@ class WatchpointOptions {
/// The baton.
Baton *GetBaton();
- /// Fetch a const version of the baton from the callback.
+ /// Fetch a const version of the baton from the callback.
///
/// \return
/// The baton.
const Baton *GetBaton() const;
/// Return the current thread spec for this option. This will return nullptr
- /// if the no thread specifications have been set for this Option yet.
+ /// if the no thread specifications have been set for this WatchpointOptions
+ /// yet.
+ ///
/// \return
/// The thread specification pointer for this option, or nullptr if none
- /// has
- /// been set yet.
+ /// has been set yet.
const ThreadSpec *GetThreadSpecNoCreate() const;
- /// Returns a pointer to the ThreadSpec for this option, creating it. if it
- /// hasn't been created already. This API is used for setting the
- /// ThreadSpec items for this option.
+ /// Returns a pointer to the ThreadSpec for this option, creating it if it
+ /// hasn't been created already. This API is used for setting the
+ /// ThreadSpec items for this WatchpointOptions.
ThreadSpec *GetThreadSpec();
void SetThreadID(lldb::tid_t thread_id);
@@ -184,11 +179,7 @@ class WatchpointOptions {
unsigned indentation) const override;
};
-protected:
- // Classes that inherit from WatchpointOptions can see and modify these
-
private:
- // For WatchpointOptions only
WatchpointHitCallback m_callback; // This is the callback function pointer
lldb::BatonSP m_callback_baton_sp; // This is the client data for the callback
bool m_callback_is_synchronous = false;
More information about the lldb-commits
mailing list