diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-04-12 18:39:22 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-04-15 08:37:54 +0200 |
commit | b52f309f2b9037ee53ab8ac2d66967c012ba82f1 (patch) | |
tree | 4a4ace081c742af0cef50909e06394d9aef80345 /jvmfwk | |
parent | 897493fbd34a1bd10320767b48cbf04d422f89b3 (diff) |
improve loplugin simplifyconstruct
to find stuff like
OUString s = OUString("xxx")
Change-Id: Ie7ed074c1ae012734c67a2a89c564c1900a4ab04
Reviewed-on: https://gerrit.libreoffice.org/70697
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'jvmfwk')
-rw-r--r-- | jvmfwk/source/elements.cxx | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/jvmfwk/source/elements.cxx b/jvmfwk/source/elements.cxx index 64373f31c8b0..57134e19b35d 100644 --- a/jvmfwk/source/elements.cxx +++ b/jvmfwk/source/elements.cxx @@ -409,8 +409,7 @@ void NodeJava::write() const //The element must exist if (m_enabled) { - OString sExpression= OString( - "/jf:java/jf:enabled"); + OString sExpression("/jf:java/jf:enabled"); pathObj = xmlXPathEvalExpression(reinterpret_cast<xmlChar const *>(sExpression.getStr()), contextUser); if ( ! pathObj || xmlXPathNodeSetIsEmpty(pathObj->nodesetval)) @@ -432,8 +431,7 @@ void NodeJava::write() const //The element must exist if (m_userClassPath) { - OString sExpression= OString( - "/jf:java/jf:userClassPath"); + OString sExpression("/jf:java/jf:userClassPath"); pathObj = xmlXPathEvalExpression(reinterpret_cast<xmlChar const *>(sExpression.getStr()), contextUser); if ( ! pathObj || xmlXPathNodeSetIsEmpty(pathObj->nodesetval)) @@ -447,8 +445,7 @@ void NodeJava::write() const //set <javaInfo> element if (m_javaInfo) { - OString sExpression= OString( - "/jf:java/jf:javaInfo"); + OString sExpression("/jf:java/jf:javaInfo"); pathObj = xmlXPathEvalExpression(reinterpret_cast<xmlChar const *>(sExpression.getStr()), contextUser); if ( ! pathObj || xmlXPathNodeSetIsEmpty(pathObj->nodesetval)) @@ -460,8 +457,7 @@ void NodeJava::write() const //set <vmParameters> element if (m_vmParameters) { - OString sExpression= OString( - "/jf:java/jf:vmParameters"); + OString sExpression("/jf:java/jf:vmParameters"); pathObj = xmlXPathEvalExpression(reinterpret_cast<xmlChar const *>(sExpression.getStr()), contextUser); if ( ! pathObj || xmlXPathNodeSetIsEmpty(pathObj->nodesetval)) @@ -500,8 +496,7 @@ void NodeJava::write() const //set <jreLocations> element if (m_JRELocations) { - OString sExpression= OString( - "/jf:java/jf:jreLocations"); + OString sExpression("/jf:java/jf:jreLocations"); pathObj = xmlXPathEvalExpression(reinterpret_cast<xmlChar const *>(sExpression.getStr()), contextUser); if ( ! pathObj || xmlXPathNodeSetIsEmpty(pathObj->nodesetval)) |