Skip to content

parser,checker: vls skip unrelated files#25531

Merged
medvednikov merged 9 commits into
vlang:masterfrom
kbkpbot:fix-parser-checker-vls-skip-files
Oct 20, 2025
Merged

parser,checker: vls skip unrelated files#25531
medvednikov merged 9 commits into
vlang:masterfrom
kbkpbot:fix-parser-checker-vls-skip-files

Conversation

@kbkpbot

@kbkpbot kbkpbot commented Oct 18, 2025

Copy link
Copy Markdown
Contributor

This is discussed at #25519

This PR skip all other files except the user file itself.
Hope this can speedup the vls process.

@vlang vlang deleted a comment from huly-for-github Bot Oct 18, 2025
@kbkpbot

kbkpbot commented Oct 19, 2025

Copy link
Copy Markdown
Contributor Author

The CI fail with (https://github.com/vlang/v/actions/runs/18616692038/job/53082198478?pr=25531)

v -o v.c cmd/v
  gcc -Werror -municode -w v.c -lws2_32
  shell: C:\Program Files\PowerShell\7\pwsh.EXE -command ". '{0}'"
  env:
    VFLAGS: -cc gcc
Error: v.c:568:44: error: two or more data types in declaration specifiers
  568 |                                 typedef u8 bool;
      |                                            ^~~~
Error: Process completed with exit code 1.

It seems that in latest gcc, bool become an internal datatype? Because I compile following c code, has no problem at all:

int main() {
    bool test = 1;
    (void)test;
    return 0;
}

So I suggest change the CI command to :

gcc -Werror -municode -w v.c -lws2_32 -std=c99

@JalonSolov

Copy link
Copy Markdown
Collaborator

That would make sense. The latest C standard did, indeed, introduce BOOL as a real datatype, not just a #define.

@medvednikov medvednikov merged commit b055769 into vlang:master Oct 20, 2025
83 checks passed
@medvednikov

Copy link
Copy Markdown
Member

Great job!

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.

3 participants