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><pre><span class="removed">In addition to<br>
</span><br></pre>should be removed, a remnant of a time when LinkedList did not implement Deque?<br><br>Martin<br><br><div class="gmail_quote">On Wed, Jul 21, 2010 at 19:04, 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;">Hello.<br>
<br>
Please code review this simple fix to the LinkedList javadoc for bug 6717780 "(coll spec) LinkedList api documentation provides the wrong method name:" in the sentence<br>
<br>
"In addition to implementing the List interface, the LinkedList class provides uniformly named methods to get, remove and insert an element at the beginning and end of the list."<br>
<br>
the word "insert" should be "add".  I've also added text to explicitly state that the beginning-of-list methods are 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>
<br>
Thanks,<br>
<br>
-Joe<br>
<br>
--- old/src/share/classes/java/util/LinkedList.java    2010-07-21 18:58:12.000000000 -0700<br>
+++ new/src/share/classes/java/util/LinkedList.java    2010-07-21 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.  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>
+ * 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 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>
</blockquote></div><br>