From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18738 invoked by alias); 13 Feb 2006 07:53:40 -0000 Received: (qmail 18729 invoked by uid 22791); 13 Feb 2006 07:53:39 -0000 X-Spam-Check-By: sourceware.org Received: from adsl-67-116-42-147.dsl.sntc01.pacbell.net (HELO avtrex.com) (67.116.42.147) by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 13 Feb 2006 07:53:32 +0000 Received: from [192.168.7.229] ([192.168.7.229]) by avtrex.com with Microsoft SMTPSVC(6.0.3790.1830); Sun, 12 Feb 2006 23:53:29 -0800 Message-ID: <43F03ACC.4020607@avtrex.com> Date: Mon, 13 Feb 2006 07:53:00 -0000 From: David Daney User-Agent: Mozilla Thunderbird 1.0.7 (Windows/20050923) MIME-Version: 1.0 To: Pedro Izecksohn CC: mauve-discuss@sources.redhat.com, classpath@gnu.org Subject: Re: ResourceBundle Mauve test. References: <20060213053025.1079.qmail@web50102.mail.yahoo.com> In-Reply-To: <20060213053025.1079.qmail@web50102.mail.yahoo.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Mailing-List: contact mauve-discuss-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: mauve-discuss-owner@sourceware.org X-SW-Source: 2006-q1/txt/msg00023.txt.bz2 Pedro Izecksohn wrote: > Proposed fix: To add a (Locale) before both null appearances. As the > .diff attached. > > What do you think about it? > I think it is correct. That is how I was going to recommend fixing it. > > ------------------------------------------------------------------------ > > --- getBundle.java~ 2006-02-13 07:14:14.000000000 +0000 > +++ getBundle.java 2006-02-13 07:14:14.000000000 +0000 > @@ -174,7 +174,7 @@ > > try > { > - ResourceBundle.getBundle (c ("Resource1"), null); > + ResourceBundle.getBundle (c ("Resource1"), (Locale)null); > harness.check (false); > } > catch (NullPointerException ex) > @@ -184,7 +184,7 @@ > > try > { > - ResourceBundle.getBundle ("no such resource", null); > + ResourceBundle.getBundle ("no such resource", (Locale)null); > harness.check (false); > } > catch (NullPointerException ex)