commons-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-commons/beanutils/src/java/org/apache/commons/beanutils DynaProperty.java
Date Sun, 24 Mar 2002 19:20:42 GMT
I don't understand the changes you made here and in MutableDynaClass.java.
You turned perfectly valid comments into incorrect English constructs.

In this checkin, you changed "i.e.", which stands for "id est", into "ie",
which is not a word. In the subsequent checkin, you changed a question and
answer into a rambling non-English construct.

Is there really a good reason for these changes? Is there a reason we can't
use valid English in our Javadoc comments?

--
Martin Cooper


----- Original Message -----
From: <dion@apache.org>
To: <jakarta-commons-cvs@apache.org>
Sent: Sunday, March 24, 2002 1:15 AM
Subject: cvs commit:
jakarta-commons/beanutils/src/java/org/apache/commons/beanutils
DynaProperty.java


> dion        02/03/24 01:15:03
>
>   Modified:    beanutils/src/java/org/apache/commons/beanutils
>                         DynaProperty.java
>   Log:
>   Fixed javadoc breakiterator issues
>
>   Revision  Changes    Path
>   1.3       +6 -6
jakarta-commons/beanutils/src/java/org/apache/commons/beanutils/DynaProperty
.java
>
>   Index: DynaProperty.java
>   ===================================================================
>   RCS file:
/home/cvs/jakarta-commons/beanutils/src/java/org/apache/commons/beanutils/Dy
naProperty.java,v
>   retrieving revision 1.2
>   retrieving revision 1.3
>   diff -u -r1.2 -r1.3
>   --- DynaProperty.java 9 Jan 2002 19:27:30 -0000 1.2
>   +++ DynaProperty.java 24 Mar 2002 09:15:03 -0000 1.3
>   @@ -1,7 +1,7 @@
>    /*
>   - * $Header:
/home/cvs/jakarta-commons/beanutils/src/java/org/apache/commons/beanutils/Dy
naProperty.java,v 1.2 2002/01/09 19:27:30 craigmcc Exp $
>   - * $Revision: 1.2 $
>   - * $Date: 2002/01/09 19:27:30 $
>   + * $Header:
/home/cvs/jakarta-commons/beanutils/src/java/org/apache/commons/beanutils/Dy
naProperty.java,v 1.3 2002/03/24 09:15:03 dion Exp $
>   + * $Revision: 1.3 $
>   + * $Date: 2002/03/24 09:15:03 $
>     *
>     * ====================================================================
>     *
>   @@ -71,7 +71,7 @@
>     * <p>The metadata describing an individual property of a DynaBean.</p>
>     *
>     * @author Craig R. McClanahan
>   - * @version $Revision: 1.2 $ $Date: 2002/01/09 19:27:30 $
>   + * @version $Revision: 1.3 $ $Date: 2002/03/24 09:15:03 $
>     */
>
>    public class DynaProperty {
>   @@ -135,7 +135,7 @@
>
>
>        /**
>   -     * Does this property represent an indexed value (i.e. an array or
List)?
>   +     * Does this property represent an indexed value (ie an array or
List)?
>         */
>        public boolean isIndexed() {
>
>   @@ -153,7 +153,7 @@
>
>
>        /**
>   -     * Does this property represent a mapped value (i.e. a Map)?
>   +     * Does this property represent a mapped value (ie a Map)?
>         */
>        public boolean isMapped() {
>
>
>
>
>
> --
> To unsubscribe, e-mail:
<mailto:commons-dev-unsubscribe@jakarta.apache.org>
> For additional commands, e-mail:
<mailto:commons-dev-help@jakarta.apache.org>
>


--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>


Mime
View raw message