fix: Corrupting classes where an attribute's name is the last CP index

pom.xml

@@ -7,7 +7,7 @@
     <groupId>me.coley</groupId>
     <artifactId>cafedude</artifactId>
     <url>https://github.com/Col-E/CAFED00D/</url>
-    <version>1.7.0</version>
+    <version>1.7.1</version>
     <name>CAFED00D</name>
 
     <properties>

src/main/java/me/coley/cafedude/ConstPool.java

@@ -110,7 +110,17 @@ public boolean isIndexOfType(int index, Class<? extends ConstPoolEntry> type) {
 	}
 
 	/**
-	 * @return Number of constants in the constant pool. Includes empty entries after wide values <i>(long/double)</i>
+	 * The number of slots in the constant pool, including empty spaces for wide entry padding
+	 * <i>(A mistake even sun regrets)</i>.
+	 * <br>
+	 * Note that even if the pool has a size of {@code 1}
+	 * you should still use {@code 1} as the argument for {@link #get(int)} since the pool is not
+	 * 0-indexed. Instead indices start at 1.
+	 *
+	 * @return Number of constants in the constant pool.
+	 * Includes empty entries after wide values <i>(long/double)</i>
+	 * <br>
+	 * {@code 0} when there are no items.
 	 */
 	public int size() {
 		if (last == null)

src/main/java/me/coley/cafedude/attribute/AttributeCpAccessValidator.java

@@ -68,7 +68,7 @@ public static <T extends Attribute> boolean isValid(ConstPool cp, T attribute) {
 	private <T extends Attribute> boolean isValid(T attribute) {
 		// Check name index
 		int maxCpIndex = pool.size();
-		if (attribute.getNameIndex() >= maxCpIndex)
+		if (attribute.getNameIndex() > maxCpIndex)
 			return false;
 		// Check indices match certain types (key=cp_index, value=mask of allowed cp_tags)
 		String name = pool.getUtf(attribute.getNameIndex());