AfflictionNetworks/narya
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Only issue this warning if we actually have one non-null and one null value.

Browse Source 
git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@5461 542714f4-19e9-0310-aa3c-eee0fc999fb1
This commit is contained in:
Michael Bayne
2008-10-24 21:43:23 +00:00
parent 3a457a2479
commit 8970582e6d
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/java/com/threerings/io/FieldMarshaller.java
+2 -1
View File
@@ -75,7 +75,8 @@ public abstract class FieldMarshaller
} }
if (reader != null && writer != null) { if (reader != null && writer != null) {
return new MethodFieldMarshaller(reader, writer); return new MethodFieldMarshaller(reader, writer);
} else if (reader == null || writer == null) { }
if ((reader == null && writer != null) || (writer == null && reader != null)) {
log.warning("Class contains one but not both custom field reader and writer", log.warning("Class contains one but not both custom field reader and writer",
"class", field.getDeclaringClass().getName(), "field", field.getName(), "class", field.getDeclaringClass().getName(), "field", field.getName(),
"reader", reader, "writer", writer); "reader", reader, "writer", writer);