On Tue, 20 Oct 2020 23:14:40 GMT, Igor Ignatyev wrote:
> Hi all,
>
> could you please review this small patch?
>
> according to [RFC3659](https://tools.ietf.org/html/rfc3659), time values in
> MLSx response have the following format:
>>time-val = 14DIGIT [ "." 1*DIGIT ]
>>
>>The lea
On Fri, 23 Oct 2020 18:05:07 GMT, Daniel Fuchs wrote:
>> Igor Ignatyev has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> use only filename in assert message
>
> Thanks for adding a test Igor!
> If that passes on all platform then this look
On Fri, 23 Oct 2020 09:14:48 GMT, Daniel Fuchs wrote:
>> The changes in src/java.desktop looks fine.
>
> Changes to `java.logging` and `java.net.http` also look good to me.
Hi Sergey,
I'll give it some testing and sponsor it next week unless someone else steps up.
best regards,
-- daniel
On Fri, 23 Oct 2020 17:50:51 GMT, Igor Ignatyev wrote:
>> Hi all,
>>
>> could you please review this small patch?
>>
>> according to [RFC3659](https://tools.ietf.org/html/rfc3659), time values in
>> MLSx response have the following format:
>>>time-val = 14DIGIT [ "." 1*DIGIT ]
>>>
>>>
> Hi all,
>
> could you please review this small patch?
>
> according to [RFC3659](https://tools.ietf.org/html/rfc3659), time values in
> MLSx response have the following format:
>>time-val = 14DIGIT [ "." 1*DIGIT ]
>>
>>The leading, mandatory, fourteen digits are to be interpreted as
On Thu, 22 Oct 2020 16:04:56 GMT, Igor Ignatyev wrote:
>> Hi Igor, is this testable - and if so shouldn't there be a test?
>> best regards,
>> -- daniel
>
>> Hi Igor, is this testable - and if so shouldn't there be a test?
>> best regards,
>> -- daniel
>
> Hi Daniel,
>
> it's testable and origi
> Hi all,
>
> could you please review this small patch?
>
> according to [RFC3659](https://tools.ietf.org/html/rfc3659), time values in
> MLSx response have the following format:
>>time-val = 14DIGIT [ "." 1*DIGIT ]
>>
>>The leading, mandatory, fourteen digits are to be interpreted as
On Fri, 23 Oct 2020 09:12:15 GMT, Daniel Fuchs wrote:
>> As discussed in https://github.com/openjdk/jdk/pull/510 there is never a
>> reason to explicitly instantiate any instance of `Atomic*` class with its
>> default value, i.e. `new AtomicInteger(0)` could be replaced with `new
>> AtomicInte
On Fri, 23 Oct 2020 08:15:00 GMT, Sergey Bylokhov wrote:
>> As discussed in https://github.com/openjdk/jdk/pull/510 there is never a
>> reason to explicitly instantiate any instance of `Atomic*` class with its
>> default value, i.e. `new AtomicInteger(0)` could be replaced with `new
>> AtomicI
On Thu, 22 Oct 2020 20:46:15 GMT, Сергей Цыпанов
wrote:
> As discussed in https://github.com/openjdk/jdk/pull/510 there is never a
> reason to explicitly instantiate any instance of `Atomic*` class with its
> default value, i.e. `new AtomicInteger(0)` could be replaced with `new
> AtomicInteg
On Thu, 22 Oct 2020 20:46:15 GMT, Сергей Цыпанов
wrote:
> As discussed in https://github.com/openjdk/jdk/pull/510 there is never a
> reason to explicitly instantiate any instance of `Atomic*` class with its
> default value, i.e. `new AtomicInteger(0)` could be replaced with `new
> AtomicInteg
On Thu, 22 Oct 2020 20:46:15 GMT, Сергей Цыпанов
wrote:
> As discussed in https://github.com/openjdk/jdk/pull/510 there is never a
> reason to explicitly instantiate any instance of `Atomic*` class with its
> default value, i.e. `new AtomicInteger(0)` could be replaced with `new
> AtomicInteg
As discussed in https://github.com/openjdk/jdk/pull/510 there is never a reason
to explicitly instantiate any instance of `Atomic*` class with its default
value, i.e. `new AtomicInteger(0)` could be replaced with `new AtomicInteger()`
which is faster:
@State(Scope.Thread)
@OutputTimeUnit(TimeUni
On Thu, 22 Oct 2020 20:46:15 GMT, Сергей Цыпанов
wrote:
> As discussed in https://github.com/openjdk/jdk/pull/510 there is never a
> reason to explicitly instantiate any instance of `Atomic*` class with its
> default value, i.e. `new AtomicInteger(0)` could be replaced with `new
> AtomicInteg
14 matches
Mail list logo
