On Wed, 1 Dec 2021 at 22:40, Frank H. Ellenberger
<frank.h.ellenber...@gmail.com> wrote:
>
> Hi Kevin,
>
> don't forget to CC the list.
>
> Am 01.12.21 um 03:22 schrieb Kevin Buckley:
> > On Sat, 27 Nov 2021 at 23:36, Frank H. Ellenberger
> > <frank.h.ellenber...@gmail.com> wrote:
> >>
> >> Hi Kevin,
> >>
> >> The comments containing "@param" are doxygen sources. How looks the reesult
> >> https://code.gnucash.org/docs/MAINT/import-main-matcher_8h.html and links?
> >>
> >> Regards
> >> Frank
> >
> > Yes the "a the" is visible there too.
> >
> > If I navigate through to, say,
> >
> > https://code.gnucash.org/docs/MAINT/group__Import__Export.html#gad26d36e9e962bb4ba174d697f59f25fb
> >
> > which is the doc for this fucntion
> >
> > gnc_gen_trans_list_empty()
> >
> > I see
> >
> > gboolean gnc_gen_trans_list_empty ( GNCImportMainMatcher *  info)
> >
> > Checks whether there are no transactions to match.
> >
> > Parameters
> > info A pointer to a the GNCImportMainMatcher structure.
> >
> > so the "a the" is in there too.
> >
> > FWIW, I have noticed a few other typos and spelling corrections in
> > the comments, so maybe I should clone the repo and submit a PR
> > with them all in?
>
> Yes, that would be the best approach.
> See also https://wiki.gnucash.org/wiki/Doxygen
>
> Regards
> Frank

Find a patch from a `git diff -p -stat` attached.
 gnucash/gnome-utils/gnc-main-window.h                |  6 +++---
 gnucash/gnome-utils/gnc-tree-view-commodity.h        |  2 +-
 gnucash/import-export/import-main-matcher.h          | 16 ++++++++--------
 gnucash/register/register-gnome/gnucash-sheet.c      |  2 +-
 libgnucash/app-utils/calculation/expression_parser.c |  2 +-
 libgnucash/app-utils/gnc-sx-instance-model.c         |  2 +-
 libgnucash/engine/qofid.h                            |  2 +-
 7 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/gnucash/gnome-utils/gnc-main-window.h b/gnucash/gnome-utils/gnc-main-window.h
index 35b70d588..305494e73 100644
--- a/gnucash/gnome-utils/gnc-main-window.h
+++ b/gnucash/gnome-utils/gnc-main-window.h
@@ -361,7 +361,7 @@ void gnc_main_window_restore_all_windows(const GKeyFile *keyfile);
  *  used on report pages to delay the creation of the report till the
  *  page is focused.
  *
- *  @param window When window whose pages should be checked.
+ *  @param window The window whose pages should be checked.
  *
  *  @return TRUE if pages are being restored
  */
@@ -383,7 +383,7 @@ void gnc_main_window_restore_default_state(GncMainWindow *window);
  *  If any page returns a failure indication, then the function stops
  *  walking pages and immediately returns a failure.
  *
- *  @param window When window whose pages should be checked.
+ *  @param window The window whose pages should be checked.
  *
  *  @return FALSE if any page could not or would not comply, which
  *  should cancel the pending operation.  TRUE otherwise */
