Skip to content

Commit

Permalink
Issue 10245 - Fixed Checkstyle issues
Browse files Browse the repository at this point in the history
  • Loading branch information
SreeramdasLavanya committed Sep 26, 2024
1 parent 84a37e9 commit c1090b7
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 6 deletions.
6 changes: 4 additions & 2 deletions api/src/context/java/io/grpc/Deadline.java
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,8 @@ public static Deadline after(long duration, TimeUnit units) {
}

public static Deadline after(Duration duration) {
return after(TimeUnit.NANOSECONDS.convert(duration.getSeconds(), TimeUnit.SECONDS), TimeUnit.NANOSECONDS, SYSTEM_TICKER);
return after(TimeUnit.NANOSECONDS.convert(duration.getSeconds(), TimeUnit.SECONDS),
TimeUnit.NANOSECONDS, SYSTEM_TICKER);
}

/**
Expand Down Expand Up @@ -101,7 +102,8 @@ public static Deadline after(long duration, TimeUnit units, Ticker ticker) {
}

public static Deadline after(Duration duration, Ticker ticker) {
return after(TimeUnit.NANOSECONDS.convert(duration.getSeconds(), TimeUnit.SECONDS), TimeUnit.NANOSECONDS, ticker);
return after(TimeUnit.NANOSECONDS.convert(duration.getSeconds(), TimeUnit.SECONDS),
TimeUnit.NANOSECONDS, ticker);
}

private final Ticker ticker;
Expand Down
5 changes: 3 additions & 2 deletions api/src/main/java/io/grpc/SynchronizationContext.java
Original file line number Diff line number Diff line change
Expand Up @@ -197,8 +197,9 @@ public String toString() {
public final ScheduledHandle scheduleWithFixedDelay (
final Runnable task, Duration initialDelay, Duration delay,
ScheduledExecutorService timerService) {
return scheduleWithFixedDelay(task, TimeUnit.NANOSECONDS.convert(initialDelay.getSeconds(), TimeUnit.SECONDS),
TimeUnit.NANOSECONDS.convert(delay.getSeconds(), TimeUnit.SECONDS), TimeUnit.NANOSECONDS, timerService);
return scheduleWithFixedDelay(task, TimeUnit.NANOSECONDS.convert(initialDelay.getSeconds(),
TimeUnit.SECONDS), TimeUnit.NANOSECONDS.convert(delay.getSeconds(), TimeUnit.SECONDS),
TimeUnit.NANOSECONDS, timerService);
}

private static class ManagedRunnable implements Runnable {
Expand Down
3 changes: 2 additions & 1 deletion api/src/test/java/io/grpc/DeadlineTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,8 @@ public void deadlineMatchesLongValue() {

@Test
public void deadLineMatchesDurationValue(){
assertEquals(10, Deadline.after(Duration.ofMinutes(10), ticker).timeRemaining(TimeUnit.MINUTES));
assertEquals(10, Deadline.after(Duration.ofMinutes(10), ticker)
.timeRemaining(TimeUnit.MINUTES));
}

@Test
Expand Down
2 changes: 1 addition & 1 deletion api/src/test/java/io/grpc/SynchronizationContextTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,7 @@ public void schedule() {
}

@Test
public void scheduleWithFixedDelayDuration(){
public void scheduleWithFixedDelayDuration() {
MockScheduledExecutorService executorService = new MockScheduledExecutorService();
ScheduledHandle handle =
syncContext.scheduleWithFixedDelay(task1, Duration.ofSeconds(10),
Expand Down

0 comments on commit c1090b7

Please sign in to comment.