Semi-automatic cSpell pass to detect typos #54
Merged
Conversation
This file contains 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
- This is the results of a pass through with cSpell, where I looked briefly at the warnings generated. This is a semi-automatic pass, which would need to be completed by another read at a slower pace.
By this I mean some more reading I intend to do outside of the scope of this PR. I have enabled for review. |
|
Thanks! |
dinosaure
added a commit
to dinosaure/opam-repository
that referenced
this pull request
Jan 13, 2025
CHANGES: - Notice the domain if it needs to look into the shared heap if a task is ready to be transfered (@dinosaure, robur-coop/miou#41) - Don't use `Option.value` but `match .. with` to calculate the optional length for `Miou_unix.{read,write}` (@kit-ty-kate, robur-coop/miou#44) - Use `List.iter` instead of `Hashtbl.iter` for internal kept file-descriptors of `Miou_unix` (@dinosaure, robur-coop/miou#45) - Improve the documentation of `Miou_unix` about suspended syscalls (@dinosaure, @kit-ty-kate, robur-coop/miou#43) - Export `reraise` (@dinosaure, robur-coop/miou#46) - Fix an issue on the `dom0` and observe if some tasks must be transfered to it (@dinosaure, robur-coop/miou#48) - Fix documentation (@mbarbin, robur-coop/miou#47) - Fix the formatter (@mbarbin, robur-coop/miou#51) - Upgrade miou to `ocamlformat.0.27.0` (@mbarbin, robur-coop/miou#52) - Add `x-maintenance-intent` (@hannesm, robur-coop/miou#56) - Improve the documentation and some `odoc` warnings (@mbarbin, robur-coop/miou#53, robur-coop/miou#54)
davesnx
pushed a commit
to davesnx/opam-repository
that referenced
this pull request
Mar 12, 2025
CHANGES: - Don't try to abusively fill the pipe to interrupt a domain Interrupting a domain involves writing to a pipe to interrupt the `select(2)` if it is running. The pipe has a limited memory, depending on the system, and if you ask to interrupt a domain too much, you end up blocking the `write`. This patch prevents writing to the pipe if it has not yet been read. (@dinosaure, robur-coop/miou#46) - Expose the Sequence module (@dinosaure, robur-coop/miou#47) - Be able to add a hook (effect free) into the scheduler It is possible to add a hook to the scheduler. If the user wants to execute a function to a domain each time the domain is busy with a task, they can do so. However, the effects are not managed in the passed function. (@dinosaure, robur-coop/miou#48) - Add `Miou.Lazy`, a domain-safe `Lazy` module like `Stdlib.Lazy` (@dinosaure, initially implemented by @polytypic, robur-coop/miou#49) - Raise an exception if the user uses syscalls (from `Miou_unix`) and `Miou.run` instead of `Miou_unix.run` If a user uses a suspend function offered by `Miou_unix` but does not use `Miou_unix.run`, the programme may block indefinitely. This patch prevents such an error by raising an exception if we want to add a suspension point and we haven't specified how to handle it (if we use `Miou.run` instead of `Miou_unix.run`). (@dinosaure, reported by @kit-ty-kate, robur-coop/miou#51) - Rename `Miou.set_signal` to `Miou.sys_signal` (@dinosaure, robur-coop/miou#50) - Improve `Miou_unix.{read,write}` (@kit-ty-kate, @dinosaure, robur-coop/miou#52, 2f552a6, robur-coop/miou#54) - Fix an issue related to the dom0 and pending tasks locked by mutexes Tasks may have been transmitted to dom0 while it was executing a task and before the `select(2)`. This patch resynchronises the pending tasks in dom0's TODO-list before making the `select(2)`: specifically to find out whether the `select(2)` can block indefinitely or not. This patch also cleans up the old states of the tables used by `Miou_unix` if it is used on an ongoing basis (as in the case of tests). (@dinosaure, robur-coop/miou#53) - Add `Miou.Domain.available` (@dinosaure, robur-coop/miou#53) - Fix a race condition (observed with TSan) when we wait the cancellation of a children This patch changes Miou's behaviour a little when waiting for a task to be cancelled and prevents invalid access to a value that does not belong to the current domain (and which can be modified by another domain). Thanks @OlivierNicole and @fabbing for their advice on using TSan. (@dinosaure, robur-coop/miou#56) - Update the layout of Miou to avoid conflicts with other packages (like `backoff`) (@dinosaure, reported by @patricoferris, robur-coop/miou#57) - OCaml 5.3 support (@kit-ty-kate, github#22) - Rename `Miou.call_cc` to `Miou.async` (@dinosaure, @kit-ty-kate, @Armael, github#23)
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.
This is the results of a pass through with cSpell, where I looked briefly at the warnings generated. This is a semi-automatic pass, which would need to be completed by another read at a slower pace. By this I mean that I only went through the warnings that cSpell automatically generated (not the entire text), and then chose manually how to resolve them.
I used a combined en_US & en_GB dictionary, since both conventions seemed to be used in conjunction in the project (e.g. finalizer + finaliser, etc.).
For added context see #50