Bug 577905 mark DEBUG only variables as ifdef DEBUG in svg

r=dholbert a=roc

--HG--
extra : rebase_source : 1ff1359b298ffc5f273f3d0448e0e676d0d95069
This commit is contained in:
timeless@mozdev.org 2010-07-11 15:30:35 +03:00
parent 5d7a393be3
commit c1d4d80ee3
3 changed files with 21 additions and 6 deletions

View File

@ -551,7 +551,11 @@ nsSMILAnimationFunction::ComputePacedPosition(const nsSMILValueArray& aValues,
NS_ASSERTION(remainingDist >= 0, "distance values must be non-negative");
double curIntervalDist;
nsresult rv = aValues[i].ComputeDistance(aValues[i+1], curIntervalDist);
#ifdef DEBUG
nsresult rv =
#endif
aValues[i].ComputeDistance(aValues[i+1], curIntervalDist);
NS_ABORT_IF_FALSE(NS_SUCCEEDED(rv),
"If we got through ComputePacedTotalDistance, we should "
"be able to recompute each sub-distance without errors");

View File

@ -75,7 +75,11 @@ nsSVGAttrTearoffTable<SimpleType, TearoffType>::GetTearoff(SimpleType* aSimple)
return nsnull;
TearoffType *tearoff = nsnull;
PRBool found = mTable.Get(aSimple, &tearoff);
#ifdef DEBUG
PRBool found =
#endif
mTable.Get(aSimple, &tearoff);
NS_ABORT_IF_FALSE(!found || tearoff,
"NULL pointer stored in attribute tear-off map");
@ -98,7 +102,10 @@ nsSVGAttrTearoffTable<SimpleType, TearoffType>::AddTearoff(SimpleType* aSimple,
return;
}
PRBool result = mTable.Put(aSimple, aTearoff);
#ifdef DEBUG
PRBool result =
#endif
mTable.Put(aSimple, aTearoff);
NS_ABORT_IF_FALSE(result, "Out of memory.");
}

View File

@ -1349,9 +1349,13 @@ nsSVGElement::UpdateAnimatedContentStyleRule()
animContentStyleRule(mappedAttrParser.CreateStyleRule());
if (animContentStyleRule) {
nsresult rv = SetProperty(SMIL_MAPPED_ATTR_ANIMVAL,
SMIL_MAPPED_ATTR_STYLERULE_ATOM,
animContentStyleRule.get(), ReleaseStyleRule);
#ifdef DEBUG
nsresult rv =
#endif
SetProperty(SMIL_MAPPED_ATTR_ANIMVAL,
SMIL_MAPPED_ATTR_STYLERULE_ATOM,
animContentStyleRule.get(),
ReleaseStyleRule);
animContentStyleRule.forget();
NS_ABORT_IF_FALSE(rv == NS_OK,
"SetProperty failed (or overwrote something)");