On Fri, 2013-09-27 at 14:52 -0700, julien2412 wrote:
> Hello,
> 
> Cppcheck reported this:
> sc/source/core/tool/interpr7.cxx
>      94                         else if(pNodeSet->nodeTab[0]->type ==
> XML_ELEMENT_NODE)
>      95                         {
>      96                             xmlNodePtr cur = pNodeSet->nodeTab[0];
>      97                             boost::shared_ptr<xmlChar>
> pChar2(xmlNodeGetContent(cur), xmlFree);
>      98                             aResult =
> OUString::createFromAscii((char*)pChar2.get());
>      99                         }
>     100                         else
>     101                         {
>     102                             xmlNodePtr cur = pNodeSet->nodeTab[0];
>     103                             boost::shared_ptr<xmlChar>
> pChar2(xmlNodeGetContent(cur), xmlFree);
>     104                             aResult =
> OUString::createFromAscii((char*)pChar2.get());
>     105                         }
> 
> See
> http://opengrok.libreoffice.org/xref/core/sc/source/core/tool/interpr7.cxx#87

So I decided to just remove the duplication for now. Markus might want
to have a look at it as the original author.

C.

(and the other issue in svx is fixed now as well as per the earlier
mail).

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to