Re: svn commit: r1548830 - /lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFSLockFactory.java

2013-12-07 Thread Han Jiang
OK! thanks for the fix!


On Sat, Dec 7, 2013 at 7:21 PM, Uwe Schindler  wrote:

> Hi Han,
>
> I committed an even better fix for this, using the native javadocs linking
> instead of an HTML link. By that its automatically also correct in
> branch_4x (where the docs have to go to Java 6 SE javadocs).
>
> Uwe
>
> -
> Uwe Schindler
> H.-H.-Meier-Allee 63, D-28213 Bremen
> http://www.thetaphi.de
> eMail: u...@thetaphi.de
>
>
> > -Original Message-
> > From: h...@apache.org [mailto:h...@apache.org]
> > Sent: Saturday, December 07, 2013 11:34 AM
> > To: comm...@lucene.apache.org
> > Subject: svn commit: r1548830 -
> > /lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleF
> > SLockFactory.java
> >
> > Author: han
> > Date: Sat Dec  7 10:34:21 2013
> > New Revision: 1548830
> >
> > URL: http://svn.apache.org/r1548830
> > Log:
> > broken link police
> >
> > Modified:
> >
> > lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFS
> > LockFactory.java
> >
> > Modified:
> > lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFS
> > LockFactory.java
> > URL:
> > http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/
> > apache/lucene/store/SimpleFSLockFactory.java?rev=1548830&r1=1548829&r
> > 2=1548830&view=diff
> > ==
> > 
> > ---
> > lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFS
> > LockFactory.java (original)
> > +++
> > lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFS
> > LockFactory.java Sat Dec  7 10:34:21 2013
> > @@ -25,7 +25,7 @@ import java.io.IOException;
> >   * File#createNewFile()}.
> >   *
> >   * NOTE: the  > - *
> > href="
> http://java.sun.com/j2se/1.4.2/docs/api/java/io/File.html#createNew
> > File()">javadocs
> > + *
> > href="
> http://docs.oracle.com/javase/7/docs/api/java/io/File.html#createNe
> > wFile()">javadocs
> >   * for File.createNewFile contain a vague
> >   * yet spooky warning about not using the API for file
> >   * locking.  This warning was added due to 
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
> For additional commands, e-mail: dev-h...@lucene.apache.org
>
>


-- 
Han Jiang

Team of Search Engine and Web Mining,
School of Electronic Engineering and Computer Science,
Peking University, China


RE: svn commit: r1548830 - /lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFSLockFactory.java

2013-12-07 Thread Uwe Schindler
Hi Han,

I committed an even better fix for this, using the native javadocs linking 
instead of an HTML link. By that its automatically also correct in branch_4x 
(where the docs have to go to Java 6 SE javadocs).

Uwe

-
Uwe Schindler
H.-H.-Meier-Allee 63, D-28213 Bremen
http://www.thetaphi.de
eMail: u...@thetaphi.de


> -Original Message-
> From: h...@apache.org [mailto:h...@apache.org]
> Sent: Saturday, December 07, 2013 11:34 AM
> To: comm...@lucene.apache.org
> Subject: svn commit: r1548830 -
> /lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleF
> SLockFactory.java
> 
> Author: han
> Date: Sat Dec  7 10:34:21 2013
> New Revision: 1548830
> 
> URL: http://svn.apache.org/r1548830
> Log:
> broken link police
> 
> Modified:
> 
> lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFS
> LockFactory.java
> 
> Modified:
> lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFS
> LockFactory.java
> URL:
> http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/
> apache/lucene/store/SimpleFSLockFactory.java?rev=1548830&r1=1548829&r
> 2=1548830&view=diff
> ==
> 
> ---
> lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFS
> LockFactory.java (original)
> +++
> lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/store/SimpleFS
> LockFactory.java Sat Dec  7 10:34:21 2013
> @@ -25,7 +25,7 @@ import java.io.IOException;
>   * File#createNewFile()}.
>   *
>   * NOTE: the  - *
> href="http://java.sun.com/j2se/1.4.2/docs/api/java/io/File.html#createNew
> File()">javadocs
> + *
> href="http://docs.oracle.com/javase/7/docs/api/java/io/File.html#createNe
> wFile()">javadocs
>   * for File.createNewFile contain a vague
>   * yet spooky warning about not using the API for file
>   * locking.  This warning was added due to