Skip to content

C: enable spelling for new language constructs (Closes #257)#258

Merged
blopker merged 1 commit intoblopker:mainfrom
zbreit:feature/more-c-spellchecking
May 4, 2026
Merged

C: enable spelling for new language constructs (Closes #257)#258
blopker merged 1 commit intoblopker:mainfrom
zbreit:feature/more-c-spellchecking

Conversation

@zbreit
Copy link
Copy Markdown
Contributor

@zbreit zbreit commented May 3, 2026

Closes #257.

  1. Make C unit tests exhaustive for all the queries in c.scm (though not necessarily a full reflection of every C language feature).
    2. Add spell checking for macro functions and parameters: #define MACRO_FUNCTION(macro_param) macro_param + 1.
  2. Add spell checking for named unions: union MyUnion {}.
  3. Add spell checking for array fields and variables: int nums[20].
  4. Made string checking more general. This change ensures that string1 and string2 are both spell-checked in the example "string1\nstring2". Also string literals outside of function calls and initializers (e.g., during assignment) also get checked.

1. Make C unit tests exhaustive for all the queries in `c.scm` (though
   not necessarily a full reflection of every C language feature).
   2. Add spell checking for macro functions and parameters: `#define
   MACRO_FUNCTION(macro_param) macro_param + 1`.
3. Add spell checking for named unions: `union MyUnion {}`.
4. Add spell checking for array fields and variables: `int nums[20]`.
5. Made string checking more general. This change ensures that `string1`
   and `string2` are both spell-checked in the example
   `"string1\nstring2"`. Also string literals outside of function calls
   and initializers (e.g., during assignment) also get checked.
@blopker
Copy link
Copy Markdown
Owner

blopker commented May 4, 2026

Very cool. Thank you! Great test cases.

@blopker blopker merged commit 980c173 into blopker:main May 4, 2026
10 checks passed
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.

[BUG]: Spell checking features for C

2 participants