Hmm don't throw the ArgumentError inside our own catch, even though

we should have re-thrown it. git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@5951 542714f4-19e9-0310-aa3c-eee0fc999fb1
2009-09-11 16:49:37 +00:00
parent 4fe12a043d
commit 32db838be2
1 changed files with 5 additions and 4 deletions
@@ -156,11 +156,9 @@ public class ObjectMarshaller
    public static function validateValue (value :Object) :void
        // throws ArgumentError
    {
+        var s :String;
        try {
-            var s :String = getValidationError(value);
-            if (s != null) {
-                throw new ArgumentError(s);
-            }
+            s = getValidationError(value);

        } catch (e :Error) {
            switch (e.errorID) {
@@ -170,6 +168,9 @@ public class ObjectMarshaller
                throw e; // re-throw
            }
        }
+        if (s != null) {
+            throw new ArgumentError(s);
+        }
    }

    /**