bugGNU troff - Bugs: bug #43905, [PATCH] duplicate glyph-codepoint...

 
 

bug #43905: [PATCH] duplicate glyph-codepoint mappings for curly braces

Submitted by:  Ingo Schwarze <schwarze>
Submitted on:  Tue 30 Dec 2014 10:10:44 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: Incorrect behaviourStatus: Fixed
Privacy: PublicAssigned to: Werner LEMBERG <wl>
Open/Closed: ClosedPlanned Release: 1.22.4

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 14 Jan 2015 09:13:02 PM UTC, comment #1:

Applied to git, thanks.

Regarding the other files: I don't have corresponding devices either – it's perhaps already dead code...

Werner LEMBERG <wl>
Project AdministratorIn charge of this item.
Tue 30 Dec 2014 10:10:44 AM UTC, original submission:

The glyph-to-codepoint and codepoint-to-glyph mappings in libgroff contain duplicate entries for the glyphs \[lt], \[lk], \[lb], \[rt], \[rk], and \[rb]. One consequence is that for the -Tutf8 output device, requesting the \[u23A7] glyph in the input stream (as documented in the groff_char(7) manual) surprisingly results in an U+256D character (0xE2 95 AD byte sequence) in the output stream because it gets internally translated u23A7 -> lt -> 256D. There may be other consequences. Using \[bracelefttp], on the other hand, works as expected.

The attached patch solves the problem.

The following lines in the following files look suspicious as well, in particular since those files do not contain "23A7", but i'm not providing patches for these three files because i don't own such devices, cannot properly test, and don't understand those files well enough:

font/devlj4/S:u256D 26346,7410,6588 1 128354 -- MSL 646 (15U 98)
font/devlj4/generate/special.map:646 256D u256D
src/utils/hpftodit/hpuni.cpp: { "646", "256D", },

Ingo Schwarze <schwarze>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #32733:  braces.patch added by schwarze (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by wl (Posted a comment)
  • -unavailable- added by schwarze (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 14 Jan 2015 09:13:02 PM UTCwlStatusNone=>Fixed
      Assigned toNone=>wl
      Open/ClosedOpen=>Closed
      Planned ReleaseNone=>1.22.4
    Tue 30 Dec 2014 10:10:44 AM UTCschwarzeAttached File-=>Added braces.patch, #32733

    Back to the top


    Powered by Savane 3.1-cleanup1