long instead of int for millis in StatsdTimer interface

nzjess / statsd-over-slf4j

A Java statsd client, combined with an SLF4J compatible logging facility.

BSD 2-Clause "Simplified" License

19 stars 6 forks source link

long instead of int for millis in StatsdTimer interface #4

Closed incubos closed 12 years ago

incubos commented 12 years ago

System.currentTimeMillis() returns long. As a consequence arithmetics with time intervals (measured in ms) is usually done in long. It would be much more convenient if you changed StatsdTimer interface to accept millis as long values.

nzjess commented 12 years ago

OK fix for this is pushed now.

Good suggestion. Thanks.