bugGNU roff - Bugs: bug #51062, fix doubled words in text

 
 

bug #51062: fix doubled words in text

Submitter:  Bjarni Ingi Gislason <bjarniig>
Submitted:  Thu 18 May 2017 04:19:22 PM UTC
   
 
Category:  General Severity:  2 - Minor
Item Group:  Lint Status:  Rejected
Privacy:  Public Assigned to:  schwarze
Open/Closed:  Closed Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 30 Mar 2023 04:49:13 PM UTC, comment #3: 

Dropping patch annotation for rejected ticket.

G. Branden Robinson <gbranden>
Group administrator
Sat 22 Jan 2022 11:41:36 AM UTC, comment #2: 

comment #1:

> changing historical information in the ChangeLog and the NEWS file would be wrong.


I agree that misrepresenting history would be wrong, but correcting solecisms like mistyped bug numbers is not a misrepresentation of history--it's a more accurate representation of history.

Because my own brain hates me, it is only after committing and pushing that I am likely to notice egregiously bad typos or missing words in my ChangeLog entries.

Material misrepresentations of fact in such files should either be corrected in-place or supplemented with "[recte: ]" or similar annotation.

There is the question of how much time has to pass before we consider that such documents (or what portions of them, in the case of "NEWS") have stabilized such that the bracketed-annotation approach is more appropriate than the former.

Anyway, like you, I don't exactly welcome further use of the Savannah system for this sort of thing.  It would be better for such "linty" patches to go to the groff@ mailing list, and better still for someone to contribute a style checker that we can run automatically as part of builds to catch these things automatically so that a developer can fix them before they're even introduced into a pushed commit.

What ever became of the Writer's Workbench, eh?

As I recall, a doubled-word finder for plain text that works across line boundaries is an exercise in the O'Reilly sed & awk book.  It would be a nice thing for a new developer to contribute!  :)

Anyway, here's the state of play for the doubled words Bjarni reported.  I intend to fix them, but I'm not reopening this ticket because this sort of issue simply doesn't rise to that level, IMO.


$ git grep -w "be be"
ChangeLog.119:  with a missing field should be be sorted before all entries that
contrib/mom/examples/copyright-chapter.mom:chapter, should be be treated as a DOCTYPE DEFAULT.  The text of
contrib/mom/examples/copyright-default.mom:A copyright page (also called an edition page) should be be treated

$ git grep -w "the the"
ChangeLog:      BR).  This way the the font escape style names in the appendment
doc/groff.texi:followed by integers arguments to number the the heading depths
doc/groff.texi:usually invoked by a localization file, which is turn loaded by the the
doc/ms.ms:effective as of the the next listed element encountered.
src/roff/groff/groff.1.man:creates font description files for the the HP LaserJet\~4 family of
tmac/groff_ms.7.man:effective as of the the next listed element encountered.

$ git grep -w "for for"
NEWS:the $r or $R register is set to a value that would be correct for for


G. Branden Robinson <gbranden>
Group administrator
Fri 16 Feb 2018 09:28:01 PM UTC, comment #1: 

In the meantime, Makefile.am and TESTS were fixed, and changing historical information in the ChangeLog and the NEWS file would be wrong.

Ingo Schwarze <schwarze>
Group Member
Thu 18 May 2017 04:19:22 PM UTC, original submission:  

From 0925353091fc5909aebc4a1dae9a10c241f69d63 Mon Sep 17 00:00:00 2001
From: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
Date: Thu, 18 May 2017 16:12:13 +0000
Subject: [PATCH] misc: Remove repeated words in files in "top_srcdir"

Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
---
 ChangeLog.119 | 2 +-
 Makefile.am   | 2 +-
 NEWS          | 2 +-
 TESTS         | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/ChangeLog.119 b/ChangeLog.119
index 9245154e..d2b9cccd 100644
--- a/ChangeLog.119
+++ b/ChangeLog.119
@@ -3077,7 +3077,7 @@ Version 1.19.1 released
  * src/preproc/refer/ref.cpp (reference::compute_sort_key): Always
  insert SORT_SEP.  With certain combinations of sort specifications,
  refer sorted entries in the wrong order.  In particular, entries
- with a missing field should be be sorted before all entries that
+ with a missing field should be sorted before all entries that
  have that field, before refer looks to the next field.
 
 2003-08-23  Werner LEMBERG  <wl@gnu.org>
diff --git a/Makefile.am b/Makefile.am
index 2697115c..c65b1107 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -457,7 +457,7 @@ man7dir=$(manroot)/man$(man7ext)
 # (Reiser) preprocessor
 # ARRAY_DELETE_NEEDS_SIZE if your C++ doesn't understand `delete []'
 #
-# PAGE=A4 if the the printer's page size is A4
+# PAGE=A4 if the printer's page size is A4
 # GHOSTSCRIPT=gs the name (and directory if required) of the
 # ghostscript program
 
diff --git a/NEWS b/NEWS
index 0085f47d..28682dbb 100644
--- a/NEWS
+++ b/NEWS
@@ -2326,7 +2326,7 @@ However, this caused problems with existing -me documents.  In this
 release, the vertical spacing is controlled by the $v and $V registers
 which have the same meaning as $r and $R in earlier groff releases.
 In addition, if the $r or $R register is set to a value that would be
-correct for for the BSD -me macros and a low resolution device is not
+correct for the BSD -me macros and a low resolution device is not
 being used, then an appropriate value for the $v or $V register
 is derived from the $r or $R register.
 
diff --git a/TESTS b/TESTS
index a7104f36..d9dd644e 100644
--- a/TESTS
+++ b/TESTS
@@ -42,7 +42,7 @@ Tests
   mkdir -p ~/tmp/automake
   make install DESTDIR=~/tmp/automake
 
--> Comparison with the tree installed by the the reference version
+-> Comparison with the tree installed by the reference version
 
   make uninstall DESTDIR=~/tmp/automake
 
--
2.11.0


Bjarni Ingi Gislason <bjarniig>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gbranden (Posted a comment)
  • -email is unavailable- added by schwarze (Posted a comment)
  • -email is unavailable- added by bgarrigues (Updated the item)
  • -email is unavailable- added by bjarniig (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-03-30 gbranden Summary[PATCH] fixed doubled words in text fix doubled words in text
    2022-01-22 gbranden CategoryNone General
        Item GroupNone Lint
        Assigned toNone schwarze
        Summarymisc: Remove repeated words in files in &quot;top_srcdir&quot; [PATCH] fixed doubled words in text
    2018-02-16 schwarze StatusNone Rejected
        Open/ClosedOpen Closed
    2017-09-01 bgarrigues Severity3 - Normal 2 - Minor

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code