Both look fine to me. No need for a webrev. Mike
On Feb 18 2013, at 22:58 , 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