Added FIXME

trustin · trustin · commit 0062cb743c75 · 2011-06-29T12:30:47.000+09:00
diff --git a/src/main/java/org/jboss/netty/handler/codec/serialization/CompactObjectInputStream.java b/src/main/java/org/jboss/netty/handler/codec/serialization/CompactObjectInputStream.java
@@ -78,11 +78,13 @@ protected Class<?> resolveClass(ObjectStreamClass desc) throws IOException, Clas
         try {
             return loadClass(className);
         } catch (ClassNotFoundException ex) {
+            // FIXME Cache the result to avoid the cost of ClassNotFoundException.
             return super.resolveClass(desc);
         }
     }
 
     protected Class<?> loadClass(String className) throws ClassNotFoundException {
+        // FIXME Cache the result to avoid the cost of loading classes.
         Class<?> clazz;
         ClassLoader classLoader = this.classLoader;
         if (classLoader == null) {

Original file line number	Diff line number	Diff line change
`@@ -78,11 +78,13 @@ protected Class<?> resolveClass(ObjectStreamClass desc) throws IOException, Clas`
`78`	`78`	`try {`
`79`	`79`	`return loadClass(className);`
`80`	`80`	`} catch (ClassNotFoundException ex) {`
	`81`	`+ // FIXME Cache the result to avoid the cost of ClassNotFoundException.`
`81`	`82`	`return super.resolveClass(desc);`
`82`	`83`	`}`
`83`	`84`	`}`
`84`	`85`
`85`	`86`	`protected Class<?> loadClass(String className) throws ClassNotFoundException {`
	`87`	`+ // FIXME Cache the result to avoid the cost of loading classes.`
`86`	`88`	`Class<?> clazz;`
`87`	`89`	`ClassLoader classLoader = this.classLoader;`
`88`	`90`	`if (classLoader == null) {`