mtz: copy/replace_column changed to copy also dataset ID

project-gemmi · Apr 18, 2023 · 787b378 · 787b378
1 parent 77b81fb
commit 787b378
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/include/gemmi/mtz.hpp b/include/gemmi/mtz.hpp
@@ -938,8 +938,7 @@ struct GEMMI_DLL Mtz {
       dst.min_value = src.min_value;
       dst.max_value = src.max_value;
       dst.source = src.source;
-      if (src_mtz == this)
-        dst.dataset_id = src.dataset_id;
+      dst.dataset_id = src.dataset_id;
     }
     if (src_mtz == this) {
       // internal copying
@@ -1006,6 +1005,7 @@ struct GEMMI_DLL Mtz {
     expand_data_rows(1 + trailing_cols.size(), dest_idx);
     // copy the data
     const Column& src_col_now = col_idx < 0 ? src_col : columns[col_idx];
+    // most of the work (hkl-based row matching and data copying) is done here:
     do_replace_column(dest_idx, src_col_now, trailing_cols);
     return columns[dest_idx];
   }