diff --git a/src/hb-subset-plan.cc b/src/hb-subset-plan.cc
index 1e48618fd..fc1e4db51 100644
--- a/src/hb-subset-plan.cc
+++ b/src/hb-subset-plan.cc
@@ -607,7 +607,8 @@ hb_subset_plan_destroy (hb_subset_plan_t *plan)
  * hb_subset_plan_old_to_new_glyph_mapping:
  * @plan: a subsetting plan.
  *
- * Returns the mapping between glyphs in the original font to glyphs
+ * Return value:
+ * the mapping between glyphs in the original font to glyphs
  * in the subset that will be produced by @plan.
  *
  * Since: REPLACEME
diff --git a/src/hb-subset.cc b/src/hb-subset.cc
index 3d3b23062..9faac05c4 100644
--- a/src/hb-subset.cc
+++ b/src/hb-subset.cc
@@ -355,11 +355,14 @@ hb_subset_or_fail (hb_face_t *source, const hb_subset_input_t *input)
 
 
 /**
- * hb_subset_from_plan_or_fail:
+ * hb_subset_plan_execute_or_fail:
  * @plan: a subsetting plan.
  *
- * Subsets a font according to provided plan. Returns nullptr
- * if the subset operation fails.
+ * Executes the provided subsetting @plan.
+ *
+ * Return value:
+ * on success returns a reference to generated font subset. If the subsetting operation fails
+ * returns nullptr.
  *
  * Since: REPLACEME
  **/