terminal width char patch

[h.z]

Enable support for B_GBK_CONVERSION and B_BIG5_CONVERSION, enable all font family not only just fixed, add "width" to "BasicTerminalBuffer::InsertChar" (CodeConv::UTF8GetFontWidth just not works), just like old MuTerm. In TermView::Draw, add IS_WIDTH attribute, or fonts drawing will be the one on the right from snapshot view, only paint left half.

[h.z]

left one is patched terminal, right one is not

[h.z]

fix fixed font checking

[jackburton]

Hi, first of all thanks for your work. There are some issues with this patch, though: I don't like much how the fonts are filtered. We should, instead, implement BFont::IsFullAndHalfFixed() so that we can use it also in other places. I can try to do that using your code. There are some minor style issue (width_ok instead of widthOk, for example), but I can take care of this, I think.

[h.z]

I think that filter is a dirty hack, just taken from font width calc. Implement of BFont::IsFullAndHalfFixed() will be great. I will notice the style issue next time, thanks.

[korli]

I'm wondering what width exactly is : is it the display width ? Isn't there another way to know this width based on a character ?

[jackburton]

Replying to korli:

I'm wondering what width exactly is : is it the display width ? Isn't there another way to know this width based on a character ?

See the comments in bug #947, please.

[h.z]

Replying to korli:

I'm wondering what width exactly is : is it the display width ? Isn't there another way to know this width based on a character ?

I think,

1. is the font width, no matter is half or full width, must be fixed width, or "TermView" will be massed up.
2. is the char width, during the "TermParse", the char width will be calculated and "TermView::Draw" use width attribute, or only paint the left half.

[bonefish]

Some quick observations:

• There are several places in BasicTerminalBuffer (and maybe other classes) that lack special handling for full width chars. Some are even marked with a TODO.
• If CodeConv::UTF8GetFontWidth() is not needed, it should be removed.

[jackburton]

I applied the patch (with a few changes from myself) in hrev29129. I tested with Konatu Tohaba font and it seems to work correctly. Let me know if I can close this. Thanks for your work!