[PATCH 1/1] Get rid of synchronization in java.util.logging.LogRecord constructor (original) (raw)

David M. Lloyd david.lloyd at redhat.com
Fri Mar 13 00:11:38 UTC 2009


On 03/12/2009 07:01 PM, Dalibor Topic wrote:

David M. Lloyd wrote:

On 03/12/2009 05:41 PM, Mark Reinhold wrote:

You might want to have a look at the new contribution process [1]. Using that will increase the probability that someone will evaluate your patch sooner rather than later.

- Mark [1] http://openjdk.java.net/contribute Consider my post to be step 2 of that process. The patch itself is probably the most succinct way of expressing the proposed change (it only took a minute or two to write in any case). That's understandable, but it's a lot easier to find interesting patches to review in a bug tracker, then to search for them among mailing list threads. [...]

OK, that's reasonable. Perhaps Step 2 should be switched with Step 3 then on the "contribute" page to make it match up then?

It seems that the change would break serialization, by changing the type of a serialized field (in both classes) away from a primitive one. See http://java.sun.com/javase/6/docs/platform/serialization/spec/version.html for details.

The fields are static so it shouldn't matter.



More information about the core-libs-dev mailing list