Skip to content

Commit f753951

Browse files
committed
Improve sprintf() and lsprintf() runtime performance
1 parent f828f47 commit f753951

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/main/java/com/laytonsmith/core/functions/StringHandling.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1434,15 +1434,16 @@ public Mixed exec(Target t, Environment environment, Mixed... args) throws Confi
14341434
}
14351435
}
14361436

1437-
if(requiredArgs(parsed) != flattenedArgs.size()) {
1437+
int requiredArgs = requiredArgs(parsed);
1438+
if(requiredArgs != flattenedArgs.size()) {
14381439
throw new CREInsufficientArgumentsException("The specified format string: \"" + formatString + "\""
1439-
+ " expects " + requiredArgs(parsed) + " argument(s),"
1440+
+ " expects " + requiredArgs + " argument(s),"
14401441
+ " but " + flattenedArgs.size() + " were provided.", t);
14411442
}
14421443

14431444
//Now figure out how to cast things, now that we know our argument numbers will match up
14441445
Object[] params = new Object[flattenedArgs.size()];
1445-
for(int i = 0; i < requiredArgs(parsed); i++) {
1446+
for(int i = 0; i < requiredArgs; i++) {
14461447
Mixed arg = flattenedArgs.get(i);
14471448
FormatString fs = parsed.get(i);
14481449
Character c = fs.getExpectedType();

0 commit comments

Comments
 (0)