Opened 13 years ago

Closed 12 years ago

#7579 closed enhancement (fixed)

[Terminal] add cp1251/koi8-r encodings

Reported by: diver Owned by: siarzhuk
Priority: normal Milestone: R1
Component: Applications/Terminal Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description (last modified by siarzhuk)

Currently looking into this because of special request from Diver.

Change History (7)

comment:1 by siarzhuk, 12 years ago

Description: modified (diff)
Owner: changed from jackburton to siarzhuk
Status: newassigned

comment:2 by axeld, 12 years ago

The best way to go about this would be to enable those encodings in libtextencoding.so - I would think that libiconv (which is our backend there) should already contain support for it.

comment:3 by siarzhuk, 12 years ago

There are no problems with the koi-8r <-> utf-8 conversion - it works out of the box now - just after adding corresponding menu entries. I can say it works well, until you try to do something. Looks like any 8-bit encoding in our Terminal aren't functional at the moment. And that is my current headache. But on the bright side - looks like I'll kill some other tickets related to ANSI parsing. ;-)

comment:4 by zooey, 12 years ago

At least to some extent the problems Terminal has with 8-bit encodings are related to our broken implementation of the multibyte-functions (mbrtowc() and friends). During BeGeistert, I worked on that and got it basically working.

The reason for me not having applied those changes is that some ICU-related internal resource handling is still missing and the fact that those changes require a complete rebuild of the buildtools.

I'm going to commit the changes as soon as I have time for it, which is going to be after the switch to Git has been completed (i.e. at the beginning of next week).

in reply to:  4 comment:5 by siarzhuk, 12 years ago

Replying to zooey:

I'm going to commit the changes as soon as I have time for it, which is going to be after the switch to Git has been completed (i.e. at the beginning of next week).

Nice to hear that we will have functional multibyte support soon! ;-) On my side the issues are in Terminal's TermParse.cpp code and related on internal handling of ANSI control sequences - and are not related on system convertion support.

comment:6 by siarzhuk, 12 years ago

Fixed in hrev43252.

comment:7 by siarzhuk, 12 years ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.