Here's a possible counter-patch that removes a lot of text by delegating most of the wording to the Deque javadoc, and makes LinkedList's Dequeness as first-class as its Listness.<br><br>diff --git a/src/share/classes/java/util/LinkedList.java b/src/share/classes/java/util/LinkedList.java<br>
--- a/src/share/classes/java/util/LinkedList.java<br>+++ b/src/share/classes/java/util/LinkedList.java<br>@@ -26,18 +26,9 @@<br> package java.util;<br> <br> /**<br>- * Linked list implementation of the {@code List} interface.  Implements all<br>
- * optional list operations, and permits all elements (including<br>- * {@code null}).  In addition to implementing the {@code List} interface,<br>- * the {@code LinkedList} class provides uniformly named methods to<br>- * {@code get}, {@code remove} and {@code insert} an element at the<br>
- * beginning and end of the list.  These operations allow linked lists to be<br>- * used as a stack, {@linkplain Queue queue}, or {@linkplain Deque<br>- * double-ended queue}.<br>- *<br>- * <p>The class implements the {@code Deque} interface, providing<br>
- * first-in-first-out queue operations for {@code add},<br>- * {@code poll}, along with other stack and deque operations.<br>+ * Linked list implementation of the {@link List} and {@link Deque} interfaces.<br>+ * Implements all optional operations, and permits all elements (including<br>
+ * {@code null}).<br>  *<br>  * <p>All of the operations perform as could be expected for a doubly-linked<br>  * list.  Operations that index into the list will traverse the list from<br><br>Martin<br><br><div class="gmail_quote">
On Thu, Jul 22, 2010 at 09:40, Joe Darcy <span dir="ltr"><<a href="mailto:joe.darcy@oracle.com">joe.darcy@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">Martin Buchholz wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Perhaps much of the text is redundant with the specification in the second paragraph, that talks about how a LinkedList is also a Deque.<br>
<br>
Perhaps the sentence starting<br>
In addition to<br>
<br>
<br>
  should be removed, a remnant of a time when LinkedList did not implement Deque?<br>
</blockquote>
<br></div>
Hmm.  I'm more inclined to keep the revised sentence<br>
<br>
"In addition to implementing the List interface, the LinkedList class provides uniformly named methods to get, remove and add an element at the beginning (operationFirst) and end (operationLast) of the list."<br>

<br>
and combine the next two sentences like so:<br>
<br>
"These operations allow linked lists to be used as a stack, queue, or deque (double-ended queue); implementing the Deque interface includes support for first-in-first-out operations for add and poll."<br>
<br>
What do you think?<br>
<br>
-Joe<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Martin<div class="im"><br>
<br>
On Wed, Jul 21, 2010 at 19:04, Joe Darcy <<a href="mailto:joe.darcy@oracle.com" target="_blank">joe.darcy@oracle.com</a> <mailto:<a href="mailto:joe.darcy@oracle.com" target="_blank">joe.darcy@oracle.com</a>>> wrote:<br>

<br>
    Hello.<br>
<br>
    Please code review this simple fix to the LinkedList javadoc for<br>
    bug 6717780 "(coll spec) LinkedList api documentation provides the<br>
    wrong method name:" in the sentence<br>
<br>
    "In addition to implementing the List interface, the LinkedList<br>
    class provides uniformly named methods to get, remove and insert<br>
    an element at the beginning and end of the list."<br>
<br>
    the word "insert" should be "add".  I've also added text to<br>
    explicitly state that the beginning-of-list methods are<br>
    operationFirst and the end of list methods are operationLast.<br>
<br>
    Patch below, full webrev at<br>
    <a href="http://cr.openjdk.java.net/%7Edarcy/6717780.0/" target="_blank">http://cr.openjdk.java.net/~darcy/6717780.0/</a><br></div>
    <<a href="http://cr.openjdk.java.net/%7Edarcy/6717780.0/" target="_blank">http://cr.openjdk.java.net/%7Edarcy/6717780.0/</a>><div><div></div><div class="h5"><br>
<br>
    Thanks,<br>
<br>
    -Joe<br>
<br>
    --- old/src/share/classes/java/util/LinkedList.java    2010-07-21<br>
    18:58:12.000000000 -0700<br>
    +++ new/src/share/classes/java/util/LinkedList.java    2010-07-21<br>
    18:58:12.000000000 -0700<br>
    @@ -26,14 +26,15 @@<br>
    package java.util;<br>
<br>
    /**<br>
    - * Linked list implementation of the {@code List} interface.<br>
     Implements all<br>
    - * optional list operations, and permits all elements (including<br>
    - * {@code null}).  In addition to implementing the {@code List}<br>
    interface,<br>
    - * the {@code LinkedList} class provides uniformly named methods to<br>
    - * {@code get}, {@code remove} and {@code insert} an element at the<br>
    - * beginning and end of the list.  These operations allow linked<br>
    lists to be<br>
    - * used as a stack, {@linkplain Queue queue}, or {@linkplain Deque<br>
    - * double-ended queue}.<br>
    + * Linked list implementation of the {@code List} interface.<br>
    + * Implements all optional list operations, and permits all elements<br>
    + * (including {@code null}).  In addition to implementing the {@code<br>
    + * List} interface, the {@code LinkedList} class provides uniformly<br>
    + * named methods to {@code get}, {@code remove} and {@code add} an<br>
    + * element at the beginning (<i>operation</i>{@code First}) and end<br>
    + * (<i>operation</i>{@code Last}) of the list.  These operations<br>
    allow<br>
    + * linked lists to be used as a stack, {@linkplain Queue queue}, or<br>
    + * {@linkplain Deque double-ended queue}.<br>
     *<br>
     * <p>The class implements the {@code Deque} interface, providing<br>
     * first-in-first-out queue operations for {@code add},<br>
<br>
<br>
</div></div></blockquote>
<br>
</blockquote></div><br>