[Poppler-bugs] [Bug 66693] Greek support package - some characters output as symbols not letters

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=66693

--- Comment #14 from Adrian Johnson <ajohnson at redneon.com> ---
Comment on attachment 84138
  --> https://bugs.freedesktop.org/attachment.cgi?id=84138
Normalize more characters in font Unicode map

Review of attachment 84138:
-----------------------------------------------------------------

::: poppler/GfxFont.cc
@@ +1438,5 @@
> +	  || u[i] == 0x220F // ∏
> +	  || u[i] == 0x2211 // ∑
> +	  || (u[i] >= 0xFB00 && u[i] <= 0xFB4F) // Alphabetic Presentation Forms
> +	  || (u[i] >= 0xFB50 && u[i] <= 0xFDFF) // Arabic Presentation Forms-A
> +	  || (u[i] >= 0xFE70 && u[i] <= 0xFEFF) // Arabic Presentation Forms-B

I don't like the way all the characters to normalize have been shoved into an
if statement like this. Could they be put in a table or something where there
is a separation between the list of characters and the code to perform the
normalization?

It would also be good if you could include the removed comment that provided
examples of the alphabetic presentation forms 'eg "fi", "ffi"' as not everyone
who will read this code is familiar with the various unicode ranges.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20130817/aed99a4d/attachment.html>