RFR Re: [PATCH] 8178117: Add public state constructors for Int/Long/DoubleSummaryStatistics

[Paul Sandoz]

> On 31 Oct 2017, at 11:24, Brian Goetz <brian.goetz at Oracle.COM> wrote:
> 
> While I agree that the overflow-detecting constraint on min/max in the early version was too aggressive, you could reasonably choose to enforce the constraint that if count == 0, then so should sum, min, and max.
> 

I’m on the fence on this one. Joe is going to follow up with another issue for DoubleSummaryStatistics adjusting to a varargs constructor and and may consider doing this at the same time.

Paul.

> On 10/30/2017 6:49 PM, Paul Sandoz wrote:
>> Hi Chris,
>> 
>> After some hiatus here is an updated webrev, i made some tweaks to the JavaDoc:
>> 
>>   http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8178117-summary-constructors/webrev/ <http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8178117-summary-constructors/webrev/>
>> 
>> And the CSR:
>> 
>>   https://bugs.openjdk.java.net/browse/JDK-8190381 <https://bugs.openjdk.java.net/browse/JDK-8190381>
>> 
>> The support for a double sum of consisting of an array of double is going beyond my complexity budget for this feature. If that is deemed important later on we can add another constructor.
>> 
>> Paul.
>>