Hi,
I did not catch the irc conversation. The backslash in "\%" seems to be
ignored internally. I don't think it's an undocumented escape sequence
understood by QString, so I would remove it.
Any objections from sebsauer and LukasT who were using those?
-matus
On Sun, Mar 18, 2012 at 4:52 AM, Andrius da Costa Ribas <
andrius...@gmail.com> wrote:
> Please find attached the patch for 2.4 branch (Either reviewboard does not
> accept patches for non-master or I don't know how to).
>
> 2012/3/17 Andrius da Costa Ribas <andrius...@gmail.com>
>
>> Hello,
>>
>> As discussed on IRC, some sources contains "\%" invalid escape sequence:
>>
>> filters/libmso/ODrawToOdf.cpp(625): style.addAttribute("svg:x1",
>> QString("%1\%").arg(x1));
>> filters/libmso/ODrawToOdf.cpp(626): style.addAttribute("svg:y1",
>> QString("%1\%").arg(y1));
>> filters/libmso/ODrawToOdf.cpp(627): style.addAttribute("svg:x2",
>> QString("%1\%").arg(x2));
>> filters/libmso/ODrawToOdf.cpp(628): style.addAttribute("svg:y2",
>> QString("%1\%").arg(y2));
>> filters/libmso/ODrawToOdf.cpp(680): style.addAttribute("svg:x1",
>> QString("%1\%").arg(x1));
>> filters/libmso/ODrawToOdf.cpp(681): style.addAttribute("svg:y1",
>> QString("%1\%").arg(y1));
>> filters/libmso/ODrawToOdf.cpp(682): style.addAttribute("svg:x2",
>> QString("%1\%").arg(x2));
>> filters/libmso/ODrawToOdf.cpp(683): style.addAttribute("svg:y2",
>> QString("%1\%").arg(y2));
>> filters/sheets/xlsx/ChartExport.cpp(240):
>> style.addProperty( "draw:opacity", QString( "%1\%" ).arg(
>> chart()->m_areaFormat->m_foreground.alphaF() * 100.0 ),
>> KoGenStyle::GraphicType );
>> filters/sheets/xlsx/ChartExport.cpp(302):
>> style.addProperty( "draw:opacity", QString( "%1\%" ).arg(
>> areaFormat->m_foreground.alphaF() * 100.0 ), KoGenStyle::GraphicType );
>>
>> Is it okay to just replace "\%" by "%" ?
>>
>> Thanks,
>> Andrius.
>>
>
>
> _______________________________________________
> calligra-devel mailing list
> calligra-devel@kde.org
> https://mail.kde.org/mailman/listinfo/calligra-devel
>
>
_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel
