[Libreoffice-commits] core.git: qadevOOo/runner sc/qa

Noel Grandin noel at peralex.com
Wed Oct 14 01:39:49 PDT 2015


 qadevOOo/runner/lib/MultiPropertyTest.java |    6 +++---
 sc/qa/complex/sc/CalcRTL.java              |    7 ++-----
 2 files changed, 5 insertions(+), 8 deletions(-)

New commits:
commit 8e3f72141625898c01e39e94d3fdaefd66c885ff
Author: Noel Grandin <noel at peralex.com>
Date:   Wed Oct 14 10:37:15 2015 +0200

    casting before calling getClass() is a waste of time
    
    Change-Id: Id88aa9e43b736e6fdeaf098a952dae5421468f83

diff --git a/qadevOOo/runner/lib/MultiPropertyTest.java b/qadevOOo/runner/lib/MultiPropertyTest.java
index 918d5e6..9946710 100644
--- a/qadevOOo/runner/lib/MultiPropertyTest.java
+++ b/qadevOOo/runner/lib/MultiPropertyTest.java
@@ -359,7 +359,7 @@ public class MultiPropertyTest extends MultiMethodTest
                         {
                             if (!util.utils.isVoid(oldValue) && oldValue instanceof Any)
                             {
-                                oldValue = AnyConverter.toObject(new Type(((Any) oldValue).getClass()), oldValue);
+                                oldValue = AnyConverter.toObject(new Type(oldValue.getClass()), oldValue);
                             }
                             log.println("result = " + toString(resValue));
                         }
@@ -393,7 +393,7 @@ public class MultiPropertyTest extends MultiMethodTest
                         {
                             if (!util.utils.isVoid(oldValue) && oldValue instanceof Any)
                             {
-                                oldValue = AnyConverter.toObject(new Type(((Any) oldValue).getClass()), oldValue);
+                                oldValue = AnyConverter.toObject(new Type(oldValue.getClass()), oldValue);
                             }
                             log.println("result = " + toString(resValue));
                         }
@@ -427,7 +427,7 @@ public class MultiPropertyTest extends MultiMethodTest
                         {
                             if (!util.utils.isVoid(oldValue) && oldValue instanceof Any)
                             {
-                                oldValue = AnyConverter.toObject(new Type(((Any) oldValue).getClass()), oldValue);
+                                oldValue = AnyConverter.toObject(new Type(oldValue.getClass()), oldValue);
                             }
                             log.println("result = " + toString(resValue));
                         }
diff --git a/sc/qa/complex/sc/CalcRTL.java b/sc/qa/complex/sc/CalcRTL.java
index 3c3fdd4..9f946a5 100644
--- a/sc/qa/complex/sc/CalcRTL.java
+++ b/sc/qa/complex/sc/CalcRTL.java
@@ -403,11 +403,8 @@ public class CalcRTL
                                 "' has changed");
 
                     try {
-                        if (!util.utils.isVoid(oldValue) &&
-                                oldValue instanceof Any) {
-                            oldValue = AnyConverter.toObject(
-                                               new Type(((Any) oldValue).getClass()),
-                                               oldValue);
+                        if (!util.utils.isVoid(oldValue) && oldValue instanceof Any) {
+                            oldValue = AnyConverter.toObject( new Type(oldValue.getClass()), oldValue);
                         }
 
                         System.out.println("old = " + toString(oldValue));


More information about the Libreoffice-commits mailing list