@vpisarev Vadim, please have a look
@kirill-kornyakov FYI

@vpisarev done

Hi, @BKNio. Looks like letter ё (yo) does not work.

Maybe I'm doing something wrong?

Hi, @mshabunin

Yep, it doesn't work, because Hershey doesn't have Ё and ё

Maybe we should use е glyph instead? I think it will be better than p and ?.

@mshabunin fixed

Thanks, it looks better now.
I have some more thoughts about this functionality:

• since we are now partially supporting UTF-8, we should test it (modules/highgui/test/test_drawing.cpp could be such place for 2.4 branch)
• you've removed font check, now drawing cyrillic with different font (or italic) causes invalid reads and segmentation fault
• malformed UTF-8 string can cause invalid reads

For example, drawing

string text = "\xC0"; // CYRILLIC CAPITAL LETTER A in CP1251 encoding

causes

==22270== Invalid read of size 1
==22270==    at 0x4F66056: cv::putText(cv::Mat&, std::string const&, cv::Point_<int>, int, double, cv::Scalar_<double>, int, int, bool) (drawing.cpp:2012)

@BKNio, @mshabunin, so what's the decision? In my opinion, this is useful patch and we should finish it. At minimum, there should be a check whether the font contains cyrillic glyphs or not. The handling of malformed strings would be nice to have, but such a check can be added in a subsequent pull request.

Let's put it in and then submit further fixes when needed.
👍