diff --git a/src/java/org/apache/poi/hssf/record/aggregates/ColumnInfoRecordsAggregate.java b/src/java/org/apache/poi/hssf/record/aggregates/ColumnInfoRecordsAggregate.java
index 1b356fb3e..1ad0488ce 100644
--- a/src/java/org/apache/poi/hssf/record/aggregates/ColumnInfoRecordsAggregate.java
+++ b/src/java/org/apache/poi/hssf/record/aggregates/ColumnInfoRecordsAggregate.java
@@ -171,7 +171,7 @@ public final class ColumnInfoRecordsAggregate extends RecordAggregate {
 	/**
 	 * 'Collapsed' state is stored in a single column col info record immediately after the outline group
 	 * @param idx
-	 * @return
+	 * @return true, if the column is collapsed, false otherwise.
 	 */
 	private boolean isColumnGroupCollapsed(int idx) {
 		int endOfOutlineGroupIdx = findEndOfColumnOutlineGroup(idx);
diff --git a/src/java/org/apache/poi/ss/formula/functions/Finance.java b/src/java/org/apache/poi/ss/formula/functions/Finance.java
index 405841912..ab7a4009e 100644
--- a/src/java/org/apache/poi/ss/formula/functions/Finance.java
+++ b/src/java/org/apache/poi/ss/formula/functions/Finance.java
@@ -107,7 +107,7 @@ public class Finance {
      * @return <code>double</code> representing principal portion of payment.
      * 
      * @see #pmt(double, int, double, double, int)
-     * @see #ipmt(double, int, int, double, double, boolean)
+     * @see #ipmt(double, int, int, double, double, int)
      */
 	static public double ppmt(double r, int per, int nper, double pv, double fv, int type) {
 	    return pmt(r, nper, pv, fv, type) - ipmt(r, per, nper, pv, fv, type);
diff --git a/src/java/org/apache/poi/ss/util/AreaReference.java b/src/java/org/apache/poi/ss/util/AreaReference.java
index 2082634d1..5d747c59c 100644
--- a/src/java/org/apache/poi/ss/util/AreaReference.java
+++ b/src/java/org/apache/poi/ss/util/AreaReference.java
@@ -261,6 +261,8 @@ public class AreaReference {
     }
 
     /**
+     * Returns a text representation of this area reference.
+     * <p/>
      *  Example return values:
      *    <table border="0" cellpadding="1" cellspacing="0" summary="Example return values">
      *      <tr><th align='left'>Result</th><th align='left'>Comment</th></tr>
diff --git a/src/java/org/apache/poi/ss/util/CellReference.java b/src/java/org/apache/poi/ss/util/CellReference.java
index f7c37e300..10b410555 100644
--- a/src/java/org/apache/poi/ss/util/CellReference.java
+++ b/src/java/org/apache/poi/ss/util/CellReference.java
@@ -437,6 +437,8 @@ public class CellReference {
 	}
 
 	/**
+	 * Returns a text representation of this cell reference.
+	 * <p/>
 	 *  Example return values:
 	 *	<table border="0" cellpadding="1" cellspacing="0" summary="Example return values">
 	 *	  <tr><th align='left'>Result</th><th align='left'>Comment</th></tr>
diff --git a/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java b/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
index 26c17c540..5765b6dd3 100644
--- a/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
+++ b/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
@@ -755,7 +755,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
 	 *             If rule M1.12 is not verified : Packages shall not contain
 	 *             equivalent part names and package implementers shall neither
 	 *             create nor recognize packages with equivalent part names.
-	 * @see {@link#createPartImpl(URI, String)}
+	 * @see #createPartImpl(PackagePartName, String, boolean)
 	 */
 	PackagePart createPart(PackagePartName partName, String contentType,
 			boolean loadRelationships) {