From 45169905fe9a32b580409d379a0bec49b95c2e5d Mon Sep 17 00:00:00 2001
From: "Kristian S. Stangeland" <kr_stang@hotmail.com>
Date: Thu, 13 Sep 2012 21:15:30 +0200
Subject: [PATCH] Fix documentation.

---
 .../protocol/injector/StructureCache.java     |  6 +++++-
 .../protocol/reflect/StructureModifier.java   | 20 +++++++++++--------
 2 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/ProtocolLib/src/com/comphenix/protocol/injector/StructureCache.java b/ProtocolLib/src/com/comphenix/protocol/injector/StructureCache.java
index 267bdfd1..6b8eab6f 100644
--- a/ProtocolLib/src/com/comphenix/protocol/injector/StructureCache.java
+++ b/ProtocolLib/src/com/comphenix/protocol/injector/StructureCache.java
@@ -24,7 +24,11 @@ import net.minecraft.server.Packet;
 
 import com.comphenix.protocol.reflect.StructureModifier;
 
-public class StructureCache {
+/**
+ * Caches structure modifiers.
+ * @author Kristian
+ */
+class StructureCache {
 	// Structure modifiers
 	private static Map<Integer, StructureModifier<Object>> structureModifiers = new HashMap<Integer, StructureModifier<Object>>();
 	
diff --git a/ProtocolLib/src/com/comphenix/protocol/reflect/StructureModifier.java b/ProtocolLib/src/com/comphenix/protocol/reflect/StructureModifier.java
index dad97dba..bd23ed76 100644
--- a/ProtocolLib/src/com/comphenix/protocol/reflect/StructureModifier.java
+++ b/ProtocolLib/src/com/comphenix/protocol/reflect/StructureModifier.java
@@ -131,7 +131,7 @@ public class StructureModifier<TField> {
 	 * @param fieldIndex - index of the field.
 	 * @param value - new value of the field.
 	 * @return This structure modifier - for chaining.
-	 * @throws IllegalAccessException The field doesn't exist, or it cannot be accessed under the current security contraints.
+	 * @throws FieldAccessException The field doesn't exist, or it cannot be accessed under the current security contraints.
 	 */
 	public StructureModifier<TField> write(int fieldIndex, TField value) throws FieldAccessException {
 		if (fieldIndex < 0 || fieldIndex >= data.size())
@@ -158,7 +158,7 @@ public class StructureModifier<TField> {
 	 * @param fieldIndex - index of the potential field.
 	 * @param value - new value of the field.
 	 * @return This structure modifer - for chaining.
-	 * @throws IllegalAccessException The field cannot be accessed under the current security contraints.
+	 * @throws FieldAccessException The field cannot be accessed under the current security contraints.
 	 */
 	public StructureModifier<TField> writeSafely(int fieldIndex, TField value) throws FieldAccessException {
 		if (fieldIndex >= 0 && fieldIndex < data.size()) {
@@ -172,7 +172,7 @@ public class StructureModifier<TField> {
 	 * @param fieldIndex - index of the field to modify.
 	 * @param select - the function that modifies the field value.
 	 * @return This structure modifier - for chaining.
-	 * @throws IllegalAccessException The field cannot be accessed under the current security contraints.
+	 * @throws FieldAccessException The field cannot be accessed under the current security contraints.
 	 */
 	public StructureModifier<TField> modify(int fieldIndex, Function<TField, TField> select) throws FieldAccessException {
 		TField value = read(fieldIndex);
@@ -189,16 +189,20 @@ public class StructureModifier<TField> {
 	}
 	
 	/**
-	 * Sets all non-primitive fields to a more fitting default value. See {@link DefaultInstance}.
+	 * Sets all non-primitive fields to a more fitting default value. See {@link DefaultInstances#getDefault(Class)}.
 	 * @return The current structure modifier - for chaining.
-	 * @throws IllegalAccessException If we're unable to write to the fields due to a security limitation.
+	 * @throws FieldAccessException If we're unable to write to the fields due to a security limitation.
 	 */
-	public StructureModifier<TField> writeDefaults() throws IllegalAccessException {
+	public StructureModifier<TField> writeDefaults() throws FieldAccessException {
 		
 		// Write a default instance to every field
 		for (Field field : defaultFields) {
-			FieldUtils.writeField(field, target, 
-					DefaultInstances.getDefault(field.getType()), true);
+			try {
+				FieldUtils.writeField(field, target, 
+						DefaultInstances.getDefault(field.getType()), true);
+			} catch (IllegalAccessException e) {
+				throw new FieldAccessException("Cannot write to field due to a security limitation.", e);
+			}
 		}
 		
 		return this;