Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
OlivierNicole created a comment on a pull request on ocsigen/js_of_ocaml
@rickyvetter If you want to run benchmarks on this branch and check that nothing behaves unexpectedly, I expect this to be close to the final version.
OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
No, the behavior is the same, as checked by the tests in `tests-ocaml/lib-effects/`.
OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
In both JS and Wasm, performing under `caml_assume_no_perform` must raise `Effect.Unhandled`, expect after installing a handler, i.e. resuming a stack. On the js_of_ocaml side, it is ensured by hav...
OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
This value is the same as in `caml_callback`. I guess it has been chosen because empirically, it avoids trampolining too often but still avoids overflow. I added a comment saying this.
OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
- CR: revert obsolete change 3109620
OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
Shouldn’t we wait for the new flag to be supported by Dune before documenting it? See https://github.com/ocsigen/js_of_ocaml/pull/1461#issuecomment-2545773411.
OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
- CR: more explicit type for effects backend 6391bd6
OlivierNicole created a comment on a pull request on ocaml/ocaml
In accordance with @goldfirere, I have removed the non-consensual indexing operator. I believe there is a consensus on the core part. If that’s the case, then the next step is to find a reviewer...
OlivierNicole pushed 293 commits to immutable-arrays OlivierNicole/ocaml
- Simplify Parsetree change action and add ppxlib maintainers mention This slighly simplifies the parsetree change act... 49048db
- Mention @ocaml/ppxlib-dev team instead Signed-off-by: Nathan Rebours <[email protected]> 910096c
- Add thread set_name API. e2c3bda
- Document exception when not implemented. 192f9c6
- fix 13501 by reverting 766bd76 1c291a6
- Fix #13501 b1a8e0f
- Rename. 3d4792f
- Split line. 4d5b3ce
- Update otherlibs/systhreads/st_stubs.c Co-authored-by: Antonin Décimo <[email protected]> 791f2b5
- Use AC_CHECK_DECL f5a68b8
- Use both methods. f372510
- noop c79b56e
- Conditionally print stderr a823742
- Update otherlibs/systhreads/st_stubs.c 0c4f429
- Add Runtime_events.EV_EMPTY_MINOR EV_EMPTY_MINOR shows when a domain is trying to empty its minor heap. It may be a ... 301e8d4
- fix escaping of true/false in printing functions (#13263) Both `true` and `false` should not be escaped using the ra... 45e6bc3
- Discuss CAMLnoreturn in memory.h It's only two decades old. Signed-off-by: Andrew Cooper <[email protected]> 6d3ec6e
- Merge pull request #13561 from andyhhp/memory-comment Discuss CAMLnoreturn in memory.h 08e6e63
- review: test record error messages with raw identifiers 60f89d6
- update Changes 2694cdc
- and 273 more ...
OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
never mind, the order of comments is probably mixed in the Github interface.
OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
I’m not sure to understand what needs to be updated.
OlivierNicole pushed 1 commit to immutable-arrays OlivierNicole/ocaml
- Remove .:() indexing operator cd2c725
OlivierNicole created a comment on a pull request on ocsigen/js_of_ocaml
@hhugo Is there something remaining for this to be merged? Do you consider https://github.com/ocaml/dune/pull/11222 to be a necessary prerequisite?
OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
- Remove duplicate effects flag 6bb65e4
OlivierNicole created a review comment on a pull request on ocaml/dune
So dune would support an `--effects=native` flag for wasm_of_ocaml which may one day be supported by wsoo itself?
OlivierNicole pushed 2 commits to jsoo-effects OlivierNicole/dune
- CR: bugs Signed-off-by: Olivier Nicole <[email protected]> 19e12d4
- CR: revert unnecessary moving of code Signed-off-by: Olivier Nicole <[email protected]> 88be4f4