Code review request for javax.lang.model doclint cleanup

[Joe Darcy]

Hello,

Please review the patch below which resolves a few remaining doclint 
warnings on protected elements in the javax.lang.model API; with this 
change, API is now doclint clean :-)

I'll file a bug after getting a review.

Thanks,

-Joe

diff -r 1936a884b290 
src/share/classes/javax/lang/model/util/ElementScanner6.java
--- a/src/share/classes/javax/lang/model/util/ElementScanner6.java Fri 
Jun 14 18:01:52 2013 +0100
+++ b/src/share/classes/javax/lang/model/util/ElementScanner6.java Sat 
Jun 15 10:24:11 2013 -0700
@@ -110,6 +110,8 @@
      /**
       * Constructor for concrete subclasses; uses the argument for the
       * default value.
+     *
+     * @param defaultValue the default value
       */
      protected ElementScanner6(R defaultValue){
          DEFAULT_VALUE = defaultValue;
diff -r 1936a884b290 
src/share/classes/javax/lang/model/util/ElementScanner7.java
--- a/src/share/classes/javax/lang/model/util/ElementScanner7.java Fri 
Jun 14 18:01:52 2013 +0100
+++ b/src/share/classes/javax/lang/model/util/ElementScanner7.java Sat 
Jun 15 10:24:11 2013 -0700
@@ -1,5 +1,5 @@
  /*
- * Copyright (c) 2010, 2012, Oracle and/or its affiliates. All rights 
reserved.
+ * Copyright (c) 2010, 2013, Oracle and/or its affiliates. All rights 
reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
@@ -101,6 +101,8 @@
      /**
       * Constructor for concrete subclasses; uses the argument for the
       * default value.
+     *
+     * @param defaultValue the default value
       */
      protected ElementScanner7(R defaultValue){
          super(defaultValue);
diff -r 1936a884b290 
src/share/classes/javax/lang/model/util/ElementScanner8.java
--- a/src/share/classes/javax/lang/model/util/ElementScanner8.java Fri 
Jun 14 18:01:52 2013 +0100
+++ b/src/share/classes/javax/lang/model/util/ElementScanner8.java Sat 
Jun 15 10:24:11 2013 -0700
@@ -1,5 +1,5 @@
  /*
- * Copyright (c) 2011, 2012, Oracle and/or its affiliates. All rights 
reserved.
+ * Copyright (c) 2011, 2013, Oracle and/or its affiliates. All rights 
reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
@@ -101,6 +101,8 @@
      /**
       * Constructor for concrete subclasses; uses the argument for the
       * default value.
+     *
+     * @param defaultValue the default value
       */
      protected ElementScanner8(R defaultValue){
          super(defaultValue);
diff -r 1936a884b290 
src/share/classes/javax/lang/model/util/SimpleTypeVisitor6.java
--- a/src/share/classes/javax/lang/model/util/SimpleTypeVisitor6.java 
Fri Jun 14 18:01:52 2013 +0100
+++ b/src/share/classes/javax/lang/model/util/SimpleTypeVisitor6.java 
Sat Jun 15 10:24:11 2013 -0700
@@ -118,6 +118,10 @@
       * The default action for visit methods.  The implementation in
       * this class just returns {@link #DEFAULT_VALUE}; subclasses will
       * commonly override this method.
+     *
+     * @param e the type to process
+     * @param p a visitor-specified parameter
+     * @return {@code DEFAULT_VALUE} unless overridden
       */
      protected R defaultAction(TypeMirror e, P p) {
          return DEFAULT_VALUE;