Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
With the update to `caml_continuation_use_and_update_handler_noexc`, `caml_fiber_stack.h === handlers` seems to always hold.
hhugo pushed 18 commits to ocaml-trunk ocsigen/js_of_ocaml
- Misc: fix doc generation 0c19536
- Misc: remove unused files e0ec46e
- Build: dune alias for js and wasm (#1760) c0f9035
- Runtime: no longer use arguments (#1740) * Runtime: no longer use arguments * no spread to build array * adapt... 145e4d2
- Doc: document mininum requirements (#1762) 425df2e
- Doc: document emcascript features we use (#1763) 25edc11
- CI: use latest binaryen (#1767) Signed-off-by: Sora Morimoto <[email protected]> 4782b7b
- Test: global flow bug c0e3651
- Global flow analysis: add some debugging code Make sure we have reached a fixed point. 18b1cbc
- Global flow analysis: bug fix We were not correctly propagating the information that a block is possibly mutated. 71702fa
- Modernize a bit Dom_html Mostly, removed opdef for properties that are widely available. eff5bac
- CI updates (adapt Jane Street packages) dc6c457
- dune: update menhir extension to 3.0 Signed-off-by: Sora Morimoto <[email protected]> 950b56a
- Runtime: fix memory leak wrt channels 7309673
- typo a79fb93
- CI: fix effects with wasmoo 0b995da
- Misc: promote to 5.3 6ccb19b
- fix 23e65ab
hhugo created a comment on a pull request on ocsigen/js_of_ocaml
Do you already have a `dune` PR to reflect changes made in this PR ?
hhugo pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
- Don't set effects_backend in _cmd_arg_ modules a66e2a9
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
The comment is stale. separate compilation now works with `--enable use-js-srtring` or `--enable effects`.
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
the effects argument could take a backend instead of a boolean.
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
Please use a `match .. with` here
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
Maybe we could update `macro.ml` to handle this case. We could have a `CONFIG` macro.
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
I would prefer an exhaustive `match .. with` here so that we reconsider the need for this every time one add a backend
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
match on both target and effect
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
a `match .. with` would be clearer I think
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
You can match the inner of the option directly, just name the backend on line 102.
hhugo created a review comment on a pull request on ocsigen/js_of_ocaml
Shouldn't this be called effects_cps ?