Rockbox

  • Status Unconfirmed
  • Percent Complete
    0%
  • Task Type Patches
  • Category User Interface
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Phinitnun Chanasabaeng - 2008-12-01

FS#9595 - Correct thai font display

This patch will correct how thai vowels and tonal makers display.

   patch (4.5 KiB)
Rafaël Carré commented on 2008-12-02 16:33

you should use only 1 > and 1 < comparison for the thai vowels when they are next to each other:

+ if(*ucs == 0xe47 || *ucs == 0xe48 || *ucs == 0xe49 || *ucs == 0xe4a
*ucs == 0xe4b || *ucs == 0xe4c || *ucs == 0xe4d || *ucs == 0xe4e)

should become

if(*ucs >= 0xe47 && *ucs ⇐ 0xe4e)

Phinitnun Chanasabaeng commented on 2008-12-02 16:51

Thank you for your comment. Here is the updated patch.

   patch (3.9 KiB)
freeMAN commented on 2008-12-29 21:21

HELP ME PLS

Jonas Häggqvist commented on 2008-12-30 02:49

A few
Rockbox is C89 - no mixing code with variable
This patch seems prone to bitrot, since you have the same code in 3 places. Would it be possible to centralise it?

Phinitnun Chanasabaeng commented on 2008-12-30 03:26

Jonas, thank you for your comment. This is a very dirty patch. I think adding somethings beyond this would be a new feature rather than a patch. I have been thinking about adding the complex text layout support to rockbox but my time doesn’t allow me to :).

About the C89, as I’m not a programmer by nature, I don’t know much about it. But I would look at it to standardize myself.

Thank you for your suggestion. :)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing