Manual: new section for warnings and extended documentation for warning 57#471
Merged
gasche merged 2 commits intoocaml:warnings-documentationfrom Feb 12, 2016
Merged
Conversation
Member
|
I suspect that the idea of mentioning the documentation chapter in the warning message might not please some. Would you mind splitting it as a separate commit? (That would be an opportunity to squash the minor fix commit.) I would eventually like to expand on the description of this warning, in particular we need to discuss how users should fix their code when they have it. But I'll merge the PR with the current content, we can iterate later. |
3323470 to
3856487
Compare
Member
Author
|
Ok, I have split the PR in two to separate the manual and in-warning documentations. |
gasche
added a commit
that referenced
this pull request
Feb 12, 2016
Manual: new section for warnings and extended documentation for warning 57
EduardoRFS
pushed a commit
to esy-ocaml/ocaml
that referenced
this pull request
May 17, 2021
This commit introduce an atfork hook in the runtime, as a follow up to the issue encountered in MPR ocaml#455 and MPR ocaml#471. To handle all the specific fork scenarios allowed in Multicore, a hook is required in order to let specific "alterations" to the runtime do their job: by default, on a single domain program, only the domain lock needs to be reset. When systhreads are in use, more cleaning up duty is required. As a result, the usage of pthreads_atfork is also removed, opting to rely in both case on manually calling the runtime hook when required.
Keryan-dev
added a commit
to Keryan-dev/ocaml
that referenced
this pull request
Jun 17, 2021
Gbury
pushed a commit
to Gbury/ocaml
that referenced
this pull request
Jun 22, 2021
sadiqj
pushed a commit
to sadiqj/ocaml
that referenced
this pull request
Jan 10, 2022
This commit introduce an atfork hook in the runtime, as a follow up to the issue encountered in MPR ocaml#455 and MPR ocaml#471. To handle all the specific fork scenarios allowed in Multicore, a hook is required in order to let specific "alterations" to the runtime do their job: by default, on a single domain program, only the domain lock needs to be reset. When systhreads are in use, more cleaning up duty is required. As a result, the usage of pthreads_atfork is also removed, opting to rely in both case on manually calling the runtime hook when required.
stedolan
pushed a commit
to stedolan/ocaml
that referenced
this pull request
Mar 22, 2022
stedolan
pushed a commit
to stedolan/ocaml
that referenced
this pull request
May 24, 2022
stedolan
added a commit
to stedolan/ocaml
that referenced
this pull request
May 24, 2022
64235a3 flambda-backend: Change Float.nan from sNaN to qNaN (ocaml#466) 14a8e27 flambda-backend: Track GC work for all managed bigarray allocations (upstream 11022) (ocaml#569) c3cda96 flambda-backend: Add two new methods to targetint for dwarf (ocaml#560) e6f1fed flambda-backend: Handle arithmetic overflow in select_addr (ocaml#570) dab7209 flambda-backend: Add Target_system to ocaml/utils (ocaml#542) 82d5044 flambda-backend: Enhance numbers.ml with more primitive types (ocaml#544) 216be99 flambda-backend: Fix flambda_o3 and flambda_oclassic attributes (ocaml#536) 4b56e07 flambda-backend: Test naked pointer root handling (ocaml#550) 40d69ce flambda-backend: Stop local function optimisation from moving code into function bodies; opaque_identity fixes for class compilation (ocaml#537) f08ae58 flambda-backend: Implemented inlining history and use it inside inlining reports (ocaml#365) ac496bf flambda-backend: Disable the local keyword in typing (ocaml#540) 7d46712 flambda-backend: Bugfix for Typedtree generation of arrow types (ocaml#539) 61a7b47 flambda-backend: Insert missing page table check in roots_nat.c (ocaml#541) 323bd36 flambda-backend: Compiler error when -disable-all-extensions and -extension are used (ocaml#534) d8956b0 flambda-backend: Persistent environment and reproducibility (ocaml#533) 4a0c89f flambda-backend: Revert "Revert bswap PRs (480 and 482)" (ocaml#506) 7803705 flambda-backend: Cause a C warning when CAMLreturn is missing in C stubs. (ocaml#376) 6199db5 flambda-backend: Improve unboxing during cmm for Flambda (ocaml#295) 96b9e1b flambda-backend: Print diagnostics at runtime for Invalid (ocaml#530) 42ab88e flambda-backend: Disable bytecode compilers in ocamltest (ocaml#504) 58c72d5 flambda-backend: Backport ocaml#10595 from upstream/trunk (ocaml#471) 1010539 flambda-backend: Use C++ name mangling convention (ocaml#483) 81881bb flambda-backend: Local allocation test no longer relies on lifting (ocaml#525) f5c4719 flambda-backend: Fix an assertion in Closure that breaks probes (ocaml#505) c2cf2b2 flambda-backend: Add some missing command line arguments to ocamlnat (ocaml#499) git-subtree-dir: ocaml git-subtree-split: 64235a3
xavierleroy
pushed a commit
to xavierleroy/ocaml
that referenced
this pull request
Oct 10, 2023
This commit introduce an atfork hook in the runtime, as a follow up to the issue encountered in MPR ocaml#455 and MPR ocaml#471. As a result, the usage of pthreads_atfork is also removed, opting to rely in both case on manually calling the runtime hook when required. This way, fork() calls from C code no longer reset the OCaml thread machinery. This should help with issues such as ocaml#12636. (cherry picked from commit c6d00ee) (and adapted to 4.14) Co-authored-by: [email protected]
xavierleroy
pushed a commit
to xavierleroy/ocaml
that referenced
this pull request
Oct 26, 2023
This commit introduce an atfork hook in the runtime, as a follow up to the issue encountered in MPR ocaml#455 and MPR ocaml#471. As a result, the usage of pthreads_atfork is also removed, opting to rely in both case on manually calling the runtime hook when required. This way, fork() calls from C code no longer reset the OCaml thread machinery. This should help with issues such as ocaml#12636. (cherry picked from commit c6d00ee) (and adapted to 4.14) Co-authored-by: [email protected]
EmileTrotignon
pushed a commit
to EmileTrotignon/ocaml
that referenced
this pull request
Jan 12, 2024
* Remove unused images * Add a script to detect unused images * Remove unused function * Do not remove `vendor/ood` on distclean * Update images sizes in Applications * Improve books layout
EmileTrotignon
pushed a commit
to EmileTrotignon/ocaml
that referenced
this pull request
Jan 12, 2024
Switch from `lazy_t` to `Lazy.t` Guidance at https://v2.ocaml.org/api/Lazy.html says: > Note: lazy_t is the built-in type constructor used by the compiler for the lazy keyword. You should not use it directly. Always use Lazy.t instead.
ejgallego
pushed a commit
to ejgallego/ocaml
that referenced
this pull request
Nov 5, 2025
This commit introduce an atfork hook in the runtime, as a follow up to the issue encountered in MPR ocaml#455 and MPR ocaml#471. As a result, the usage of pthreads_atfork is also removed, opting to rely in both case on manually calling the runtime hook when required. This way, fork() calls from C code no longer reset the OCaml thread machinery. This should help with issues such as ocaml#12636. (cherry picked from commit c6d00ee) (and adapted to 4.14) Co-authored-by: [email protected]
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See #468 and the PR title. I have also fixed the test suite that I had forgotten to update before.