Re: svn commit: r605850 - in /cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/ apache/cayenne: ./ access/ access/jdbc/ access/types/ dba/ dba/openbase/ dba/oracle/ dba/sqlite/ dba/sqlserver/ exp/parser/ query/ reflect/ util/ wocompat/

From: Tore Halset (halse..vv.ntnu.no)
Date: Fri Dec 21 2007 - 02:03:34 EST

  • Next message: Tore Halset: "performQuery generics"

    On Dec 20, 2007, at 22:49 , Aristedes Maniatis wrote:

    > I agree that it can be confusing in some cases, but it isn't like we
    > can just switch it off. The places where it is confusing is where it
    > happens without you realising it is happening.

    Thanks, I have realized that now. It is not possible to get it turned
    off by not using it as it can pop up everywhere. So I rest my "-1" and
    are now a "+0" on this issue :)

    > The places where it makes the code easier to read (IMO) are when you
    > use it intentionally and might look like this:
    >
    > static public Integer SOME_CONSTANT = 5;
    >
    > But really, I think it a pretty minor issue.

    Not that big win for me with a big monitor :)

    > I wasn't aware of the speed enhancement of Integer.valueOf(int) and
    > that is pretty nice as long as we are sure those particular Integers
    > are immutable.

    Integer is immutable. Autoboxing of ints probably uses
    Integer.valueOf(int).

      - Tore.



    This archive was generated by hypermail 2.0.0 : Fri Dec 21 2007 - 02:04:19 EST