RFR 8181832: Broken link in javax/sql/rowset/spi/package-summary.html
Lance Andersen
lance.andersen at oracle.com
Tue Jun 13 22:17:27 UTC 2017
A few more updates as there was a bit more crud to address. The webrev is at:
http://cr.openjdk.java.net/~lancea/8181832/webrev.00/ <http://cr.openjdk.java.net/~lancea/8181832/webrev.00/>
Best
Lance
> On Jun 9, 2017, at 9:38 AM, Lance Andersen <lance.andersen at oracle.com> wrote:
>
>>
>> On Jun 9, 2017, at 9:26 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>>
>>
>>
>> On 09/06/2017 14:15, Lance Andersen wrote:
>>> Hi all,
>>>
>>> This patches addresses the broken link reported in JDK-8181832
>>>
>>> ————
>>> $ hg diff
>>> diff -r 67ad6b89dd96 src/java.sql.rowset/share/classes/javax/sql/rowset/spi/package.html
>>> --- a/src/java.sql.rowset/share/classes/javax/sql/rowset/spi/package.html Wed Jun 07 15:05:35 2017 -0400
>>> +++ b/src/java.sql.rowset/share/classes/javax/sql/rowset/spi/package.html Fri Jun 09 09:13:49 2017 -0400
>>> @@ -119,7 +119,7 @@
>>> techniques for maintaining the integrity of data. However, specific standards
>>> groups have considered offering XML-based synchronization. For details, see
>>> <PRE>
>>> - <a href="http://www.syncml.org">http://www.syncml.org</a>
>>> + <a href="http://www.openmobilealliance.org/wp/Affiliates/SyncML.html">http://www.openmobilealliance.org/wp/Affiliates/SyncML.html</a>
>>> </PRE>
>>>
>> Would it be better to drop this paragraph and link completely?
>>
>> I also wonder if the paragraphs about the RI should move into their own section so that it is clear that they are implementation specific - users of this API can't assume the two providers discussed are present.
>
> I think for 9, I would prefer to just address the broken link and look to revisit for 10. This has been documented this way since JDK 5, so I think if I am going to make changes, I would like to spend some more time thinking about it.
>
> Is that acceptable?
>
> Best
> Lance
>>
>> -Alan
>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif <http://oracle.com/us/design/oracle-email-sig-198324.gif>>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif <http://oracle.com/us/design/oracle-email-sig-198324.gif>> <http://oracle.com/us/design/oracle-email-sig-198324.gif <http://oracle.com/us/design/oracle-email-sig-198324.gif>>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif <http://oracle.com/us/design/oracle-email-sig-198324.gif>>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
> Oracle Java Engineering
> 1 Network Drive
> Burlington, MA 01803
> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com> <mailto:Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev
mailing list