On Thu, 13 Jul 2023 11:31:40 GMT, Ferenc Rakoczi wrote:
>> Valerie Peng has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Address review feedbacks, e.g. Removed RSAPadding.Output and use byte[] as
>> before.
>
> src/java.base/share/classe
On Fri, 14 Jul 2023 20:08:54 GMT, Sean Mullan wrote:
>> Valerie Peng has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Address review feedbacks, e.g. Removed RSAPadding.Output and use byte[] as
>> before.
>
> src/java.base/share/classes/s
On Mon, 17 Jul 2023 18:39:37 GMT, Daniel Jeliński wrote:
> I think `oid.hashCode() + Arrays.hashCode(nameValue)` would have been good
> enough, but deepHashCode is also acceptable.
Fixed as you suggested, in 3139043370f.
-
PR Review Comment: https://git.openjdk.org/jdk/pull/14738#
> Please review this PR to use modern APIs and language features to simplify
> `equals` and `hashCode` in security area.
>
> I understand that security area is sensitive and a non-expert, such as
> myself, should tread carefully; so below are my notes to assist the review.
>
> * Unlike `hashCod
On Tue, 7 Mar 2023 01:40:48 GMT, Cesar Soares Lucas wrote:
> Can I please get reviews for this PR?
>
> The most common and frequent use of NonEscaping Phis merging object
> allocations is for debugging information. The two graphs below show numbers
> for Renaissance and DaCapo benchmarks - si
On Thu, 13 Jul 2023 04:25:19 GMT, Xue-Lei Andrew Fan wrote:
>> Valerie Peng has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Address review feedbacks, e.g. Removed RSAPadding.Output and use byte[] as
>> before.
>
> src/java.base/share/cl
On Tue, 20 Jun 2023 16:44:28 GMT, Vladimir Ivanov wrote:
>> Thank you once more for the comments @iwanowww . I’ll address them asap.
>>
>> Can I ask what requirements are there for a product flag?
>
>> Can I ask what requirements are there for a product flag?
>
> Product flags are treated as p
On Mon, 17 Jul 2023 17:45:56 GMT, Matthew Donovan wrote:
> In this PR, i raised the client timeout from 5 to 60 seconds. I considered
> refactoring the SimpleOSCPServer class a little but ultimately, the client
> needs to just wait until the server is ready or a time-out is reached.
> Alternat
On Mon, 17 Jul 2023 18:13:38 GMT, Pavel Rappo wrote:
>> Thanks for noticing this difference!
>>
>> Yes, the initial value would've been exponentiated (^ or ** in your
>> notation). However, I note that the original code computed this:
>>
>> if nameValue.length > 0
>>
>> (37^nameValue.leng
On Mon, 17 Jul 2023 12:23:17 GMT, Pavel Rappo wrote:
>> The original code calculated `(37 **
>> nameValue.length)*(37+oid.hashCode)+Arrays.hashCode`; since you already
>> dropped the multiplication, you can also drop the addition.
>
> Thanks for noticing this difference!
>
> Yes, the initial v
In this PR, i raised the client timeout from 5 to 60 seconds. I considered
refactoring the SimpleOSCPServer class a little but ultimately, the client
needs to just wait until the server is ready or a time-out is reached.
Alternately, we can wait indefinitely and let the test time out but that
r
On Tue, 13 Jun 2023 20:36:28 GMT, Anthony Scarpino
wrote:
> Hi,
>
> I need a code review for moving the contents of the jdk.crypto.ec module into
> java.base. This moves the SunEC JCE Provider (Elliptic Curve) into
> java.base. EC has always been separate from the base module/pkg because of
On Mon, 17 Jul 2023 11:02:46 GMT, Daniel Jeliński wrote:
>> You think it was a typo (+ instead of *) in the original code?
>
> The original code calculated `(37 **
> nameValue.length)*(37+oid.hashCode)+Arrays.hashCode`; since you already
> dropped the multiplication, you can also drop the addit
On Mon, 1 May 2023 17:39:02 GMT, Matthew Donovan wrote:
> In this PR, I added methods to the TransportContext class to synchronize
> access to the handshakeContext field. I also updated locations in the code
> that rely on the handshakeContext field to not be null to use the
> synchronized met
On Mon, 17 Jul 2023 10:59:42 GMT, Pavel Rappo wrote:
>> src/java.base/share/classes/sun/security/x509/OtherName.java line 212:
>>
>>> 210: public int hashCode() {
>>> 211: if (myhash == -1) {
>>> 212: myhash = 37 + oid.hashCode() + Arrays.hashCode(nameValue);
>>
>> I thi
On Thu, 13 Jul 2023 22:57:49 GMT, Pavel Rappo wrote:
>> Please review this PR to use modern APIs and language features to simplify
>> `equals` and `hashCode` in security area.
>>
>> I understand that security area is sensitive and a non-expert, such as
>> myself, should tread carefully; so bel
On Mon, 17 Jul 2023 10:15:18 GMT, Daniel Jeliński wrote:
>> Pavel Rappo has updated the pull request incrementally with five additional
>> commits since the last revision:
>>
>> - Feedback: avoid intermediate assignments
>> - More previously missed cases
>> - Fix: log hashCode as an unsigned
17 matches
Mail list logo
