I had done this as a subclass, but decided not to.

Remove the `virtual` marker I no longer need.

Author: jimingham

lldb/include/lldb/Breakpoint/BreakpointLocationCollection.h

@@ -30,7 +30,7 @@ class BreakpointLocationCollection {
   /// not get destroyed. 
   BreakpointLocationCollection(bool preserving = false);
 
-  virtual ~BreakpointLocationCollection();
+  ~BreakpointLocationCollection();
 
   BreakpointLocationCollection &operator=(const BreakpointLocationCollection &rhs);
 
@@ -39,7 +39,7 @@ class BreakpointLocationCollection {
   /// \param[in] bp_loc_sp
   ///     Shared pointer to the breakpoint location that will get added
   ///     to the list.
-  virtual void Add(const lldb::BreakpointLocationSP &bp_loc_sp);
+  void Add(const lldb::BreakpointLocationSP &bp_loc_sp);
 
   /// Removes the breakpoint location given by \b breakID from this
   /// list.
@@ -52,7 +52,7 @@ class BreakpointLocationCollection {
   ///
   /// \result
   ///     \b true if the breakpoint was in the list.
-  virtual bool Remove(lldb::break_id_t break_id, lldb::break_id_t break_loc_id);
+  bool Remove(lldb::break_id_t break_id, lldb::break_id_t break_loc_id);
 
   /// Returns a shared pointer to the breakpoint location with id \a
   /// breakID.