git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Roger C. Soares" <rogersoares@intelinet.com.br>
To: Robin Rosenberg <robin.rosenberg@dewire.com>
Cc: "Shawn O. Pearce" <spearce@spearce.org>,
	git@vger.kernel.org, Dave Watson <dwatson@mimvista.com>
Subject: Re: [EGIT PATCH 2/2] Use Integer.valueOf instead of new Integer
Date: Fri, 08 Feb 2008 00:52:45 -0200	[thread overview]
Message-ID: <47ABC3FD.1000803@intelinet.com.br> (raw)
In-Reply-To: <200802062115.38212.robin.rosenberg@dewire.com>


Robin Rosenberg escreveu:
> onsdagen den 6 februari 2008 skrev Shawn O. Pearce:
>   
>> Use Integer.valueOf(int).  Post Java 5 implementations of the J2SE
>> are required to cache values between -128 and 127 (inclusive).
>>
>> Actually, when the Java compiler autoboxes values it does so through
>> these static valueOf methods, which were mostly introduced as part
>> of the Java 5 API updates.  For small common values its cached and
>> will thus avoid garbage generation, for less common values it goes
>> back to allocating the object.
>>     
>
> Thank you for this information.
>
> -- robin
Yep, this is very good to know. Thanks from me too :)
This patches series looks fine to me.

[]s,
Roger.

  reply	other threads:[~2008-02-08  2:52 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-05  0:15 [EGIT PATCH Series] Cleanups and javadocs Robin Rosenberg
2008-02-05  0:15 ` [EGIT PATCH 1/6] Comment some empty blocks that should be empty Robin Rosenberg
2008-02-05  0:15   ` [EGIT PATCH 2/6] Cleanup unboxing/boxing Robin Rosenberg
2008-02-05  0:15     ` [EGIT PATCH 3/6] Do not require javadoc in egit.core test project Robin Rosenberg
2008-02-05  0:15       ` [EGIT PATCH 4/6] Javadoc update for Egit Robin Rosenberg
2008-02-05  0:15         ` [EGIT PATCH 5/6] Drop unused GitBlobStorage class Robin Rosenberg
2008-02-05  0:15           ` [EGIT PATCH 6/6] Tighten requirements on javadoc Robin Rosenberg
2008-02-06  7:06     ` [EGIT PATCH 2/6] Cleanup unboxing/boxing Shawn O. Pearce
2008-02-06 20:14       ` [EGIT PATCH 1/2] Use Boolean.valueOf instead of new Boolean Robin Rosenberg
2008-02-06 20:15       ` [EGIT PATCH 2/2] Use Integer.valueOf instead of new Integer Robin Rosenberg
2008-02-08  2:52         ` Roger C. Soares [this message]
2008-02-05 15:09 ` [EGIT PATCH Series] Cleanups and javadocs Roger C. Soares
2008-02-05 21:47   ` Robin Rosenberg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=47ABC3FD.1000803@intelinet.com.br \
    --to=rogersoares@intelinet.com.br \
    --cc=dwatson@mimvista.com \
    --cc=git@vger.kernel.org \
    --cc=robin.rosenberg@dewire.com \
    --cc=spearce@spearce.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).