diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-05-16 14:06:30 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-05-18 09:17:30 +0200 |
commit | 60861faa8653afebb504cfbcaeed633d2373a27d (patch) | |
tree | 1dec7a4ba05a212052fd64439645f024643d78ec /basegfx/source/polygon/b2dsvgpolypolygon.cxx | |
parent | d89fa2bd4944625e2dbe56d5709a3f126db24f21 (diff) |
loplugin:comparisonwithconstant in basegfx
Change-Id: I6953640a1aa2e58fe2ea6555291c4f4a5271770f
Reviewed-on: https://gerrit.libreoffice.org/37680
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'basegfx/source/polygon/b2dsvgpolypolygon.cxx')
-rw-r--r-- | basegfx/source/polygon/b2dsvgpolypolygon.cxx | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/basegfx/source/polygon/b2dsvgpolypolygon.cxx b/basegfx/source/polygon/b2dsvgpolypolygon.cxx index 71b6717e2625..7514abe6e3c8 100644 --- a/basegfx/source/polygon/b2dsvgpolypolygon.cxx +++ b/basegfx/source/polygon/b2dsvgpolypolygon.cxx @@ -62,7 +62,7 @@ namespace basegfx bool bRelative(false); const sal_Unicode aCurrChar(rSvgDStatement[nPos]); - if(o_rPolyPolygon.count() && !aCurrPoly.count() && !('m' == aCurrChar || 'M' == aCurrChar)) + if(o_rPolyPolygon.count() && !aCurrPoly.count() && !(aCurrChar == 'm' || aCurrChar == 'M')) { // we have a new sub-polygon starting, but without a 'moveto' command. // this requires to add the current point as start point to the polygon @@ -113,7 +113,7 @@ namespace basegfx case 'l' : case 'L' : { - if('m' == aCurrChar || 'l' == aCurrChar) + if(aCurrChar == 'm' || aCurrChar == 'l') { bRelative = true; } @@ -756,8 +756,8 @@ namespace basegfx // That's what is done from our import, so avoid exporting it as first statement // is necessary. const bool bSymmetricAtEdgeStart( - 0 != nIndex - && B2VectorContinuity::C2 == aPolygon.getContinuityInPoint(nIndex)); + nIndex != 0 + && aPolygon.getContinuityInPoint(nIndex) == B2VectorContinuity::C2); if(bDetectQuadraticBeziers) { @@ -849,7 +849,7 @@ namespace basegfx else { // straight edge - if(0 == nNextIndex) + if(nNextIndex == 0) { // it's a closed polygon's last edge and it's not a bezier edge, so there is // no need to write it |