Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (nike.apache.org: domain of sebbaz@gmail.com designates
 209.85.214.171 as permitted sender)
Received-SPF: pass (google.com: domain of sebbaz@gmail.com designates
 10.182.53.66 as permitted sender) client-ip=10.182.53.66;
MIME-Version: 1.0
In-Reply-To: <4F4E4E37.6090901@systemoutprintln.de>
References: 
 <CACZkXPwgs5eGhNS8yKecD16y_XMHJgU4_=QFeW61V_Qgzqr8qg@mail.gmail.com>
	<4F4E4E37.6090901@systemoutprintln.de>
Date: Wed, 29 Feb 2012 16:54:32 +0000
Message-ID: 
 <CAOGo0VasVVnTc1SMANX-8taL0FQ-GB_vPF0ytH=WNmDRU7PZTw@mail.gmail.com>
Subject: Re: [lang] Please fix DateParser Javadocs
From: sebb <sebbaz@gmail.com>
To: Commons Developers List <dev@commons.apache.org>
Content-Type: text/plain; charset=ISO-8859-1

On 29 February 2012 16:11, Benedikt Ritter <bene@systemoutprintln.de> wrote:
> Am 29.02.2012 15:28, schrieb Gary Gregory:
>
>> Can someone please fix:
>>
>> [WARNING]
>>
>> C:\svn\org\apache\commons\trunks-proper\lang\src\main\java\org\apache\commons\lang3\time\DateParser.java:75:
>> warning - Tag @link: can't find getTimeZo
>> neOverridesCalendar() in org.apache.commons.lang3.time.DateParser
>
>
> The problem is, that there is no method with name
> "getTimeZoneOverridesCalendar()" in DateParser, so that link can't be
> resolved (I think you already knew that :).
> I suppose the comment is meant to refer to getTimeZone() in
> java.util.Calendar.

I don't think that's correct.

The Javadoc originally said:

     * <p>This zone is always used for <code>Date</code> formatting.
     * If a <code>Calendar</code> is passed in to be formatted, the
     * time zone on that may be used depending on
     * {@link #getTimeZoneOverridesCalendar()}.</p>

To me, this implies that there was/is a method called something like

getTimeZoneOverridesCalendar()

which was/is used to determine whether or not to use the timezone from
the provided calendar.

Date.getTimeZone() just happens to be a similar method name; I don't
see how it's relevant here.

The way to find out is to check what the code actually does (did).

> If so, you can fix that by changing the link to: {@link
> java.util.Calendar#getTimeZone()}
>
> Regards,
> Benedikt
>
>
>>
>> It's not obvious to me what the right fix is.
>>
>> Thank you,
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org