cppcheck: fix some redundantAssignment
Change-Id: I9ea73bada743b195d778b17038137e9089d03404
This commit is contained in:
@@ -406,10 +406,9 @@ namespace basegfx
|
|||||||
B2DPoint* pNewPoint = bAIsLonger
|
B2DPoint* pNewPoint = bAIsLonger
|
||||||
? maNewPoints.allocatePoint(aEdgeA.getStart() + (fCutA * aDeltaA))
|
? maNewPoints.allocatePoint(aEdgeA.getStart() + (fCutA * aDeltaA))
|
||||||
: maNewPoints.allocatePoint(aEdgeB.getStart() + (fCutB * aDeltaB));
|
: maNewPoints.allocatePoint(aEdgeB.getStart() + (fCutB * aDeltaB));
|
||||||
bool bRetval = false;
|
|
||||||
|
|
||||||
// try to split both edges
|
// try to split both edges
|
||||||
bRetval = splitEdgeAtGivenPoint(aEdgeA, *pNewPoint, aCurrent);
|
bool bRetval = splitEdgeAtGivenPoint(aEdgeA, *pNewPoint, aCurrent);
|
||||||
bRetval |= splitEdgeAtGivenPoint(aEdgeB, *pNewPoint, aCurrent);
|
bRetval |= splitEdgeAtGivenPoint(aEdgeB, *pNewPoint, aCurrent);
|
||||||
|
|
||||||
if(!bRetval)
|
if(!bRetval)
|
||||||
|
@@ -1437,10 +1437,9 @@ bool DiagramHelper::isSeriesMoveable(
|
|||||||
const Reference< XDataSeries >& xGivenDataSeries,
|
const Reference< XDataSeries >& xGivenDataSeries,
|
||||||
bool bForward )
|
bool bForward )
|
||||||
{
|
{
|
||||||
bool bIsMoveable = false;
|
|
||||||
const bool bDoMove = false;
|
const bool bDoMove = false;
|
||||||
|
|
||||||
bIsMoveable = lcl_moveSeriesOrCheckIfMoveIsAllowed(
|
bool bIsMoveable = lcl_moveSeriesOrCheckIfMoveIsAllowed(
|
||||||
xDiagram, xGivenDataSeries, bForward, bDoMove );
|
xDiagram, xGivenDataSeries, bForward, bDoMove );
|
||||||
|
|
||||||
return bIsMoveable;
|
return bIsMoveable;
|
||||||
@@ -1448,10 +1447,9 @@ bool DiagramHelper::isSeriesMoveable(
|
|||||||
|
|
||||||
bool DiagramHelper::moveSeries( const Reference< XDiagram >& xDiagram, const Reference< XDataSeries >& xGivenDataSeries, bool bForward )
|
bool DiagramHelper::moveSeries( const Reference< XDiagram >& xDiagram, const Reference< XDataSeries >& xGivenDataSeries, bool bForward )
|
||||||
{
|
{
|
||||||
bool bMoved = false;
|
|
||||||
const bool bDoMove = true;
|
const bool bDoMove = true;
|
||||||
|
|
||||||
bMoved = lcl_moveSeriesOrCheckIfMoveIsAllowed(
|
bool bMoved = lcl_moveSeriesOrCheckIfMoveIsAllowed(
|
||||||
xDiagram, xGivenDataSeries, bForward, bDoMove );
|
xDiagram, xGivenDataSeries, bForward, bDoMove );
|
||||||
|
|
||||||
return bMoved;
|
return bMoved;
|
||||||
|
@@ -398,8 +398,7 @@ uno::Any SwAccessibleCell::getMinimumValue( )
|
|||||||
|
|
||||||
static OUString ReplaceOneChar(OUString oldOUString, OUString replacedChar, OUString replaceStr)
|
static OUString ReplaceOneChar(OUString oldOUString, OUString replacedChar, OUString replaceStr)
|
||||||
{
|
{
|
||||||
int iReplace = -1;
|
int iReplace = oldOUString.lastIndexOf(replacedChar);
|
||||||
iReplace = oldOUString.lastIndexOf(replacedChar);
|
|
||||||
if (iReplace > -1)
|
if (iReplace > -1)
|
||||||
{
|
{
|
||||||
for(;iReplace>-1;)
|
for(;iReplace>-1;)
|
||||||
|
Reference in New Issue
Block a user