RFR: 6594697 - varargs message and Throwable methods for java.util.Logger

[Jim Gish]

I've prepared a new webrev retaining the parameter ordering, but 
removing the isLoggable() refactoring for now.

http://cr.openjdk.java.net/~jgish/Bug6594697-AddLogThrowable/ 
<http://cr.openjdk.java.net/%7Ejgish/Bug6594697-AddLogThrowable/>

Thanks,
     Jim

If this looks o.k. I'll submit to CCC for approval.

On 10/30/2012 02:30 PM, Alan Bateman wrote:
> On 30/10/2012 14:21, Jim Gish wrote:
>> I was one the fence with the parameter ordering and would like 
>> additional feedback on this point.  I started off as you suggested, 
>> but didn't like the fact that the params were separated from the msg 
>> by the Throwable.  I could go either way, but would like to hear from 
>> others on this point.
>>
>> Thanks,
>>     Jim
> I think it make sense to keep the message and its parameters together, 
> and since the parameters have to be at the end then it does mean the 
> Throwable comes before the message. It does mean that there is a bit 
> of inconsistency in the ordering but it might be a bit odd to put the 
> Throwable between the message and its parameters.
>
> I think the changes to use isLoggable should be separated out into 
> their own issue.
>
> -Alan

-- 
Jim Gish | Consulting Member of Technical Staff | +1.781.442.0304
Oracle Java Platform Group | Core Libraries Team
35 Network Drive
Burlington, MA 01803
jim.gish at oracle.com