Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix many typos #104

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix many typos #104

wants to merge 2 commits into from

Conversation

chu11
Copy link

@chu11 chu11 commented Apr 7, 2023

Fix many typos / spelling errors found with catch-ci typos tool.


We recently tried to add some typo checking into our project and found a bunch in ccan. Went ahead and fixed more of them beyond the ones we use.

Some judgement calls on ones not to fix, like RFCs which I figure were copied from ietf. Two fixes were within code, which I hope is ok (local variable spelled wrong, static internal function name spelled wrong). Everything else is typos in comments or documentation.

rustyrussell and others added 2 commits January 9, 2023 12:36
Seems to be happening with gcc 12.2.0-3ubuntu1:

```
lightningd/jsonrpc.c: In function ‘destroy_json_command’:
lightningd/jsonrpc.c:1180:63: error: the comparison will always evaluate as ‘false’ for the address of ‘canary’ will never be NULL [-Werror=address]
lightningd/jsonrpc.c:108:53: note: ‘canary’ declared here
```

Signed-off-by: Rusty Russell <[email protected]>
Fix many typos / spelling errors found with catch-ci typos tool.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants