Re: URL guarantees that the ftp protocol handler is present, worth re-considering this?

2012-10-16 Thread Tom Hawtin

On 15/10/2012 18:20, Mike Duigou wrote:

We should also nuke gopher and netdoc (in sun.net.www.protocol package)

I am not sure if mailto should continue as there are better solutions.


I think it's worth pointing out that in practice URL is not used just to 
open a connection. It's also used as a strong type to pass URL text 
around (poor choice of class, but there it is). For instance, it's used 
in deployment to pass to the browser. With the URL design, you need a 
handler for the protocol in order to construct it. So no passing a 
javascript: URL to showDocument (unless a third-party has registered a 
handler, or a handler is specified in the constructor within a 
privileged context).


Tom


Re: URL guarantees that the ftp protocol handler is present, worth re-considering this?

2012-10-16 Thread Alan Bateman

On 16/10/2012 13:24, Tom Hawtin wrote:

On 15/10/2012 18:20, Mike Duigou wrote:

We should also nuke gopher and netdoc (in sun.net.www.protocol package)

I am not sure if mailto should continue as there are better solutions.


I think it's worth pointing out that in practice URL is not used just 
to open a connection. It's also used as a strong type to pass URL text 
around (poor choice of class, but there it is). For instance, it's 
used in deployment to pass to the browser. With the URL design, you 
need a handler for the protocol in order to construct it. So no 
passing a javascript: URL to showDocument (unless a third-party has 
registered a handler, or a handler is specified in the constructor 
within a privileged context).
Yes, that's right, there is still a lot of code that uses URL instead of 
URI. In any case, the proposal here is just to change the specification 
to remove "ftp" from the list of protocol handlers that are required to 
be present. The ftp protocol handler will of course still be there so 
existing code will still work.


-Alan


hg: jdk8/tl/jdk: 8000245: SimpleDateFormat.format(date, StringBuffer, FieldPosition) doesn't work as expected with custom extensions; ...

2012-10-16 Thread naoto . sato
Changeset: 32452042b781
Author:naoto
Date:  2012-10-16 10:59 -0700
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/32452042b781

8000245: SimpleDateFormat.format(date, StringBuffer, FieldPosition) doesn't 
work as expected with custom extensions
8000273: java.util.Locale.getDisplayVariant(Locale l) isn't transferred to the 
custom service provider
8000615: JRE adapter: timezone name of en_US is changed when extension 
directory is added
Reviewed-by: okutsu

! src/share/classes/sun/util/locale/provider/CurrencyNameProviderImpl.java
! src/share/classes/sun/util/locale/provider/LocaleNameProviderImpl.java
! src/share/classes/sun/util/locale/provider/LocaleServiceProviderPool.java
! src/share/classes/sun/util/locale/provider/TimeZoneNameProviderImpl.java
! test/java/util/Locale/LocaleProviders.java
! test/java/util/Locale/LocaleProviders.sh
! test/java/util/PluggableLocale/CurrencyNameProviderTest.java
! test/java/util/PluggableLocale/LocaleNameProviderTest.java
! test/java/util/PluggableLocale/LocaleNameProviderTest.sh
! test/java/util/PluggableLocale/ProviderTest.java
! test/java/util/PluggableLocale/TimeZoneNameProviderTest.java
! test/java/util/PluggableLocale/providersrc/LocaleNameProviderImpl.java



hg: jdk8/tl/jdk: 7198073: (prefs) user prefs not saved [macosx]

2012-10-16 Thread kurchi . subhra . hazra
Changeset: 3a6b76a468bd
Author:khazra
Date:  2012-10-16 15:23 -0700
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3a6b76a468bd

7198073: (prefs) user prefs not saved [macosx]
Summary: Using class member field to get node instead of argument
Reviewed-by: alanb

! src/macosx/classes/java/util/prefs/MacOSXPreferences.java
+ test/java/util/prefs/CheckUserPrefFirst.java
+ test/java/util/prefs/CheckUserPrefLater.java
+ test/java/util/prefs/CheckUserPrefsStorage.sh



hg: jdk8/tl/langtools: 8000673: remove dead code from HtmlWriter and subtypes

2012-10-16 Thread jonathan . gibbons
Changeset: 2013982bee34
Author:jjg
Date:  2012-10-16 21:03 -0700
URL:   http://hg.openjdk.java.net/jdk8/tl/langtools/rev/2013982bee34

8000673: remove dead code from HtmlWriter and subtypes
Reviewed-by: bpatel

! src/share/classes/com/sun/tools/doclets/formats/html/AbstractMemberWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/HtmlDocletWriter.java
! 
src/share/classes/com/sun/tools/doclets/formats/html/HtmlSerialFieldWriter.java
! 
src/share/classes/com/sun/tools/doclets/formats/html/HtmlSerialMethodWriter.java
! 
src/share/classes/com/sun/tools/doclets/formats/html/SerializedFormWriterImpl.java
! 
src/share/classes/com/sun/tools/doclets/formats/html/SubWriterHolderWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/markup/HtmlDocWriter.java
! src/share/classes/com/sun/tools/doclets/formats/html/markup/HtmlWriter.java
! 
src/share/classes/com/sun/tools/doclets/internal/toolkit/AnnotationTypeWriter.java