still leaking memory

@@ -196,3 +232,3 @@
  * \param keying_mask is expected to have the same size as `rna_path`. A false bit means that index
  * will be skipped.
- * \returns The number of keys inserted.
+ * \returns How often keyframe insertion was successful how often it failed for which reason.

@@ -85,0 +75,4 @@
+{
+  /* For loop starts at 1 to skip the SUCCESS flag. Assumes that SUCCESS is 0 and the rest of the
+   * enum are sequential values. */
+  for (int i = 1; i < result_counter.size(); i++) {

@@ -89,0 +83,4 @@
+  return false;
+}
+
+void CombinedKeyingResult::generate_reports(ReportList *reports)

@@ -369,0 +377,4 @@
+                                                  anim_eval_context));
+  }
+
+  if (combined_result.get_count(animrig::SingleKeyingResult::SUCCESS) == 0) {

@@ -89,0 +135,4 @@
+  }
+
+  if (has_reported_error) {
+    BKE_reportf(reports, RPT_ERROR, "%s", error.c_str());

@@ -339,0 +341,4 @@
+  }
+
+  if (BLI_listbase_is_empty(&selection)) {
+    BKE_reportf(op->reports, RPT_WARNING, "Nothing selected to keyframe");