Cleanup: Alembic: use pointers instead of references when passing progress variables
authorSybren A. Stüvel <sybren@blender.org>
Tue, 17 Sep 2019 10:15:49 +0000 (12:15 +0200)
committerSybren A. Stüvel <sybren@blender.org>
Tue, 17 Sep 2019 12:09:42 +0000 (14:09 +0200)
Using pointers instead of references when passing progress variables
makes the C++ code more in line with the C code (as it doesn't transform
pointer parameters to reference parameters). Also makes it easier to
spot when a common Blender pattern is implemented incorrectly (fix will
be in the next commit).

source/blender/alembic/intern/abc_exporter.cc
source/blender/alembic/intern/abc_exporter.h
source/blender/alembic/intern/alembic_capi.cc

index a7549cc1f470701af1b6a0a358f7aa31a68b884c..94263f615181c7d06a9ba9edc3d24f815410e993 100644 (file)
@@ -263,7 +263,7 @@ void AbcExporter::getFrameSet(unsigned int nr_of_samples, std::set<double> &fram
   }
 }
 
-void AbcExporter::operator()(float &progress, bool &was_canceled)
+void AbcExporter::operator()(float *progress, bool *was_canceled)
 {
   std::string scene_name;
 
@@ -332,10 +332,10 @@ void AbcExporter::operator()(float &progress, bool &was_canceled)
   size_t i = 0;
 
   for (; begin != end; ++begin) {
-    progress = (++i / size);
+    *progress = (++i / size);
 
     if (G.is_break) {
-      was_canceled = true;
+      *was_canceled = true;
       break;
     }
 
index e6a7a3fc7f4c14726cad438aec79fbd2fbb08758..cc8d9a757654ce41a1795db8ff22898bc31b397e 100644 (file)
@@ -104,7 +104,7 @@ class AbcExporter {
   AbcExporter(Main *bmain, const char *filename, ExportSettings &settings);
   ~AbcExporter();
 
-  void operator()(float &progress, bool &was_canceled);
+  void operator()(float *progress, bool *was_canceled);
 
  protected:
   void getShutterSamples(unsigned int nr_of_samples,
index 9ace0a8faa9be3f6382f0648fce6cb652bef878a..f9dc688c15947ac68e15c5282f4ef18ae8056730 100644 (file)
@@ -261,7 +261,7 @@ static void export_startjob(void *customdata, short *stop, short *do_update, flo
     const int orig_frame = CFRA;
 
     data->was_canceled = false;
-    exporter(*data->progress, data->was_canceled);
+    exporter(progress, &data->was_canceled);
 
     if (CFRA != orig_frame) {
       CFRA = orig_frame;