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

Struct signals not checked for signal-name-style #2147

Open
matlupi opened this issue Mar 21, 2024 · 0 comments
Open

Struct signals not checked for signal-name-style #2147

matlupi opened this issue Mar 21, 2024 · 0 comments
Labels
style-linter Verilog style-linter issues

Comments

@matlupi
Copy link

matlupi commented Mar 21, 2024

Describe the bug

Case formatting of structure signals is not checked.

Test case (preferably reduced)

module test();
typedef struct packed {
  logic OutSelectL;
  logic OutSelectC;
} channel_cfg_t;
endmodule
$ verible-verilog-lint --version
v0.0-3481-gc517d611
Commit	2024-01-05
Built	2024-01-05T17:03:26Z

Actual vs. expected behavior

Actual: Linting passes

$ verible-verilog-lint test.sv
$

Expected: the file should not pass according to signal-name-style which defaults to snake_case

False-negative

@matlupi matlupi added the style-linter Verilog style-linter issues label Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
style-linter Verilog style-linter issues
Projects
None yet
Development

No branches or pull requests

1 participant