Skip to content

Commit

Permalink
core: fix floating-point number formatting Locale (#7473)
Browse files Browse the repository at this point in the history
  • Loading branch information
dapengzhang0 authored and sergiitk committed Oct 29, 2020
1 parent e9831c5 commit af97bef
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion context/src/main/java/io/grpc/Deadline.java
Expand Up @@ -17,6 +17,7 @@
package io.grpc;

import java.util.Arrays;
import java.util.Locale;
import java.util.concurrent.ScheduledExecutorService;
import java.util.concurrent.ScheduledFuture;
import java.util.concurrent.TimeUnit;
Expand Down Expand Up @@ -206,7 +207,7 @@ public String toString() {
}
buf.append(seconds);
if (nanos > 0) {
buf.append(String.format(".%09d", nanos));
buf.append(String.format(Locale.US, ".%09d", nanos));
}
buf.append("s from now");
if (ticker != SYSTEM_TICKER) {
Expand Down
3 changes: 2 additions & 1 deletion core/src/main/java/io/grpc/internal/ClientCallImpl.java
Expand Up @@ -54,6 +54,7 @@
import io.perfmark.Tag;
import java.io.InputStream;
import java.nio.charset.Charset;
import java.util.Locale;
import java.util.concurrent.CancellationException;
import java.util.concurrent.Executor;
import java.util.concurrent.ScheduledExecutorService;
Expand Down Expand Up @@ -441,7 +442,7 @@ public void run() {
buf.append('-');
}
buf.append(seconds);
buf.append(String.format(".%09d", nanos));
buf.append(String.format(Locale.US, ".%09d", nanos));
buf.append("s. ");
buf.append(insight);
stream.cancel(DEADLINE_EXCEEDED.augmentDescription(buf.toString()));
Expand Down
3 changes: 2 additions & 1 deletion core/src/main/java/io/grpc/internal/DelayedClientCall.java
Expand Up @@ -30,6 +30,7 @@
import io.grpc.Status;
import java.util.ArrayList;
import java.util.List;
import java.util.Locale;
import java.util.concurrent.Executor;
import java.util.concurrent.ScheduledExecutorService;
import java.util.concurrent.ScheduledFuture;
Expand Down Expand Up @@ -117,7 +118,7 @@ private ScheduledFuture<?> scheduleDeadlineIfNeeded(
buf.append("Deadline exceeded after ");
}
buf.append(seconds);
buf.append(String.format(".%09d", nanos));
buf.append(String.format(Locale.US, ".%09d", nanos));
buf.append("s. ");
/** Cancels the call if deadline exceeded prior to the real call being set. */
class DeadlineExceededRunnable implements Runnable {
Expand Down

0 comments on commit af97bef

Please sign in to comment.