You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@empire-db.apache.org by do...@apache.org on 2014/07/29 10:33:23 UTC

svn commit: r1614282 - /empire-db/trunk/empire-db/src/main/java/org/apache/empire/exceptions/EmpireException.java

Author: doebele
Date: Tue Jul 29 08:33:23 2014
New Revision: 1614282

URL: http://svn.apache.org/r1614282
Log:
EMPIREDB-206

Modified:
    empire-db/trunk/empire-db/src/main/java/org/apache/empire/exceptions/EmpireException.java

Modified: empire-db/trunk/empire-db/src/main/java/org/apache/empire/exceptions/EmpireException.java
URL: http://svn.apache.org/viewvc/empire-db/trunk/empire-db/src/main/java/org/apache/empire/exceptions/EmpireException.java?rev=1614282&r1=1614281&r2=1614282&view=diff
==============================================================================
--- empire-db/trunk/empire-db/src/main/java/org/apache/empire/exceptions/EmpireException.java (original)
+++ empire-db/trunk/empire-db/src/main/java/org/apache/empire/exceptions/EmpireException.java Tue Jul 29 08:33:23 2014
@@ -46,7 +46,7 @@ public class EmpireException extends Run
             if (pattern==null)
                 pattern=errType.getMessagePattern();
             // init format args
-            Object[] formatArgs = (Object[])params;
+            Object[] formatArgs = params;
             // Check parameter count
             int patParamCount = errType.getNumParams();
             int paramCount = (params!=null) ? params.length : 0;            
@@ -59,24 +59,27 @@ public class EmpireException extends Run
             }
             // more params than expected
             else if (paramCount>patParamCount)
-            {   // Wildcard for the rest
+            {   // check for wildcard
                 if (pattern.contains("{*}")) 
                 {   pattern = pattern.replace("{*}", "{"+String.valueOf(patParamCount)+"}");
                     patParamCount++;
                 }
                 // Build new array
-                formatArgs = new String[patParamCount];
-                int i=0;
-                for (; i<patParamCount-1; i++)
-                    formatArgs[i]=params[i];
-                // Build a array for the rest
-                StringBuilder b = new StringBuilder();
-                for (;i<paramCount;i++)
-                {   if (b.length()>0)
-                        b.append(", ");
-                    b.append(String.valueOf(params[i]));
+                if (patParamCount>0)
+                {	// reduce number of arguments
+	                formatArgs = new String[patParamCount];
+	                int i=0;
+	                for (; i<patParamCount-1; i++)
+	                    formatArgs[i]=params[i];
+	                // Build a array for the rest
+	                StringBuilder b = new StringBuilder();
+	                for (;i<paramCount;i++)
+	                {   if (b.length()>0)
+	                        b.append(", ");
+	                    b.append(String.valueOf(params[i]));
+	                }
+	            	formatArgs[patParamCount-1]=b.toString();
                 }
-                formatArgs[patParamCount-1]=b.toString();
             }
             // format now
             String msg = MessageFormat.format(pattern, formatArgs);