Return-Path: X-Original-To: apmail-incubator-flex-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-flex-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 36E0D9195 for ; Mon, 26 Mar 2012 15:42:13 +0000 (UTC) Received: (qmail 76781 invoked by uid 500); 26 Mar 2012 15:42:12 -0000 Delivered-To: apmail-incubator-flex-dev-archive@incubator.apache.org Received: (qmail 76735 invoked by uid 500); 26 Mar 2012 15:42:12 -0000 Mailing-List: contact flex-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: flex-dev@incubator.apache.org Delivered-To: mailing list flex-dev@incubator.apache.org Received: (qmail 76717 invoked by uid 99); 26 Mar 2012 15:42:12 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 26 Mar 2012 15:42:12 +0000 X-ASF-Spam-Status: No, hits=2.2 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (athena.apache.org: local policy) Received: from [209.85.161.175] (HELO mail-gx0-f175.google.com) (209.85.161.175) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 26 Mar 2012 15:42:04 +0000 Received: by ggcy3 with SMTP id y3so3855112ggc.6 for ; Mon, 26 Mar 2012 08:41:43 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=from:to:subject:date:message-id:mime-version:content-type:x-mailer :thread-index:content-language:x-gm-message-state; bh=g6Moun/IEAy3PTJcsBw2XRI12ZSVGYYall0EE7h7txw=; b=ZGLVL8aC0XUIM0F2XSufewF+r/mMrR6nu4evOFsIRVsAjoWEtJhPqtLokYXDGdn359 a6jh1ty0M2lc0BoIv4KO7ObY/hoxmI446/tkg6LUxGqJ7lZTZKPWtY9JdnY7p+RQc3au dk+v/NbCbkK2/asT3/IhOlLZHHYC3+D57mC7Iq274FAO4SlVAdwphq4123jpWV3zLKgt gwCdgmmewFit67sujIie00sERURT+ZkUcJ3owyyKYEmEFgHo/f/WvwVBxCquiboU80Ez 3ToKdIngTUDOOdUy7ii85gINXApXPgsqkSpqCTLHMZ0KbicinpsbI7LD792SlQJp//mH 6++g== Received: by 10.236.80.105 with SMTP id j69mr22412208yhe.93.1332776503379; Mon, 26 Mar 2012 08:41:43 -0700 (PDT) Received: from TAPST420 (cpe-72-227-143-209.nyc.res.rr.com. [72.227.143.209]) by mx.google.com with ESMTPS id a24sm22206750ana.10.2012.03.26.08.41.38 (version=TLSv1/SSLv3 cipher=OTHER); Mon, 26 Mar 2012 08:41:38 -0700 (PDT) From: "Jeff Tapper" To: Subject: minor Validator improvement Date: Mon, 26 Mar 2012 11:41:37 -0400 Message-ID: <0e6a01cd0b66$eebb1460$cc313d20$@spoon.as> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_NextPart_000_0E6B_01CD0B45.67A9E990" X-Mailer: Microsoft Outlook 14.0 Thread-Index: Ac0LZlCvPicYo9N7SK2vngT7rBK9gA== Content-Language: en-us X-Gm-Message-State: ALoCoQk2fPqZqub7JsnIZyZdiUhHDz2C8U8aQbrd6vyPPeXgyeoVfVisHALmGlZvCzy+WjZunVu2 X-Virus-Checked: Checked by ClamAV on apache.org ------=_NextPart_000_0E6B_01CD0B45.67A9E990 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit In my whiteboard, I just checked in a minor change to the validator class, so the ValidateAll method will return a Vector of ValidationResultEvent instances, rather than a generic array. Since the array cant contain anything other than ValidationResultEvent instances, a vector makes more sense. Additionally, the strong typing allows for code hinting and compile time checking of the results. If anyone has a few minutes, I'd appreciate a quick code review and any feedback. Another change im contemplating is for the argument to be a Vector of IValidator instances, rather than a generic array. Thoughts? ------=_NextPart_000_0E6B_01CD0B45.67A9E990--