I can strongly recommend using link checker tools.
For a long time, linklint has been the recommendation. However, linklint
cannot cope with "id" attribute on tags that are not tags. That's
legal in HTML 5, but linklint is an old tool. There are newer tools out
there, for both online and off
On 2017-05-12 17:23, Roger Riggs wrote:
Hi Magnus,
Webrev updated:
http://cr.openjdk.java.net/~rriggs/webrev-broken-links-8180082/
Looks good to me.
On 5/12/2017 4:26 AM, Magnus Ihse Bursie wrote:
On 2017-05-10 23:05, Brian Burkhalter wrote:
Hi Roger,
Looks all right to me. I assume
+1
Mandy
> On May 12, 2017, at 11:07 AM, Lance Andersen
> wrote:
>
> Hi all,
>
> This change updates the JDBC spec that is being pointed to
>
> —
> hg diff
> diff -r e2b414957632
> src/java.sql.rowset/share/classes/javax/sql/rowset/package.html
> --- a/src/java.sql.rowset/share/classes
Hi all,
This change updates the JDBC spec that is being pointed to
—
hg diff
diff -r e2b414957632
src/java.sql.rowset/share/classes/javax/sql/rowset/package.html
--- a/src/java.sql.rowset/share/classes/javax/sql/rowset/package.html Fri May
12 10:43:28 2017 -0700
+++ b/src/java.sql.rowset
On 5/12/17 9:11 AM, Mandy Chung wrote:
Minor comment:
95 private final List COMMON_ARGS;
This is an instance field and you can use lower case as the convention.
238 return !s.equals("");
You can consider using String::isEmpty.
Thanks, Mandy. I will make these cha
> On May 12, 2017, at 4:39 AM, Daniel Fuchs wrote:
>
> Hi,
>
> I have refreshed my webrev to use {@extLink } instead as this
> is what is needed now for links to technical guides:
>
> http://cr.openjdk.java.net/~dfuchs/webrev_8180176/webrev.00/index.html
Looks good.
Mandy
On 12/05/17 12:39, Daniel Fuchs wrote:
Hi,
I have refreshed my webrev to use {@extLink } instead as this
is what is needed now for links to technical guides:
http://cr.openjdk.java.net/~dfuchs/webrev_8180176/webrev.00/index.html
Looks good.
-Chris.
> On May 11, 2017, at 3:25 PM, Brent Christian
> wrote:
>
> Hi,
>
> I have one more update, with a couple of suggested changes to simplify the
> execute() calls:
>
> * execute() takes a vararg, so explicit String[] creation can be omitted
> (mostly).
>
> * args common to every execute() ca
Looks fine.
On 5/12/2017 11:19 AM, Stuart Marks wrote:
Hi all,
Another quick doc change. This one removes a link from Iterable to
technotes/guides and adjusts adjacent wording to match. Patch below.
Bug link: https://bugs.openjdk.java.net/browse/JDK-8180137
Thanks,
s'marks
# HG changeset
Hi Magnus,
Webrev updated:
http://cr.openjdk.java.net/~rriggs/webrev-broken-links-8180082/
On 5/12/2017 4:26 AM, Magnus Ihse Bursie wrote:
On 2017-05-10 23:05, Brian Burkhalter wrote:
Hi Roger,
Looks all right to me. I assume you will have already built the
actual docs and clicked throug
Hi all,
Another quick doc change. This one removes a link from Iterable to
technotes/guides and adjusts adjacent wording to match. Patch below.
Bug link: https://bugs.openjdk.java.net/browse/JDK-8180137
Thanks,
s'marks
# HG changeset patch
# User smarks
# Date 1494544788 25200
# Thu Ma
On 11.05.2017 18:29, Cédric Champeau wrote:
Unfortunately, they are not safely mutable in multi-threaded
programs on many operating system/libc combinations.
But the problem is less about mutating, that it is about reading: the VM
returns wrong values at some point, because it _assumes
Hi Daniel,
Changes looks good, i was under impression that if {@extLink }
description is multi word then you have to put it under double quotes
like {xtLink jndi_overview "JNDI documentation"} but looks like that is
not the case ExtLink.java will take care of this.
Thanks,
Vyom
On Friday
Hi,
I have refreshed my webrev to use {@extLink } instead as this
is what is needed now for links to technical guides:
http://cr.openjdk.java.net/~dfuchs/webrev_8180176/webrev.00/index.html
best regards,
-- daniel
On 11/05/2017 12:54, Daniel Fuchs wrote:
Hi,
Please find below a patch for:
Hi Kumar,
On 12/05/2017 00:16, Kumar Srinivasan wrote:
Hi Daniel,
As Jon surmised, this is an ARIA/accessibility requirement, in that one
can't
have holes in the usage of h* tags, as javadoc tool itself uses h1 and h2,
the API docs have to start with h3.
With respect to your comment, the h2-
Apologies if this is the wrong place for this post, but there doesn't
appear to be a webstart specific list.
Is it possible to generate a JNLP file with arguments that change each
invocation, and avoid the webstart security popup being displayed each time?
Background
I have a signed webstart
On 11/05/2017 20:46, Jonathan Gibbons wrote:
Daniel,
As a general comment, it is an accessibility error to have "gaps" in the
header numbering, such as an following an without an
intervening .
Oh! Thank you for the precision. I will take a second look with
that in mind.
If you can point t
On 11/05/2017 22:25, Andrew Guibert wrote:
Alan, could you please commit this patch for me? We've tested it
locally and all of the jdk_other tests pass with this change on jdk9.
93d92
< * Once installed, the builder cannot be replaced.
101d99
< * @exception IllegalStateException If a factory h
Hi Roger,
Looks good!
Nit:
java/lang/management/package.html:
38 conforms to the {@link javax.management JMX}
should probably use {@linkplain } instead.
(no need for a new webrev)
best regards,
-- daniel
On 11/05/2017 20:41, Roger Riggs wrote:
Hi,
Thanks for the review and suggestions.
{
On 2017-05-10 23:05, Brian Burkhalter wrote:
Hi Roger,
Looks all right to me. I assume you will have already built the actual docs and
clicked through the updated links. Always a bit painful …
Roger,
Did you test the actual links?
I found one needing updating:
In src/java.base/share/classe
20 matches
Mail list logo