Skip to content

Commit cc75024

Browse files
Do not attempt to pad time string if current length is sufficient (#3676)
* Do not attempt to pad time string if current length is sufficient * Remove 'final' modifier from local variables --------- Co-authored-by: Anna-Karin Salander <[email protected]>
1 parent 1dfdfdd commit cc75024

File tree

1 file changed

+11
-6
lines changed
  • services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/internal/converter

1 file changed

+11
-6
lines changed

services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/internal/converter/ConverterUtils.java

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,14 +53,19 @@ public static void validateFloat(Float input) {
5353
}
5454

5555
public static String padLeft(int paddingAmount, int valueToPad) {
56+
String result;
5657
String value = Integer.toString(valueToPad);
57-
int padding = paddingAmount - value.length();
58-
StringBuilder result = new StringBuilder(paddingAmount);
59-
for (int i = 0; i < padding; i++) {
60-
result.append('0');
58+
if (value.length() == paddingAmount) {
59+
result = value;
60+
} else {
61+
int padding = paddingAmount - value.length();
62+
StringBuilder sb = new StringBuilder(paddingAmount);
63+
for (int i = 0; i < padding; i++) {
64+
sb.append('0');
65+
}
66+
result = sb.append(value).toString();
6167
}
62-
result.append(value);
63-
return result.toString();
68+
return result;
6469
}
6570

6671
public static String[] splitNumberOnDecimal(String valueToSplit) {

0 commit comments

Comments
 (0)