Move cancel_all() out of SwervePathBuilder FFI class (#225)

include/trajopt/path/SwervePathBuilder.hpp

@@ -181,11 +181,6 @@ class TRAJOPT_DLLEXPORT SwervePathBuilder {
   void AddIntermediateCallback(
       const std::function<void(SwerveSolution&, int64_t)> callback);
 
-  /**
-   * Cancel all currently generating SwervePathBuilders.
-   */
-  void CancelAll();
-
  private:
   SwervePath path;
 

src/RustFFI.cpp

@@ -17,6 +17,7 @@
 #include "trajopt/drivetrain/SwerveModule.hpp"
 #include "trajopt/trajectory/HolonomicTrajectory.hpp"
 #include "trajopt/trajectory/HolonomicTrajectorySample.hpp"
+#include "trajopt/util/Cancellation.hpp"
 #include "trajoptlib/src/lib.rs.h"
 
 namespace trajopt::rsffi {
@@ -240,12 +241,12 @@ void SwervePathBuilder::add_progress_callback(
       });
 }
 
-void SwervePathBuilder::cancel_all() {
-  path_builder.CancelAll();
-}
-
 std::unique_ptr<SwervePathBuilder> swerve_path_builder_new() {
   return std::make_unique<SwervePathBuilder>();
 }
 
+void cancel_all() {
+  trajopt::GetCancellationFlag() = 1;
+}
+
 }  // namespace trajopt::rsffi

src/RustFFI.hpp

@@ -65,12 +65,12 @@ class SwervePathBuilder {
   void add_progress_callback(
       rust::Fn<void(HolonomicTrajectory, int64_t)> callback);
 
-  void cancel_all();
-
  private:
   trajopt::SwervePathBuilder path_builder;
 };
 
 std::unique_ptr<SwervePathBuilder> swerve_path_builder_new();
 
+void cancel_all();
+
 }  // namespace trajopt::rsffi

src/lib.rs

@@ -162,9 +162,9 @@ mod ffi {
             callback: fn(HolonomicTrajectory, i64),
         );
 
-        fn cancel_all(self: Pin<&mut SwervePathBuilder>);
-
         fn swerve_path_builder_new() -> UniquePtr<SwervePathBuilder>;
+
+        fn cancel_all();
     }
 }
 
@@ -424,10 +424,6 @@ impl SwervePathBuilder {
     pub fn add_progress_callback(&mut self, callback: fn(HolonomicTrajectory, i64)) {
         crate::ffi::SwervePathBuilder::add_progress_callback(self.path_builder.pin_mut(), callback);
     }
-
-    pub fn cancel_all(&mut self) {
-        crate::ffi::SwervePathBuilder::cancel_all(self.path_builder.pin_mut());
-    }
 }
 
 impl Default for SwervePathBuilder {
@@ -436,6 +432,10 @@ impl Default for SwervePathBuilder {
     }
 }
 
+pub fn cancel_all() {
+    crate::ffi::cancel_all();
+}
+
 pub use ffi::HolonomicTrajectory;
 pub use ffi::HolonomicTrajectorySample;
 pub use ffi::Pose2d;

src/path/SwervePathBuilder.cpp

@@ -198,10 +198,6 @@ void SwervePathBuilder::AddIntermediateCallback(
   path.callbacks.push_back(callback);
 }
 
-void SwervePathBuilder::CancelAll() {
-  trajopt::GetCancellationFlag() = 1;
-}
-
 void SwervePathBuilder::NewWpts(size_t finalIndex) {
   int64_t targetIndex = finalIndex;
   int64_t greatestIndex = path.waypoints.size() - 1;