Mailing-List: contact commons-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Jakarta Commons Developers List" <commons-dev@jakarta.apache.org>
Reply-To: <epugh@upstate.com>
From: "Eric Pugh" <epugh@upstate.com>
To: "'Jakarta Commons Developers List'" <commons-dev@jakarta.apache.org>
Subject: RE: [configuration] Change getVector to getList
Date: Wed, 24 Dec 2003 15:56:06 +0100
Message-ID: <008701c3ca2e$0f699570$0100a8c0@IQUITOS>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_NextPart_000_0088_01C3CA36.712DFD70"
In-Reply-To: <3FE8710E.9050303@cross-systems.com>
Importance: Normal

------=_NextPart_000_0088_01C3CA36.712DFD70
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 7bit

[configuration] Change getVector to getListOkay, I applied your patch.  The
other patch for a ConfigurationException we should redo then.  I was
thinking about ConfigurationException..   We bascially never expect it to
happen..  Should it be a RuntimeException?  Otherwise, everywhere you do
configuratoin.getString("myval") you have to check for the darn exception...

Eric
  -----Original Message-----
  From: Emmanuel Bourg [mailto:e.bourg@cross-systems.com]
  Sent: Tuesday, December 23, 2003 5:45 PM
  To: Jakarta Commons Developers List
  Subject: [configuration] Change getVector to getList


  This change was already discussed briefly in september, with the move
  out of the sandbox and the near 1.0 release I'd like to push this patch
  to clean the API before the first public release. The patch replaces the
  getVector methods in the Configuration interface by equivalent getList
  methods. All Vectors in the implementation have been converted to
  List/ArrayLists. The only Vector left is in the
  ConfigurationConverter.getExtendedProperties method since the
  ExtendedProperties class doesn't seem to accept Lists. The documentation
  and the test cases have been modified accordingly.

  Emmanuel Bourg


------=_NextPart_000_0088_01C3CA36.712DFD70--