Return-Path: Delivered-To: apmail-cxf-dev-archive@www.apache.org Received: (qmail 1220 invoked from network); 10 Oct 2008 11:58:40 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 10 Oct 2008 11:58:40 -0000 Received: (qmail 7478 invoked by uid 500); 10 Oct 2008 11:58:38 -0000 Delivered-To: apmail-cxf-dev-archive@cxf.apache.org Received: (qmail 7441 invoked by uid 500); 10 Oct 2008 11:58:38 -0000 Mailing-List: contact dev-help@cxf.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cxf.apache.org Delivered-To: mailing list dev@cxf.apache.org Received: (qmail 7430 invoked by uid 99); 10 Oct 2008 11:58:38 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 10 Oct 2008 04:58:38 -0700 X-ASF-Spam-Status: No, hits=0.1 required=10.0 tests=DNS_FROM_SECURITYSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of bimargulies@gmail.com designates 72.14.220.155 as permitted sender) Received: from [72.14.220.155] (HELO fg-out-1718.google.com) (72.14.220.155) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 10 Oct 2008 11:57:33 +0000 Received: by fg-out-1718.google.com with SMTP id l26so317552fgb.10 for ; Fri, 10 Oct 2008 04:57:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to :subject:in-reply-to:mime-version:content-type :content-transfer-encoding:content-disposition:references; bh=4aKkv9tAA6hCKeaJ96wW9hilLc7acFMrwmpZquczWRo=; b=WCW4fZ+2gdnrUYdwmHPE8UaxEg5pIL6qE2mRVhedfbYxxgJ7xCHR0t4zJ6rcY0SzN9 4pSu1IiepJXfo5oL361MawjaMNvZJY41Ri44FreRErYlXyuaUk67XCdQoyReIyLeSYLw LOEhQn3wkyTpVQgbI3KZBatmIZXME2bPa6Em4= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=G7IBinxS5FZ88/YDOh4tiqVRtIaqW9+0zmzEQuBYAdM28ZFkPaBOtOarJo52CbJTgO +oygs7OjLx8L8JM1Z5u4pao+HPQf28lx2rTJQCmkgike4pR0Admk75+2IIM6G9u5zhwn fDss5oZMf4gv9qo70YKrtwBOfCc0HyZuuut8g= Received: by 10.181.52.18 with SMTP id e18mr1410093bkk.89.1223639872145; Fri, 10 Oct 2008 04:57:52 -0700 (PDT) Received: by 10.180.253.10 with HTTP; Fri, 10 Oct 2008 04:57:52 -0700 (PDT) Message-ID: <61b5d9410810100457u6052f001u6a31d1f91352ff45@mail.gmail.com> Date: Fri, 10 Oct 2008 07:57:52 -0400 From: "Benson Margulies" To: dev@cxf.apache.org Subject: Re: How to handle a configuration problem generally in CXF code In-Reply-To: <48ED3FA7.3000705@die-schneider.net> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <48EA902F.2050805@die-schneider.net> <200810071617.45941.dkulp@apache.org> <48EBD447.3010809@die-schneider.net> <61b5d9410810071503t1769dfbai6953773a203603fa@mail.gmail.com> <48ED3FA7.3000705@die-schneider.net> X-Virus-Checked: Checked by ClamAV on apache.org Are you sure that none of the tools are using the Message class directly? The message class still seems to be good modularity to me. I'm +1 on the new constructors, all the same. On Wed, Oct 8, 2008 at 7:17 PM, Christian Schneider wrote: > I would propose to add the following to UncheckedException and corresponding > "super" calls to all exceptions. > > public UncheckedException(String code, ResourceBundle bundle, > Object...params) { > this(new Message(code, bundle, params)); > } > public UncheckedException(String code, Throwable t, ResourceBundle > bundle, Object...params) { > this(new Message(code, bundle, params), t); > } > > To use these you could do the following at the start of the class: > static final Logger LOG = LogUtils.getL7dLogger(JMSConfigFeature.class); > static final ResourceBundle BUNDLE = LOG.getResourceBundle(); > > and then: > throw new ConfigurationException("JMSCONFIGFEATURE_ONLY_JMS", BUNDLE); > > I have checked where the i18n Message is used and it seems only to be used > for exceptions. So would it perhaps make sense to deprecate it and always > create exceptions like above? > Was there a special reason for the introduction of i18n Message? > Any opinions? > > Greetings > > Christian > > > Benson Margulies schrieb: >> >> I'm +1 to the constructor. I find the new-ing of messages to be a pain. >> >> On Tue, Oct 7, 2008 at 5:27 PM, Christian Schneider >> > >>> >>> wrote: >>> >> >> > > > -- > > Christian Schneider > --- > http://www.liquid-reality.de > >