[PATCH 1/1] Get rid of synchronization in java.util.logging.LogRecord constructor
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.
- DML
More information about the core-libs-dev
mailing list