Skip to content

Fix many typos#104

Open
chu11 wants to merge 2 commits into
rustyrussell:masterfrom
chu11:fix_typos
Open

Fix many typos#104
chu11 wants to merge 2 commits into
rustyrussell:masterfrom
chu11:fix_typos

Conversation

@chu11

@chu11 chu11 commented Apr 7, 2023

Copy link
Copy Markdown

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 <rusty@rustcorp.com.au>
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