On Sun, Nov 29, 2009 at 03:29:18PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko
wrote:
> Hello. Basic Multilingual Plane is range of Unicode characters in
> 0-65535 and it contains most of the characters needed by most of the
> languages of the world. By keeping an array with pointers to such
> characters at the cost of 128KiB per font we can almost instantenously
> lookup characters which are likely to be used in grub. Available in
> experimental
Seems fine (for trunk as well), just one thing:
> + grub_memset (font->bmp_idx, 0xff, 0x10000 * sizeof (grub_uint16_t));
> [...]
> + if (entry->code < 0x10000)
> [...]
> + /* Use BMP index if possible. */
> + if (code < 0x10000)
> + {
> + if (font->bmp_idx[code] == 0xffff)
Could these be macroified? Makes it harder to do mistakes in the future.
--
Robert Millan
The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
how) you may access your data; but nobody's threatening your freedom: we
still allow you to remove your data and not access it at all."
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
