@ferdnyc commented on this pull request.


> -All whitespace surrounding \<body\> is removed.  Name may be composed
-of alphanumeric characters, and the character `_' and must be at least
-3 characters in length. A macro without an (opts) field is "simple" in that
-only recursive macro expansion is performed. A parameterized macro contains
-an (opts) field. "-" as opts disables all option processing, otherwise
-the opts (i.e. string between parentheses) are passed
+All whitespace surrounding `<body>` is removed.  Name may be composed
+of alphanumeric characters and the character `_`, and must be at least
+3 characters in length. A macro without an `(opts)` field is "simple"
+in that only recursive macro expansion is performed. A parameterized
+macro contains an `(opts)` field. `-` as opts disables all option
+processing, otherwise the opts (i.e. string between parentheses) are passed

I snuck in a grammar change here that really should've been a separate commit, 
moving the comma in this sentence:

>  Name may be composed of alphanumeric characters, and the character `_` and 
> must be at least 3 characters in length.

to separate the two clauses:

>  Name may be composed of alphanumeric characters and the character `_`, and 
> must be at least 3 characters in length.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2251#pullrequestreview-1162690898
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2251/review/1162690...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to