diff options
author | Noel Grandin <noel@peralex.com> | 2016-04-20 10:34:01 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2016-04-21 08:32:47 +0200 |
commit | 5abc669599001bf888b97c4d3c2715e1fb7523b9 (patch) | |
tree | 2407c6fc040a795e6ffc69de02ba940285c04c7f /svgio | |
parent | 5bb308a9ad16f6002486a60e4a753693818580b6 (diff) |
new plugin stylepolice
check for local variables which follow our member field naming
convention, which is highly confusing
Change-Id: Idacedf7145d09843e96a584237b385f7662eea10
Diffstat (limited to 'svgio')
-rw-r--r-- | svgio/source/svgreader/svgimagenode.cxx | 6 | ||||
-rw-r--r-- | svgio/source/svgreader/svgstyleattributes.cxx | 14 | ||||
-rw-r--r-- | svgio/source/svgreader/svgusenode.cxx | 10 |
3 files changed, 15 insertions, 15 deletions
diff --git a/svgio/source/svgreader/svgimagenode.cxx b/svgio/source/svgreader/svgimagenode.cxx index d58dd716b8cd..47c802353322 100644 --- a/svgio/source/svgreader/svgimagenode.cxx +++ b/svgio/source/svgreader/svgimagenode.cxx @@ -258,11 +258,11 @@ namespace svgio } else if(!maXLink.isEmpty()) { - const SvgNode* mpXLink = getDocument().findSvgNodeById(maXLink); + const SvgNode* pXLink = getDocument().findSvgNodeById(maXLink); - if(mpXLink && Display_none != mpXLink->getDisplay()) + if(pXLink && Display_none != pXLink->getDisplay()) { - mpXLink->decomposeSvgNode(aNewTarget, true); + pXLink->decomposeSvgNode(aNewTarget, true); if(!aNewTarget.empty()) { diff --git a/svgio/source/svgreader/svgstyleattributes.cxx b/svgio/source/svgreader/svgstyleattributes.cxx index 1a5b279432aa..2d5bc8548495 100644 --- a/svgio/source/svgreader/svgstyleattributes.cxx +++ b/svgio/source/svgreader/svgstyleattributes.cxx @@ -1145,21 +1145,21 @@ namespace svgio aSource = drawinglayer::primitive2d::Primitive2DContainer { xRef }; } - const SvgClipPathNode* mpClip = accessClipPathXLink(); - while(mpClip) + const SvgClipPathNode* pClip = accessClipPathXLink(); + while(pClip) { // #i124852# transform may be needed when userSpaceOnUse - mpClip->apply(aSource, pTransform); - mpClip = mpClip->getSvgStyleAttributes()->accessClipPathXLink(); + pClip->apply(aSource, pTransform); + pClip = pClip->getSvgStyleAttributes()->accessClipPathXLink(); } if(!aSource.empty()) // test again, applied clipPath may have lead to empty geometry { - const SvgMaskNode* mpMask = accessMaskXLink(); - if(mpMask) + const SvgMaskNode* pMask = accessMaskXLink(); + if(pMask) { // #i124852# transform may be needed when userSpaceOnUse - mpMask->apply(aSource, pTransform); + pMask->apply(aSource, pTransform); } if(!aSource.empty()) // test again, applied mask may have lead to empty geometry diff --git a/svgio/source/svgreader/svgusenode.cxx b/svgio/source/svgreader/svgusenode.cxx index 826aed2729f4..84f4bbe75785 100644 --- a/svgio/source/svgreader/svgusenode.cxx +++ b/svgio/source/svgreader/svgusenode.cxx @@ -141,9 +141,9 @@ namespace svgio void SvgUseNode::decomposeSvgNode(drawinglayer::primitive2d::Primitive2DContainer& rTarget, bool /*bReferenced*/) const { // try to access link to content - const SvgNode* mpXLink = getDocument().findSvgNodeById(maXLink); + const SvgNode* pXLink = getDocument().findSvgNodeById(maXLink); - if(mpXLink && Display_none != mpXLink->getDisplay()) + if(pXLink && Display_none != pXLink->getDisplay()) { // decompose children drawinglayer::primitive2d::Primitive2DContainer aNewTarget; @@ -151,9 +151,9 @@ namespace svgio // todo: in case mpXLink is a SVGTokenSvg or SVGTokenSymbol the // SVG docs want the getWidth() and getHeight() from this node // to be valid for the subtree. - const_cast< SvgNode* >(mpXLink)->setAlternativeParent(this); - mpXLink->decomposeSvgNode(aNewTarget, true); - const_cast< SvgNode* >(mpXLink)->setAlternativeParent(); + const_cast< SvgNode* >(pXLink)->setAlternativeParent(this); + pXLink->decomposeSvgNode(aNewTarget, true); + const_cast< SvgNode* >(pXLink)->setAlternativeParent(); if(!aNewTarget.empty()) { |