HI Joe,

These look fine.

I'd have been happy to fix these as ongoing ThreeTen work.
We'll need to be more careful merging now that changes will be coming from other
than the Threeten project.

Thanks, Roger


On 2/19/13 1:58 AM, Joe Darcy wrote:
Hello,

Please review these two simple fixes for javadoc warnings I noticed during a build (I'll file a bug after getting a review):

diff -r bcde0486261e src/share/classes/java/time/format/DateTimeFormatterBuilder.java --- a/src/share/classes/java/time/format/DateTimeFormatterBuilder.java Mon Feb 18 08:14:18 2013 +0000 +++ b/src/share/classes/java/time/format/DateTimeFormatterBuilder.java Mon Feb 18 22:55:56 2013 -0800
@@ -1007,7 +1007,7 @@
      * is used, with {@code IsoChronology} as the fallback.
      * <p>
* Note that this method provides similar functionality to methods on - * {@code DateFormat} such as {@link DateFormat#getDateTimeInstance(int, int)}. + * {@code DateFormat} such as {@link java.text.DateFormat#getDateTimeInstance(int, int)}.
      *
* @param dateStyle the date style to use, null means no date required * @param timeStyle the time style to use, null means no time required
diff -r bcde0486261e src/share/classes/java/util/TimeZone.java
--- a/src/share/classes/java/util/TimeZone.java Mon Feb 18 08:14:18 2013 +0000 +++ b/src/share/classes/java/util/TimeZone.java Mon Feb 18 22:55:56 2013 -0800
@@ -534,7 +534,7 @@
     /**
      * Gets the {@code TimeZone} for the given {@code zoneId}.
      *
- * @param zoneid a {@link ZoneId} from which the time zone ID is obtained + * @param zoneId a {@link ZoneId} from which the time zone ID is obtained * @return the specified {@code TimeZone}, or the GMT zone if the given ID
      *         cannot be understood.
      * @throws NullPointerException if {@code zoneId} is {@code null}

Thanks,

-Joe

Reply via email to