[RFC][icedtea-web] Use description param when creating a LaunchException
Deepak Bhole
dbhole at redhat.com
Thu Feb 2 13:06:14 PST 2012
* Danesh Dadachanji <ddadacha at redhat.com> [2012-02-02 15:06]:
> On 02/02/12 02:00 PM, Danesh Dadachanji wrote:
> >
> >Pushed.
> >
> >http://icedtea.classpath.org/hg/release/icedtea-web-1.2/rev/466a2440ed59
> >
>
> Bah sorry, I forgot to test the case where the locale's value is an
> empty string. I did not realize it returns null, I'll have to push
> another changeset. =(
>
> Patch attached handles the null case.
>
OK for HEAD and 1.2.
Deepak
> +2012-02-02 Danesh Dadachanji <ddadacha at redhat.com>
> +
> + * netx/net/sourceforge/jnlp/LaunchException.java: Fix message
> + to handle null description.
> +
>
> Regards,
> Danesh
> diff --git a/netx/net/sourceforge/jnlp/LaunchException.java b/netx/net/sourceforge/jnlp/LaunchException.java
> --- a/netx/net/sourceforge/jnlp/LaunchException.java
> +++ b/netx/net/sourceforge/jnlp/LaunchException.java
> @@ -52,7 +52,8 @@ public class LaunchException extends Exc
> * Creates a LaunchException without detail message.
> */
> public LaunchException(JNLPFile file, Exception cause, String severity, String category, String summary, String description) {
> - super(severity + ": " + category + ": " + summary + " " + description);
> + super(severity + ": " + category + ": " + summary + " "
> + + description == null ? "" : description);
>
> this.file = file;
> this.category = category;
More information about the distro-pkg-dev
mailing list