This is the mail archive of the mauve-discuss@sourceware.org mailing list for the Mauve project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: ResourceBundle Mauve test.

From: David Daney <ddaney at avtrex dot com>
To: Pedro Izecksohn <izecksohn at yahoo dot com>
Cc: mauve-discuss at sources dot redhat dot com, classpath at gnu dot org
Date: Sun, 12 Feb 2006 23:52:44 -0800
Subject: Re: ResourceBundle Mauve test.
References: <20060213053025.1079.qmail@web50102.mail.yahoo.com>

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)

References:
- Re: ResourceBundle Mauve test.
  - From: Pedro Izecksohn

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]