diff --git a/WHATSNEW b/WHATSNEW
index 719b509da..d780b227e 100644
--- a/WHATSNEW
+++ b/WHATSNEW
@@ -63,6 +63,8 @@ Other changes:
 * Added -noclasspath option to allow running ant WITHOUT using CLASSPATH env
   variable. Modified ant.bat to do this so that %CLASSPATH% is not looked at.
 
+* Added revision and userid attributes to <pvcs> documentation.
+
 Changes from Ant 1.6.2 to current Ant 1.6 CVS version
 =====================================================
 
diff --git a/docs/manual/OptionalTasks/pvcstask.html b/docs/manual/OptionalTasks/pvcstask.html
index b13647360..0e113d5a9 100644
--- a/docs/manual/OptionalTasks/pvcstask.html
+++ b/docs/manual/OptionalTasks/pvcstask.html
@@ -170,6 +170,16 @@ tag expects the executables to be found using the PATH environment variable.</td
           mapping.</td>
       <td valign="TOP">No</td>
     </tr>
+    <tr>
+      <td valign="TOP">revision</td>
+      <td valign="TOP">Retrieve the specified revision.</td>
+      <td valign="TOP">No</td>
+    </tr>
+    <tr>
+      <td valign="TOP">userid</td>
+      <td valign="TOP">Use the specified userid.</td>
+      <td valign="TOP">No</td>
+    </tr>
 </table>
 <h3><a name="nested">Nested Elements</a></h3>
 
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/pvcs/Pvcs.java b/src/main/org/apache/tools/ant/taskdefs/optional/pvcs/Pvcs.java
index eb6396245..ee7cee1d6 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/pvcs/Pvcs.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/pvcs/Pvcs.java
@@ -622,8 +622,7 @@ public class Pvcs extends org.apache.tools.ant.Task {
     }
 
     /**
-     * User ID; unused.
-     * @ant.attribute ignore="true"
+     * User ID
      */
 
     public void setUserId(String u) {