@@ -412,7 +412,7 @@ void gnc_main_window_all_action_set_sensitive (const gchar *action_name, gboolea
 
 /** Find action in main window.
  *
- *  @param window When window which should be checked for the action.
+ *  @param window The window which should be checked for the action.
  *
  *  @param name The name of the command to be retrieved.
  *
diff --git a/gnucash/gnome-utils/gnc-tree-view-commodity.h b/gnucash/gnome-utils/gnc-tree-view-commodity.h
index eeacdeb7b..1f7e80bb5 100644
--- a/gnucash/gnome-utils/gnc-tree-view-commodity.h
+++ b/gnucash/gnome-utils/gnc-tree-view-commodity.h
@@ -208,7 +208,7 @@ gnc_commodity * gnc_tree_view_commodity_get_selected_commodity  (GncTreeViewComm
  *
  *  @param view A pointer to an commodity tree view.
  *
- *  @param The commodity to select.
+ *  @param commodity The commodity to select.
  */
 void gnc_tree_view_commodity_select_commodity (GncTreeViewCommodity *view, gnc_commodity *commodity);
 
diff --git a/gnucash/import-export/import-main-matcher.h b/gnucash/import-export/import-main-matcher.h
index 490d54c44..51ddf4f53 100644
--- a/gnucash/import-export/import-main-matcher.h
+++ b/gnucash/import-export/import-main-matcher.h
@@ -112,7 +112,7 @@ GNCImportMainMatcher * gnc_gen_trans_assist_new (GtkWidget *parent,
 /**  This starts the import process for transaction from an assistant.
  *   assistant button callback.
  *
- * @param info. A pointer to a the GNCImportMainMatcher structure.
+ * @param info. A pointer to the GNCImportMainMatcher structure
 */
 void gnc_gen_trans_assist_start (GNCImportMainMatcher *info);
 
@@ -177,38 +177,38 @@ void gnc_gen_trans_list_add_trans_with_ref_id (GNCImportMainMatcher *gui,
 /** Run this dialog and return only after the user pressed Ok, Cancel,
   or closed the window. This means that all actual importing will
   have been finished upon returning.
- * @param info A pointer to a the GNCImportMainMatcher structure.
+ * @param info A pointer to the GNCImportMainMatcher structure.
  * @return The boolean return value of the dialog run.
 */
 gboolean gnc_gen_trans_list_run (GNCImportMainMatcher *info);
 
 
 /** Returns the widget of this dialog.
- * @param info A pointer to a the GNCImportMainMatcher structure.
+ * @param info A pointer to the GNCImportMainMatcher structure.
  * @return A GtkWidget pointer to the dialog's widget.
  */
 GtkWidget *gnc_gen_trans_list_widget (GNCImportMainMatcher *info);
 
 /** Returns the append_text widget of this dialog.
- * @param info A pointer to a the GNCImportMainMatcher structure.
+ * @param info A pointer to the GNCImportMainMatcher structure.
  * @return A GtkWidget pointer to the append_text widget.
  */
 GtkWidget *
 gnc_gen_trans_list_append_text_widget (GNCImportMainMatcher *info);
 
 /** Checks whether there are no transactions to match.
- * @param info A pointer to a the GNCImportMainMatcher structure.
+ * @param info A pointer to the GNCImportMainMatcher structure.
  * @return A boolean indicating whether the transaction list is empty.
  */
 gboolean gnc_gen_trans_list_empty (GNCImportMainMatcher *info);
 
 /** Shows widgets.
- * @param info A pointer to a the GNCImportMainMatcher structure.
+ * @param info A pointer to the GNCImportMainMatcher structure.
  */
 void gnc_gen_trans_list_show_all (GNCImportMainMatcher *info);
 
 /** Show and set the reconcile after close check button.
- * @param info A pointer to a the GNCImportMainMatcher structure.
+ * @param info A pointer to the GNCImportMainMatcher structure.
  * @param reconcile_after_close A gboolean that shows or hides the button.
  * @param active A gboolean to set or clear the check button.
  */
@@ -216,7 +216,7 @@ void gnc_gen_trans_list_show_reconcile_after_close_button (GNCImportMainMatcher
                                                            gboolean reconcile_after_close,
                                                            gboolean active);
 /** Returns the reconcile after close check button.
- * @param info A pointer to a the GNCImportMainMatcher structure.
+ * @param info A pointer to the GNCImportMainMatcher structure.
  * @return The check button.
  */
 GtkWidget* gnc_gen_trans_list_get_reconcile_after_close_button (GNCImportMainMatcher *info);
diff --git a/gnucash/register/register-gnome/gnucash-sheet.c b/gnucash/register/register-gnome/gnucash-sheet.c
index 54d32ba61..03581106b 100644
--- a/gnucash/register/register-gnome/gnucash-sheet.c
+++ b/gnucash/register/register-gnome/gnucash-sheet.c
@@ -1530,7 +1530,7 @@ gnucash_sheet_button_press_event (GtkWidget *widget, GdkEventButton *event)
 
     gnucash_sheet_cursor_move (sheet, new_virt_loc);
 
-    // if clicked in ocument link cell, run call back
+    // if clicked in document link cell, run call back
     if (g_strcmp0 (gnc_table_get_cell_name (table, new_virt_loc), DOCLINK_CELL) == 0)
     {
         if (sheet->open_doclink_cb)
diff --git a/libgnucash/app-utils/calculation/expression_parser.c b/libgnucash/app-utils/calculation/expression_parser.c
index 9159deaaa..3115217c7 100644
--- a/libgnucash/app-utils/calculation/expression_parser.c
+++ b/libgnucash/app-utils/calculation/expression_parser.c
@@ -297,7 +297,7 @@
  *         to the parser. The calling function(s) should never rely on
  *         manipulating any information inside this structure
  *         directly, since it may and could change in the future.  --
- *         The first parameter is a pointer to a the first element in
+ *         The first parameter is a pointer to the first element in
  *         a linked list of "pre-defined" variables the caller wishes
  *         to use with subsequent calls to the parser.  -- The second
  *         parameter is the radix character to use in numeric strings
diff --git a/libgnucash/app-utils/gnc-sx-instance-model.c b/libgnucash/app-utils/gnc-sx-instance-model.c
index 717a371f8..924c20b60 100644
--- a/libgnucash/app-utils/gnc-sx-instance-model.c
+++ b/libgnucash/app-utils/gnc-sx-instance-model.c
@@ -1049,7 +1049,7 @@ _get_sx_formula_value(const SchedXaction* sx,
 	!gnc_numeric_zero_p(*numeric_val))
     {
         /* If there are no variables to parse and we had a valid numeric stored
-         * then we can skip parsing the formual, which might save some
+         * then we can skip parsing the formula, which might save some
          * localization problems with separators. */
 	numeric->num = numeric_val->num;
 	numeric->denom = numeric_val->denom;
diff --git a/libgnucash/engine/qofid.h b/libgnucash/engine/qofid.h
index 1e84d1395..6c1caac85 100644
--- a/libgnucash/engine/qofid.h
+++ b/libgnucash/engine/qofid.h
@@ -43,7 +43,7 @@ extern "C"
     type, each uniquely identified by a given ID.  QOF Entities
     can be used independently of any other part of the system.
     In particular, Entities can be useful even if one is not using
-    the Query one Object parts of the QOF system.
+    the Query and Object parts of the QOF system.
 
     Identifiers are globally-unique and permanent, i.e., once
     an entity has been assigned an identifier, it retains that same
_______________________________________________
gnucash-devel mailing list
gnucash-devel@gnucash.org
https://lists.gnucash.org/mailman/listinfo/gnucash-devel

Reply via email to