Formatting.

Author: jimingham

lldb/include/lldb/API/SBBreakpoint.h

@@ -157,12 +157,11 @@ class LLDB_API SBBreakpoint {
   /// Can only be called from a ScriptedBreakpointResolver...
   SBError
   AddLocation(SBAddress &address);
-  /// Add a "Facade location" to the breakpoint.  This returns the Facade 
-  /// Location that was added, which you can then use in 
+  /// Add a "Facade location" to the breakpoint.  This returns the Facade
+  /// Location that was added, which you can then use in
   /// get_location_description and was_hit in your breakpoint resolver.
   /// Can only be called from a ScriptedBreakpointResolver.
-  SBBreakpointLocation
-  AddFacadeLocation();
+  SBBreakpointLocation AddFacadeLocation();
 
   SBStructuredData SerializeToStructuredData();
 

lldb/include/lldb/API/SBBreakpointLocation.h

@@ -25,6 +25,7 @@ namespace lldb {
 
 class LLDB_API SBBreakpointLocation {
   friend class lldb_private::ScriptInterpreter;
+
 public:
   SBBreakpointLocation();
 

lldb/include/lldb/API/SBFrame.h

@@ -225,7 +225,7 @@ class LLDB_API SBFrame {
   friend class SBInstruction;
   friend class SBThread;
   friend class SBValue;
-  
+
   friend class lldb_private::ScriptInterpreter;
   friend class lldb_private::python::SWIGBridge;
   friend class lldb_private::lua::SWIGBridge;

lldb/include/lldb/Breakpoint/Breakpoint.h

@@ -248,8 +248,8 @@ class Breakpoint : public std::enable_shared_from_this<Breakpoint>,
   ///    Returns a pointer to the new location.
   lldb::BreakpointLocationSP AddLocation(const Address &addr,
                                          bool *new_location = nullptr);
-  /// Add a `facade` location to the breakpoint's collection of facade locations.  
-  /// This is only meant to be called by the breakpoint's resolver.
+  /// Add a `facade` location to the breakpoint's collection of facade
+  /// locations. This is only meant to be called by the breakpoint's resolver.
   /// Facade locations are placeholders that a scripted breakpoint can use to
   /// represent the stop locations provided by the breakpoint.  The scripted
   /// breakpoint should record the id of the facade location, and provide
@@ -263,7 +263,7 @@ class Breakpoint : public std::enable_shared_from_this<Breakpoint>,
   /// \return
   ///    Returns a pointer to the new location.
   lldb::BreakpointLocationSP AddFacadeLocation();
-  
+
   lldb::BreakpointLocationSP GetFacadeLocationByID(lldb::break_id_t);
 
   /// Find a breakpoint location by Address.
@@ -293,28 +293,30 @@ class Breakpoint : public std::enable_shared_from_this<Breakpoint>,
   ///    The ID specifying the location.
   ///
   /// \param[in] use_facade
-  /// If \b true, then prefer facade locations over "real" ones if they exist.  
+  /// If \b true, then prefer facade locations over "real" ones if they exist.
   ///
   /// \return
   ///    Returns a shared pointer to the location with ID \a bp_loc_id.  The
   ///    pointer
   ///    in the shared pointer will be nullptr if there is no location with that
   ///    ID.
-  lldb::BreakpointLocationSP FindLocationByID(lldb::break_id_t bp_loc_id, bool use_facade = true);
+  lldb::BreakpointLocationSP FindLocationByID(lldb::break_id_t bp_loc_id,
+                                              bool use_facade = true);
 
   /// Get breakpoint locations by index.
   ///
   /// \param[in] index
   ///    The location index.
   ///
   /// \param[in] use_facade
-  /// If \b true, then prefer facade locations over "real" ones if they exist.  
+  /// If \b true, then prefer facade locations over "real" ones if they exist.
   ///
   /// \return
   ///     Returns a shared pointer to the location with index \a
   ///     index. The shared pointer might contain nullptr if \a index is
   ///     greater than then number of actual locations.
-  lldb::BreakpointLocationSP GetLocationAtIndex(size_t index, bool use_facade = true);
+  lldb::BreakpointLocationSP GetLocationAtIndex(size_t index,
+                                                bool use_facade = true);
 
   /// Removes all invalid breakpoint locations.
   ///
@@ -436,7 +438,7 @@ class Breakpoint : public std::enable_shared_from_this<Breakpoint>,
   /// breakpoint sites.
   ///
   /// \param[in] use_facade
-  /// If \b true, then prefer facade locations over "real" ones if they exist.  
+  /// If \b true, then prefer facade locations over "real" ones if they exist.
   ///
   /// \return
   ///     The number locations resolved breakpoint sites.
@@ -451,7 +453,7 @@ class Breakpoint : public std::enable_shared_from_this<Breakpoint>,
   /// Return the number of breakpoint locations.
   ///
   /// \param[in] use_facade
-  /// If \b true, then prefer facade locations over "real" ones if they exist.  
+  /// If \b true, then prefer facade locations over "real" ones if they exist.
   ///
   /// \return
   ///     The number breakpoint locations.
@@ -563,17 +565,14 @@ class Breakpoint : public std::enable_shared_from_this<Breakpoint>,
 
   enum TypeDisplay {
     eDisplayFacade = 1,
-    eDisplayReal   = 1 << 1,
+    eDisplayReal = 1 << 1,
     eDisplayHeader = 1 << 2
   };
 
   void GetDescriptionForType(Stream *s, lldb::DescriptionLevel level,
-                      uint8_t display_type, bool show_locations);
-                      
-  bool HasFacadeLocations() {
-    return m_facade_locations.GetSize() != 0;
-  }
+                             uint8_t display_type, bool show_locations);
 
+  bool HasFacadeLocations() { return m_facade_locations.GetSize() != 0; }
 
 public:
   bool MatchesName(const char *name) {

lldb/include/lldb/Breakpoint/BreakpointLocation.h

@@ -62,17 +62,17 @@ class BreakpointLocation
   Target &GetTarget();
 
   /// This is a programmatic version of a breakpoint "condition".  When a
-  /// breakpoint is hit, WasHit will get called before the synchronous ShouldStop
-  /// callback is run, and if it returns an empty BreakpointLocationSP, lldb will
-  /// act as if that breakpoint wasn't hit.
+  /// breakpoint is hit, WasHit will get called before the synchronous
+  /// ShouldStop callback is run, and if it returns an empty
+  /// BreakpointLocationSP, lldb will act as if that breakpoint wasn't hit.
   ///
   /// \param[in] context
   ///   The context at the stop point
-  ///    
+  ///
   /// \return
   ///    This will return the breakpoint location that was hit on this stop.
   ///    If there was no facade location this will be the original location.
-  ///    If the shared pointer is empty, then we'll treat it as if the 
+  ///    If the shared pointer is empty, then we'll treat it as if the
   ///    breakpoint was not hit.
   lldb::BreakpointLocationSP WasHit(StoppointCallbackContext *context);
 
@@ -84,16 +84,16 @@ class BreakpointLocation
   ///
   /// \param[in] context
   ///   The context at the stop point
-  ///    
+  ///
   /// \param[out] facade_loc_sp
   ///   If this stop should be attributed not to the location that was hit, but
   ///   to a facade location, it will be returned in this facade_loc_sp.
-  ///    
+  ///
   /// \return
   ///     \b true if this breakpoint location thinks we should stop,
   ///     \b false otherwise.
-  bool ShouldStop(StoppointCallbackContext *context, 
-          lldb::BreakpointLocationSP &facade_loc_sp);
+  bool ShouldStop(StoppointCallbackContext *context,
+                  lldb::BreakpointLocationSP &facade_loc_sp);
 
   // The next section deals with various breakpoint options.
 
@@ -373,7 +373,7 @@ class BreakpointLocation
   /// Constructor.
   ///
   /// \param[in] loc_id
-  ///     The location id of the new location. 
+  ///     The location id of the new location.
   ///
   /// \param[in] owner
   ///     A back pointer to the breakpoint that owns this location.
@@ -390,10 +390,10 @@ class BreakpointLocation
                      bool check_for_resolver = true);
 
   /// This is the constructor for locations with no address.  Currently this is
-  /// just used for Facade locations. 
+  /// just used for Facade locations.
   ///
   /// \param[in] loc_id
-  ///     The location id of the new location. 
+  ///     The location id of the new location.
   ///
   /// \param[in] owner
   ///     A back pointer to the breakpoint that owns this location.
@@ -403,6 +403,7 @@ class BreakpointLocation
   BreakpointLocation(lldb::break_id_t loc_id, Breakpoint &owner);
   bool IsValid() const { return m_is_valid; }
   bool IsFacade() const { return m_is_facade; }
+
 private:
   // Data members:
   bool m_should_resolve_indirect_functions;
@@ -431,10 +432,10 @@ class BreakpointLocation
   /// location was given somewhere in the virtual inlined call stack since the
   /// Address always resolves to the lowest entry in the stack.
   std::optional<LineEntry> m_preferred_line_entry;
-  bool m_is_valid = true;  /// Because Facade locations don't have sites
-                           /// we can't use the presence of the site to mean
-                           /// this breakpoint is valid, but must manage
-                           /// the state directly.
+  bool m_is_valid = true;   /// Because Facade locations don't have sites
+                            /// we can't use the presence of the site to mean
+                            /// this breakpoint is valid, but must manage
+                            /// the state directly.
   bool m_is_facade = false; /// Facade locations aren't directly triggered
                             /// and don't have a breakpoint site.  They are
                             /// a useful fiction when you want to represent

lldb/include/lldb/Breakpoint/BreakpointLocationCollection.h

@@ -111,7 +111,8 @@ class BreakpointLocationCollection {
   ///
   /// \return
   ///    \b true if we should stop, \b false otherwise.
-  bool ShouldStop(StoppointCallbackContext *context, BreakpointLocationCollection &stopped_bp_locs);
+  bool ShouldStop(StoppointCallbackContext *context,
+                  BreakpointLocationCollection &stopped_bp_locs);
 
   /// Print a description of the breakpoint locations in this list
   /// to the stream \a s.

lldb/include/lldb/Breakpoint/BreakpointLocationList.h

@@ -140,8 +140,8 @@ class BreakpointLocationList {
   ///
   /// \return
   ///     \b true if we should stop, \b false otherwise.
-  bool ShouldStop(StoppointCallbackContext *context, lldb::break_id_t breakID, 
-      lldb::BreakpointLocationSP &bp_loc_sp);
+  bool ShouldStop(StoppointCallbackContext *context, lldb::break_id_t breakID,
+                  lldb::BreakpointLocationSP &bp_loc_sp);
 
   /// Returns the number of elements in this breakpoint location list.
   ///

lldb/include/lldb/Breakpoint/BreakpointResolverScripted.h

@@ -45,11 +45,12 @@ class BreakpointResolverScripted : public BreakpointResolver {
 
   void GetDescription(Stream *s) override;
 
-  lldb::BreakpointLocationSP WasHit(lldb::StackFrameSP frame_sp, 
+  lldb::BreakpointLocationSP WasHit(lldb::StackFrameSP frame_sp,
                                     lldb::BreakpointLocationSP bp_loc_sp);
-  
-  std::optional<std::string> GetLocationDescription(lldb::BreakpointLocationSP bp_loc_sp,
-      lldb::DescriptionLevel level);
+
+  std::optional<std::string>
+  GetLocationDescription(lldb::BreakpointLocationSP bp_loc_sp,
+                         lldb::DescriptionLevel level);
 
   void Dump(Stream *s) const override;
 

lldb/include/lldb/Breakpoint/BreakpointSite.h

@@ -99,8 +99,8 @@ class BreakpointSite : public std::enable_shared_from_this<BreakpointSite>,
   ///
   /// \return
   ///    \b true if we should stop, \b false otherwise.
-  bool ShouldStop(StoppointCallbackContext *context, 
-          BreakpointLocationCollection &stopping_bp_loc) override;
+  bool ShouldStop(StoppointCallbackContext *context,
+                  BreakpointLocationCollection &stopping_bp_loc) override;
 
   /// Standard Dump method
   void Dump(Stream *s) const override;

lldb/include/lldb/Breakpoint/StoppointSite.h

@@ -38,10 +38,12 @@ class StoppointSite {
 
   virtual bool IsHardware() const = 0;
 
-  virtual bool ShouldStop(StoppointCallbackContext* context) { return false; };
+  virtual bool ShouldStop(StoppointCallbackContext *context) { return false; };
 
-  virtual bool ShouldStop(StoppointCallbackContext* context,
-          BreakpointLocationCollection &stopping_bp_locs) { return false; };
+  virtual bool ShouldStop(StoppointCallbackContext *context,
+                          BreakpointLocationCollection &stopping_bp_locs) {
+    return false;
+  };
 
   virtual void Dump(Stream* stream) const = 0;