----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111264/#review35240 -----------------------------------------------------------
Hi Friedrich, the m_name variable is not used anywhere else in the class but if it would be used, then it must have the same value as was assigned to the styleName variable. Both should contain the same style name with spaces replaced by an underscore. - Matus Uzak On June 26, 2013, 9:22 p.m., Friedrich W. H. Kossebau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111264/ > ----------------------------------------------------------- > > (Updated June 26, 2013, 9:22 p.m.) > > > Review request for Calligra, Inge Wallin and Matus Uzak. > > > Description > ------- > > Saw that when going over all the usages of QString::replace(...) (which > changes the object itself and returns a reference to it, as you surely know) > > Looks fishy to me that m_name is changed by the call to replace on creating a > name for the style. No idea if that is wanted, I rather guess not. > Matus, Inge, you are more the experts here, so please give it a check :) > > Okay to backport to 2.7 & 2.6 ? > > > Diffs > ----- > > filters/words/docx/import/DocxXmlStylesReader.cpp 8b38716 > > Diff: http://git.reviewboard.kde.org/r/111264/diff/ > > > Testing > ------- > > > Thanks, > > Friedrich W. H. Kossebau > >
_______________________________________________ calligra-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/calligra-devel
