#476 Broken support for cyrillic characters in openmsx-catapult in mingw32

open
nobody
wxcatapult (3)
5
2013-10-05
2013-07-18
joxy
No

openmsx catapult 0.9.1:

Cyrillic chars show up as garbage:

Status Info:

Screen saved to /home/egp/Изображения/MSX/ALESTE.png

1 Attachments

Discussion

  • Manuel Bilderbeek

    • labels: --> wxcatapult
    • Group: Next_release --> Postpone_until_after_next_release
    • Priority: 5 --> 9
     
  • Manuel Bilderbeek

    As the old Catapult is currently unmaintained, the current openMSX team will not fix it. Please provide a patch to get this fixed.

     
  • joxy

    joxy - 2013-07-24

    In process of debugging -

    (11:19:46) egp_: got to the meat
    (11:23:02) egp_: =) this stuff fails - an intermediate debug log of catapult

    in CatapultXMLParser::cb_text .cpp

    #define X1 (wxT("UTF-8"))
    #define X2 (wxCSConv(X1))
    #define X3 ((const wxChar *)X2.cMB2WX((const char *)chars))
    #define X4 (wxString(X3,len))
    
    {Joe
    x1 'UTF-8'
    x3 'Screen saved to /home/egp/&#x418;&#x437;&#x43E;&#x431;&#x440;&#x430;&#x436;&#x435;&#x43D;&#x438;&#x44F;/MSX/tmp1.png</log>
    '
    x4 'Screen saved to /home/egp/'
    len=26
    }
    {Joe
    x1 'UTF-8'
    x3 'И'
    x4 'И'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'з'
    x4 'з'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'о'
    x4 'о'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'б'
    x4 'б'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'р'
    x4 'р'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'а'
    x4 'а'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'ж'
    x4 'ж'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'е'
    x4 'е'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'н'
    x4 'н'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'и'
    x4 'и'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 'я'
    x4 'я'
    len=2
    }
    {Joe
    x1 'UTF-8'
    x3 '/MSX/tmp1.png</log>
    '
    x4 '/MSX/tmp1.png'
    len=13
    }
    
     
    Last edit: joxy 2013-07-24
  • joxy

    joxy - 2013-07-24

    Here's a patch (attached). It fixes this bug completely. It is based on on mth's words that openmsx communicates using UTF-8.

     
    Last edit: joxy 2013-07-24
  • joxy

    joxy - 2013-07-24

    More efficient patch2, created with help by mth. Works OK.

     
  • joxy

    joxy - 2013-07-30

    patch is not yet committed to git. Test on Linux shows:

    Screen saved to /home/egp/Изображения/tmp1.png

     
  • joxy

    joxy - 2013-07-30

    committed. tested on linux, all works OK. tests on msvc+mingw are pending.

     
  • joxy

    joxy - 2013-07-30

    broken under mingw32:

    Screen saved to C:/MinGW/msys/1.0/home/egp/Изаобра.png

     
  • Manuel Bilderbeek

    • summary: Broken support for cyrillic characters in openmsx-catapult --> Broken support for cyrillic characters in openmsx-catapult in mingw32
    • Priority: 9 --> 5
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks