9 (docs-only) RFR: 8180428: Clarify implementation note in Clock.java to match implementation changes made by JDK-8068730

David Holmes david.holmes at oracle.com
Thu May 18 10:51:06 UTC 2017


Still looks good to me!

Thanks,
David

On 18/05/2017 7:58 PM, Daniel Fuchs wrote:
> Hi,
>
> Please find below a clarification of the implementation note
> in Clock.java:
>
> 8180428: Clarify implementation note in Clock.java to match
> implementation changes made by JDK-8068730
> https://bugs.openjdk.java.net/browse/JDK-8180428
>
> The change below was already discussed here:
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2017-May/047671.html
>
> -- daniel
>
> Here is the proposed diff:
>
> diff --git a/src/java.base/share/classes/java/time/Clock.java
> b/src/java.base/share/classes/java/time/Clock.java
> --- a/src/java.base/share/classes/java/time/Clock.java
> +++ b/src/java.base/share/classes/java/time/Clock.java
> @@ -127,10 +127,13 @@
>   * document whether or not they do support serialization.
>   *
>   * @implNote
> - * The clock implementation provided here is based on {@link
> System#currentTimeMillis()}.
> - * That method provides little to no guarantee about the accuracy of
> the clock.
> - * Applications requiring a more accurate clock must implement this
> abstract class
> - * themselves using a different external clock, such as an NTP server.
> + * The clock implementation provided here is based on the same
> underlying clock
> + * as {@link System#currentTimeMillis()}, but may have a precision
> finer than
> + * milliseconds if available.
> + * However, little to no guarantee is provided about the accuracy of the
> + * underlying clock. Applications requiring a more accurate clock must
> implement
> + * this abstract class themselves using a different external clock,
> such as an
> + * NTP server.
>   *
>   * @since 1.8
>   */


More information about the core-libs-dev mailing list