Review request for 7145913 CachedRowSetSwriter.insertNewRow() throws SQLException
Lance Andersen - Oracle
Lance.Andersen at oracle.com
Wed Jun 13 17:01:48 UTC 2012
Hi Remi,
Thank you for the suggestion. Over the years, I have gotten different views on whether to have multiple return points vs just one.
Is there a specific style preference that should be used going forward? At this time, I would prefer to not make another change and if the consensus going forward is that multiple return points is OK, I will make the change in my next push of this code.
Best
Lance
On Jun 13, 2012, at 12:49 PM, Rémi Forax wrote:
> On 06/13/2012 06:18 PM, Joe Wang wrote:
>> Hi Lance,
>>
>> The changes look good to me.
>>
>> Joe
>
> Hi Lance,
> just a minor comment, in isPKNameValid, you don't need the boolean isValid because
> you can return true instead of using break and return false at the end.
>
> cheers,
> Rémi
>
>>
>> On 6/13/2012 4:09 AM, Paul Sandoz wrote:
>>> On Jun 13, 2012, at 1:04 PM, Lance Andersen - Oracle wrote:
>>>> Hi Paul,
>>>>
>>>> Thank you for taking the time to review the code.
>>>>
>>>>
>>>> I made the change you suggested below
>>>>
>>>> http://cr.openjdk.java.net/~lancea/7145913/webrev.02
>>>>
>>>> Let me know if you are good with the change and I will get this puppy put back.
>>>>
>>> Looks good!
>>>
>>> Paul.
>
>
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com
More information about the core-libs-dev
mailing list