Skip to content

Commit 024e4b6

Browse files
committed
[IO-860] Use a long instead of an int for
ThrottledInputStream.Builder.setMaxBytes(long, ChronoUnit)
1 parent b522681 commit 024e4b6

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/changes/changes.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ The <action> type attribute can be add,update,fix,remove.
4949
<release version="2.19.0" date="YYYY-MM-DD" description="Version 2.18.1: Java 8 is required.">
5050
<!-- FIX -->
5151
<!-- ADD -->
52-
<action dev="ggregory" type="add" issue="IO-860" due-to="Nico Strecker, Gary Gregory">Add ThrottledInputStream.Builder.setMaxBytes(int, ChronoUnit).</action>
52+
<action dev="ggregory" type="add" issue="IO-860" due-to="Nico Strecker, Gary Gregory">Add ThrottledInputStream.Builder.setMaxBytes(long, ChronoUnit).</action>
5353
<!-- UPDATE -->
5454
</release>
5555
<release version="2.18.0" date="2024-11-16" description="Version 2.18.0: Java 8 is required.">

src/main/java/org/apache/commons/io/input/ThrottledInputStream.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ public ThrottledInputStream get() throws IOException {
126126
* @throws IllegalArgumentException Thrown if maxBytesPerSecond &lt;= 0.
127127
* @since 2.19.0
128128
*/
129-
public Builder setMaxBytes(final int value, final ChronoUnit chronoUnit) {
129+
public Builder setMaxBytes(final long value, final ChronoUnit chronoUnit) {
130130
setMaxBytes(value, chronoUnit.getDuration());
131131
return asThis();
132132
}
@@ -149,7 +149,7 @@ public Builder setMaxBytes(final int value, final ChronoUnit chronoUnit) {
149149
* @throws IllegalArgumentException Thrown if maxBytesPerSecond &lt;= 0.
150150
*/
151151
// Consider making public in the future
152-
Builder setMaxBytes(final int value, final Duration duration) {
152+
Builder setMaxBytes(final long value, final Duration duration) {
153153
setMaxBytesPerSecond((double) Objects.requireNonNull(duration, "duration").toMillis() / 1_000 * value);
154154
return asThis();
155155
}

0 commit comments

Comments
 (0)