Fixed xmlpatterns review comments.

In comments it was stated that RVCT version is just fine for all
platforms and ifdef are not needed. Also the compilers will really
optimize extra temp variables for the other compilers.

Reviewed-by: Janne Koskinen

1 files changed, 0 insertions, 7 deletions

diff --git a/src/xmlpatterns/data/qabstractfloatcasters.cpp b/src/xmlpatterns/data/qabstractfloatcasters.cpp
index cdbeb67..f41bb4a 100644
--- a/src/xmlpatterns/data/qabstractfloatcasters.cpp
+++ b/src/xmlpatterns/data/qabstractfloatcasters.cpp
@@ -63,7 +63,6 @@ template <const bool isDouble>
 Item BooleanToAbstractFloatCaster<isDouble>::castFrom(const Item &from,
                                                            const QExplicitlySharedDataPointer<DynamicContext> &context) const
 {
-#if defined(Q_CC_RVCT)
     // RVCT doesn't like using template parameter in trinary operator when the trinary operator result is
     // passed directly into another constructor. 
     bool tempDouble = isDouble;
@@ -71,11 +70,5 @@ Item BooleanToAbstractFloatCaster<isDouble>::castFrom(const Item &from,
         return tempDouble ? toItem(CommonValues::DoubleOne) : toItem(CommonValues::FloatOne);
     else
         return tempDouble ? toItem(CommonValues::DoubleZero) : toItem(CommonValues::FloatZero);
-#else
-    if(from.template as<AtomicValue>()->evaluateEBV(context))
-        return isDouble ? toItem(CommonValues::DoubleOne) : toItem(CommonValues::FloatOne);
-    else
-        return isDouble ? toItem(CommonValues::DoubleZero) : toItem(CommonValues::FloatZero);
-#endif    
 }