[groff] 04/04: tmac/s.tmac: Fix .R macro to handle arguments.

[G. Branden Robinson]

gbranden pushed a commit to branch master
in repository groff.

commit 41843c792b1bfac5f9b02741526ee7c7a3346e44
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
AuthorDate: Mon Oct 26 19:46:11 2020 +1100

    tmac/s.tmac: Fix .R macro to handle arguments.
    
    Fix ms .R macro to work as documented, by handling its arguments instead
    of ignoring them.
    
    * tmac/s.tmac (R): Delete.  During set up, call par*define-font-macro
      for R font just as we do for B, I, and BI.
    
    Problem dates back at least to groff 1.02, June 1991.
    
    * tmac/tests/s_R-handles-its-arguments.sh: Test it.
    * tmac/tmac.am (tmac_TEST): Run test.
    
    Fixes <https://savannah.gnu.org/bugs/?59348>.
---
 ChangeLog   | 19 +++++++++++++++++--
 tmac/s.tmac |  4 +---
 2 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index e714d73..6e75336 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2020-10-26  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       Fix ms .R macro to work as documented, by handling its arguments
+       instead of ignoring them.
+
+       * tmac/s.tmac (R): Delete.  During set up, call
+       par*define-font-macro for R font just as we do for B, I, and BI.
+
+       Problem dates back at least to groff 1.02, June 1991.
+
+       * tmac/tests/s_R-handles-its-arguments.sh: Test it.
+       * tmac/tmac.am (tmac_TEST): Run test.
+
+       Fixes <https://savannah.gnu.org/bugs/?59348>.
+
 2020-10-25  G. Branden Robinson <g.branden.robinson@gmail.com>
 
        Fix ms .TC macro by allowing it to actually use lowercase Roman
@@ -21,8 +36,8 @@
        (pg*end-page): Assign pg*next-format to %, not PN.
        (XA): Define toc*num with the interpolation of %, not PN.
 
-       * tmac/tests/s-TC-works-with-percent-in-custom-titles.sh: Test.
-       * tmac/tmac.an (tmac_TEST): Run test.
+       * tmac/tests/s_TC-works-with-percent-in-custom-titles.sh: Test.
+       * tmac/tmac.am (tmac_TEST): Run test.
 
        Problem appears to be very old; as I read it, Larry Kollar was
        complaining of it in his ms.ms document over 20 years ago.
diff --git a/tmac/s.tmac b/tmac/s.tmac
index 0be7bb3..782a3c6 100644
--- a/tmac/s.tmac
+++ b/tmac/s.tmac
@@ -1466,9 +1466,6 @@ along with this program.  If not, see 
<http://www.gnu.org/licenses/>.
 .de LG
 .ps +2
 ..
-.de R
-.ft R
-..
 .\" par*define-font-macro macro font
 .de par*define-font-macro
 .de \\$1
@@ -1479,6 +1476,7 @@ along with this program.  If not, see 
<http://www.gnu.org/licenses/>.
 .el .ft \\$2
 \\..
 ..
+.par*define-font-macro R R
 .par*define-font-macro B B
 .par*define-font-macro I I
 .par*define-font-macro BI BI