--- Comment #2 from 2012-03-06 04:13:54 PST ---
(In reply to comment #1)

Thank you. Your patch seems nice.

Two suggestions:

1) If they don't already say it, then I suggest to add in the ddoc of the
functions that the translation arrays have a length 128. This for both old D
programmers and Python programmers.

2) This code in translate():

+    bool[128] remTable;
+    remTable[] = false;

I suggest to write it like this, that's shorter and avoids a double

+    bool[128] remTable = false;

Configure issuemail:
------- You are receiving this mail because: -------

Reply via email to