leosutic 2004/09/11 12:07:42 Modified: attributes/api/src/java/org/apache/commons/attributes Util.java Log: Fixed bug that would make Target checking get bypassed. Revision Changes Path 1.2 +11 -10 jakarta-commons/attributes/api/src/java/org/apache/commons/attributes/Util.java Index: Util.java =================================================================== RCS file: /home/cvs/jakarta-commons/attributes/api/src/java/org/apache/commons/attributes/Util.java,v retrieving revision 1.1 retrieving revision 1.2 diff -u -r1.1 -r1.2 --- Util.java 6 Jul 2004 20:41:56 -0000 1.1 +++ Util.java 11 Sep 2004 19:07:42 -0000 1.2 @@ -74,18 +74,18 @@ return sb.toString (); } - public static void checkTarget (int target, Object attribute, String element) { + private static void checkTarget (int target, Object attribute, String element) { Target targetAttr = (Target) Attributes.getAttribute (attribute.getClass (), Target.class); if (targetAttr == null) { return; - } + } if ((targetAttr.getFlags () & target) == 0) { throw new InvalidAttributeTargetError (attribute.getClass ().getName (), element, targetAttr.getFlags ()); } } - public static void checkTarget (int target, Set attributes, String element) { + private static void checkTargets (int target, Collection attributes, String element) { Iterator iter = attributes.iterator (); while (iter.hasNext ()) { checkTarget (target, iter.next (), element); @@ -93,12 +93,13 @@ } public static void validateRepository (Class owningClass, AttributeRepositoryClass repo) { - checkTarget (Target.CLASS, repo.getClassAttributes (), owningClass.getName ()); + checkTargets (Target.CLASS, repo.getClassAttributes (), owningClass.getName ()); Map fieldAttrs = repo.getFieldAttributes (); for (Iterator iter = fieldAttrs.keySet ().iterator(); iter.hasNext ();) { String fieldName = (String) iter.next (); - checkTarget (Target.FIELD, (Collection) fieldAttrs.get (fieldName), owningClass.getName () + "." + fieldName); + + checkTargets (Target.FIELD, (Collection) fieldAttrs.get (fieldName), owningClass.getName () + "." + fieldName); } Map ctorAttrs = repo.getConstructorAttributes (); @@ -109,11 +110,11 @@ for (int i = 0; i < bundle.size (); i++) { switch (i) { case 0: - checkTarget (Target.CONSTRUCTOR, (Collection) bundle.get (0), owningClass.getName () + "." + ctorName); + checkTargets (Target.CONSTRUCTOR, (Collection) bundle.get (0), owningClass.getName () + "." + ctorName); break; default: - checkTarget (Target.CONSTRUCTOR_PARAMETER, (Collection) bundle.get (i), "parameter " + (i) + " of " + owningClass.getName () + ctorName); + checkTargets (Target.CONSTRUCTOR_PARAMETER, (Collection) bundle.get (i), "parameter " + (i) + " of " + owningClass.getName () + ctorName); } } } @@ -126,15 +127,15 @@ for (int i = 0; i < bundle.size (); i++) { switch (i) { case 0: - checkTarget (Target.METHOD, (Collection) bundle.get (0), owningClass.getName () + "." + methodName); + checkTargets (Target.METHOD, (Collection) bundle.get (0), owningClass.getName () + "." + methodName); break; case 1: - checkTarget (Target.RETURN, (Collection) bundle.get (1), "return value of " + owningClass.getName () + "." + methodName); + checkTargets (Target.RETURN, (Collection) bundle.get (1), "return value of " + owningClass.getName () + "." + methodName); break; default: - checkTarget (Target.METHOD_PARAMETER, (Collection) bundle.get (i), "parameter " + (i - 1) + " of " + owningClass.getName () + "." + methodName); + checkTargets (Target.METHOD_PARAMETER, (Collection) bundle.get (i), "parameter " + (i - 1) + " of " + owningClass.getName () + "." + methodName); } } }
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]