diff --git a/fillpdf.module b/fillpdf.module
index 440b336..602a194 100644
--- a/fillpdf.module
+++ b/fillpdf.module
@@ -177,12 +177,34 @@ function fillpdf_parse_uri(){
     fillpdf_merge_pdf($fid, $nids, $webforms, $sample, $force_download, FALSE, $flatten);
 }
 
-
 /**
- * @return doesn't return anything, actually constructs a page from scratch (pdf content-type)
- *   and sends it to the browser or saves it, depending on if a custom path is configured
- *   or not.
- * @seealso fillpdf_pdf_link for $_GET params
+ * Constructs a page from scratch (pdf content-type) and sends it to the
+ * browser or saves it, depending on if a custom path is configured or not.
+ *
+ * @param $fid
+ *   The integer ID of the PDF.
+ * @param $nids
+ *   Array of integer IDs of the CCK nodes from which to draw data.
+ * @param $webform_arr
+ *   Array of integer IDs of the Webform nodes from which to draw data.
+ * @param $sample
+ *   If "true" (exact string), each field will be filled with its field name.
+ * @param $force_download
+ *   Boolean. If TRUE, always send a PDF to the browser, even if a
+ *   destination_path is set for the PDF.
+ * @param $skip_access_check
+ *   Boolean. If TRUE, do not do any access checks. Allow the user to download
+ *   any PDF with data from any node. Only use when access checks are being
+ *   done some other way.
+ * @param $flatten
+ *   Boolean. If TRUE, flatten the PDF so that fields cannot be edited.
+ *   Otherwise leave fields editable.
+ *
+ * @return
+ *   Nothing.
+ *
+ * @seealso fillpdf_pdf_link()
+ * for $_GET params
  */
 function fillpdf_merge_pdf($fid, $nids = null, $webform_arr = null, $sample = null, $force_download = FALSE, $skip_access_check = FALSE, $flatten = TRUE) {
   // Case 1: No $fid