Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

openjdk/jdk

dougxc created a comment on a pull request on openjdk/jdk
Thanks for the review. /integrate

View on GitHub

openjdk-notifier[bot] pushed 1 commit to pr/21577 openjdk/jdk
  • Remove duplicate description from docs 7d0a8b7

View on GitHub

mrserb created a review comment on a pull request on openjdk/jdk
You can change the CHECK_EXCEPTION_FATAL macro to something like ``` #define CHECK_EXCEPTION_FATAL(env, message) \ if ((*env)->ExceptionCheck(env)) { \ (*env)->ExceptionDescribe(env...

View on GitHub

mrserb created a review on a pull request on openjdk/jdk

View on GitHub

mpdonova created a review comment on a pull request on openjdk/jdk
can clean this a little bit with ``` return switch(primeSize) { case 2048 -> DiffieHellmanGroup.ffdhe2048; case 3072 -> DiffieHellmanGroup.ffdhe3072; case 4096 -> DiffieHellmanGr...

View on GitHub

mpdonova created a review comment on a pull request on openjdk/jdk
Why not just ``` DHParameterSpec dhParamSpec = new DHParameterSpec(dhGroup.getPrime(), dhGroup.getBase()); System.err.println("Using " + dhGroup.name() + " Diffie-Hellman parameters"); ``` I...

View on GitHub

mpdonova created a review comment on a pull request on openjdk/jdk
don't need fully qualified names here.

View on GitHub

mpdonova created a review on a pull request on openjdk/jdk

View on GitHub

openjdk-notifier[bot] pushed 1 commit to pr/21577 openjdk/jdk
  • Re-ordering and simplifying the checks. Restricting new constraint to TLS. Updating docs. 85c4a16

View on GitHub

rhalade created a review comment on a pull request on openjdk/jdk
Using current directory here may be incorrect. Can this be changed to `Path.of(System.getProperty("test.classes"), ".")`

View on GitHub

rhalade created a review comment on a pull request on openjdk/jdk
Check jdk.test.lib if there is a method you can use to copy files. If not then this method should be added to library.

View on GitHub

rhalade created a review on a pull request on openjdk/jdk

View on GitHub

blperez01 created a review comment on a pull request on openjdk/jdk
Some of these values are constants in the spec and others are commonly used values derived from the constants. For example, `s1PackedLength = (mlDsa_l * (eta + 1) * 256) / 8` because the vector `s1...

View on GitHub

blperez01 created a review on a pull request on openjdk/jdk

View on GitHub

plummercj created a review comment on a pull request on openjdk/jdk
I think you can use getSizet() here.

View on GitHub

plummercj created a review on a pull request on openjdk/jdk

View on GitHub

openjdk-notifier[bot] deleted a branch openjdk/jdk

pr/21628

openjdk-notifier[bot] commented on a commit on openjdk/jdk
<!-- COMMIT COMMENT NOTIFICATION --> ### Review - [openjdk/jdk/21628](https://git.openjdk.org/jdk/pull/21628) ### Issues - [JDK-8341939](https://bugs.openjdk.org/browse/JDK-8341939)

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2433289730) --> @sashamatveev Pushed as commit 9a7a850e2892990cf6755a0ccb19711816ad1b51. :bulb: You may see a message that your pull request was closed with unme...

View on GitHub

openjdk[bot] closed a pull request on openjdk/jdk
8341939: SigningOptionsTest fails without Xcode with command line developer tools after JDK-8341443
- Removed hard check for "Xcode with command line developer tools" when `--mac-sign` is specified and instead we will show information message if `codesign` fails for any reason that possible root ...
openjdk[bot] pushed 1 commit to master openjdk/jdk
  • 8341939: SigningOptionsTest fails without Xcode with command line developer tools after JDK-8341443 Reviewed-by: ase... 9a7a850

View on GitHub

openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- prepush 9a7a850e2892990cf6755a0ccb19711816ad1b51 --> Going to push as commit 9a7a850e2892990cf6755a0ccb19711816ad1b51. Since your change was applied there have been 34 commits pushed to the `m...

View on GitHub

sashamatveev created a comment on a pull request on openjdk/jdk
/integrate

View on GitHub

jyxzwd closed a pull request on openjdk/jdk
8331467: ImageReaderFactory can cause a ClassNotFoundException if the default FileSystemProvider is not the system-default provider
Use the built-in file system provider rather than the custom file system provider. Add "public static FileSystemProvider create" method in DefaultFileSystemProvider which is from java8API to be co...
mpdonova created a review comment on a pull request on openjdk/jdk
It looks like the DiffieHelmanGroup fields are just used to intialize the DHParameterSpec. It might be cleaner to create the DHParameterSpec objects in a static initializer. Also, the DHParamet...

View on GitHub

mpdonova created a review on a pull request on openjdk/jdk

View on GitHub

tkrodriguez created a review on a pull request on openjdk/jdk
Looks good.

View on GitHub

honkar-jdk created a review comment on a pull request on openjdk/jdk
@prrace I might have missed removing this check which was in the original test. The latest update to this test has two run tags but it fails when isFileCacheExpected is set to true. Did you mean...

View on GitHub

honkar-jdk created a review on a pull request on openjdk/jdk

View on GitHub

egahlin created a review on a pull request on openjdk/jdk

View on GitHub

Load more