struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Martin Cooper" <martin.coo...@tumbleweed.com>
Subject RE: cvs commit: jakarta-struts/src/share/org/apache/struts/util L abelValueBean.java
Date Wed, 02 Jul 2003 04:15:18 GMT
Just curious - what's the reason for explicitly initialising members to
null, given that that's what happens anyway?

--
Martin Cooper


> -----Original Message-----
> From: dgraham@apache.org [mailto:dgraham@apache.org]
> Sent: Tuesday, July 01, 2003 7:20 PM
> To: jakarta-struts-cvs@apache.org
> Subject: cvs commit: jakarta-struts/src/share/org/apache/struts/util
> LabelValueBean.java
> 
> 
> dgraham     2003/07/01 19:20:28
> 
>   Modified:    src/share/org/apache/struts/util LabelValueBean.java
>   Log:
>   Added default constructor for PR# 18142.
>   
>   Revision  Changes    Path
>   1.3       +14 -11    
> jakarta-struts/src/share/org/apache/struts/util/LabelValueBean.java
>   
>   Index: LabelValueBean.java
>   ===================================================================
>   RCS file: 
> /home/cvs/jakarta-struts/src/share/org/apache/struts/util/Labe
> lValueBean.java,v
>   retrieving revision 1.2
>   retrieving revision 1.3
>   diff -u -r1.2 -r1.3
>   --- LabelValueBean.java	12 Mar 2002 06:30:56 -0000	1.2
>   +++ LabelValueBean.java	2 Jul 2003 02:20:28 -0000	1.3
>   @@ -7,7 +7,7 @@
>     *
>     * The Apache Software License, Version 1.1
>     *
>   - * Copyright (c) 1999-2001 The Apache Software Foundation. 
>  All rights
>   + * Copyright (c) 1999-2003 The Apache Software Foundation. 
>  All rights
>     * reserved.
>     *
>     * Redistribution and use in source and binary forms, with 
> or without
>   @@ -59,13 +59,10 @@
>     *
>     */
>    
>   -
>    package org.apache.struts.util;
>    
>   -
>    import java.io.Serializable;
>    
>   -
>    /**
>     * A simple JavaBean to represent label-value pairs. This 
> is most commonly used
>     * when constructing user interface elements which have a 
> label to be displayed
>   @@ -76,7 +73,6 @@
>     * @author  Martin F N Cooper
>     * @version $Revision$ $Date$
>     */
>   -
>    public class LabelValueBean implements Serializable {
>    
>    
>   @@ -84,6 +80,13 @@
>    
>    
>        /**
>   +     * Default constructor.
>   +     */
>   +    public LabelValueBean() {
>   +        this(null, null);
>   +    }
>   +
>   +    /**
>         * Construct an instance with the supplied property values.
>         *
>         * @param label The label to be displayed to the user.
>   @@ -101,7 +104,7 @@
>        /**
>         * The property which supplies the option label 
> visible to the end user.
>         */
>   -    private String label;
>   +    private String label = null;
>    
>        public String getLabel() {
>            return this.label;
>   @@ -115,7 +118,7 @@
>        /**
>         * The property which supplies the value returned to 
> the server.
>         */
>   -    private String value;
>   +    private String value = null;
>    
>        public String getValue() {
>            return this.value;
>   
>   
>   
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: struts-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: struts-dev-help@jakarta.apache.org
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: struts-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: struts-dev-help@jakarta.apache.org


Mime
View raw message