Eric K Richardson comments

Results 394 comments of


                                            Eric K Richardson

nativelib unsafe Zone alloc has an unfriendly gotcha

That's unfortunate. The issue you mention is not linked if you could fix that. @catap did some good work and forced me to do some good optimizations to the build...

javalib Process needs extensive co-ordinated design & re-work

Isn't this OBE (overcome by events)

javalib Process needs extensive co-ordinated design & re-work

Maybe rename to match the current incarnation?

sn how to package sn project as execute application not jar

You should be able to use sbt native packager. https://www.scala-sbt.org/sbt-native-packager/

Do not use `Double` arithmetics in `Integer.parseInt()`.

I was wondering if you could take a peek and make any recommendations based on this work? https://github.com/scala-native/scala-native/blob/main/javalib/src/main/scala/java/lang/Integer.scala

Possible async issue: native version of scalafmt on macos periodically gets stuck halfway through

This is very possible. I use `scripts/scalafmt` and got rid of the pre-hook. If `scalafmt` stalls I just terminate and re-run. Normally `scalafmt` is fine and the script above downloads...

Possible async issue: native version of scalafmt on macos periodically gets stuck halfway through

Also PR for `scalafmt` latest needs merging https://github.com/scala-native/scala-native/pull/4472

javalib open file max at scale needs study

Just as a data point I ran the soft and hard limits on my mac m1 os Sonoma 14.8. Not sure if that is helpful. ```sh % ulimit -Sn 256...

replace to new wildcard syntax

If the new syntax is supported with the 2.12.x versions that needs supporting, wouldn't it make sense to just make this change for the whole repo?

replace to new wildcard syntax

I guess I read the above incorrectly and I did not know the 2.12 version where the change got added.