Merge pull request #989 from stevenschlansker/timer-runnable

Add an overload of Timer#time that takes Runnable

metrics-core/src/main/java/com/codahale/metrics/Timer.java

@@ -104,6 +104,21 @@ public <T> T time(Callable<T> event) throws Exception {
         }
     }
 
+    /**
+     * Times and records the duration of event.
+     *
+     * @param event a {@link Runnable} whose {@link Runnable#run()} method implements a process
+     *              whose duration should be timed
+     */
+    public void time(Runnable event) {
+        final long startTime = clock.getTick();
+        try {
+            event.run();
+        } finally {
+            update(clock.getTick() - startTime);
+        }
+    }
+
     /**
      * Returns a new {@link Context}.
      *

metrics-core/src/test/java/com/codahale/metrics/TimerTest.java

@@ -4,6 +4,7 @@
 
 import java.util.concurrent.Callable;
 import java.util.concurrent.TimeUnit;
+import java.util.concurrent.atomic.AtomicBoolean;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.offset;
@@ -69,6 +70,25 @@ public String call() throws Exception {
         verify(reservoir).update(50000000);
     }
 
+    @Test
+    public void timesRunnableInstances() throws Exception {
+        final AtomicBoolean called = new AtomicBoolean();
+        timer.time(new Runnable() {
+            @Override
+            public void run() {
+                called.set(true);
+            }
+        });
+
+        assertThat(timer.getCount())
+                .isEqualTo(1);
+
+        assertThat(called.get())
+                .isTrue();
+
+        verify(reservoir).update(50000000);
+    }
+
     @Test
     public void timesContexts() throws Exception {
         timer.time().stop();