← Back to team overview

kicad-developers team mailing list archive

Re: UTF8 class and wchar_t issues fix

 

Hi Wayne,

I have tested it and indeed it fixes the problem with Eagle import as well.

Cheers,
Orson

On 12/08/2017 02:42 PM, Wayne Stambaugh wrote:
> Orson,
> 
> Did you test this with any eagle projects that contain non-ascii
> characters before pushing this patch?
> 
> Cheers,
> 
> Wayne
> 
> On 12/08/2017 08:38 AM, Maciej Sumiński wrote:
>> Hi Jean-Pierre,
>>
>> Thank you for the fix, I think you are right. I applied the patch to the
>> master branch.
>>
>> Regards,
>> Orson
>>
>> On 12/08/2017 01:53 PM, jp charras wrote:
>>> Hi Orson
>>>
>>> Could you have a look into this patch.
>>>
>>> I am thinking it is also a fix for bug 1737143.
>>>
>>> Currently our UTF8 class has only a append operator for ASCII7 chars.
>>> But if the code uses something like (it is accepted by the compiler):
>>> my_utf_8_string += mychar;
>>> and if mychar is a int (or a wchar_t) containing a non ASCII7 value, the UTF8 string is broken.
>>> the patch adds a append operator for non ASCII7 value (wchar_t wide chars)
>>>
>>>
>>> I am pretty sure this kind of issues also exists in eagle import (when eagle files uses non ASCII7
>>> chars in texts).
>>>
>>> Thanks.
>>>
>>
>>
>>
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
>>
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
> 


Attachment: signature.asc
Description: OpenPGP digital signature


Follow ups

References