RFR: JDK-8240136: Cleanup/simplify HTML/CSS for definition lists

Pavel Rappo pavel.rappo at oracle.com
Fri Feb 28 20:04:24 UTC 2020


Jon, you might be confusing this directory with a similarly named
*file* TestExternalOverridenMethod.java, which you indeed renamed.

The directory I'm talking about is called "testExternalOverridenMethod",
and it is still there:

http://hg.openjdk.java.net/jdk/jdk/file/52367bbd4bd4/test/langtools/jdk/javadoc/doclet/testExternalOverridenMethod

Kindly correct me if I'm mistaken. Today is Friday,
and I may see all sorts of things which are not there.

-Pavel

> On 28 Feb 2020, at 19:50, Jonathan Gibbons <jonathan.gibbons at oracle.com> wrote:
> 
> Mercurial doesn't clean up empty directories, nor can I do anything to the main repo
> to make the empty directory in your repo go away.
> 
> `rmdir` is your friend.
> 
> -- Jon
> 
> On 2/28/20 11:47 AM, Pavel Rappo wrote:
>> That's fine. However, there's still a *directory*, called "testExternalOverridenMethod"
>> 
>> $ find . -iname "*overriden*"
>> ./test/langtools/jdk/javadoc/doclet/testExternalOverridenMethod
>> 
>>> On 28 Feb 2020, at 19:00, Jonathan Gibbons <jonathan.gibbons at oracle.com> wrote:
>>> 
>>> On 02/28/2020 07:25 AM, Pavel Rappo wrote:
>>>> If it is still possible, could you please fix the spelling in the name of the
>>>> "testExternalOverridenMethod" folder? I think we missed that yesterday while
>>>> reviewing (now pushed)http://hg.openjdk.java.net/jdk/jdk/rev/52367bbd4bd4.
>>> Pavel,
>>> 
>>> That patch has the rename in it.  There are no such badly-named files any more:
>>> 
>>> $ find open/test/langtools/ -name \*riden\*.java
>>> $
>>> 
>>> 
>>> -- Jon



More information about the javadoc-dev mailing list