Issue #97
Parser issue with empty ports
| Status: | Closed | Start date: | 06/11/2009 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | Wilson Snyder | % Done: | 0% |
|
| Category: | - | |||
| Target version: | - | Estimated time: | 0.10 hour |
Description
Hello, As discussed earleir with Wilson, found a bug in the Parser, description: when using the following code (notice the ',' after lastPort) the parser gives a syntax error:
Module my_module( portA, portB, . . . lastPort, );
the spec says: list_of_ports ::= '(' port { ',' port } ')' port ::= [ port_expression ]
So that's actually legal. However it doesn't do what you think, as it defines another port that's empty that should be connected if doing position based interconnection, so really your code should be fixed as I doubt that's ever the intent.
I think that the best way to resolve this issue is not to let it pass by, but to give a warning about it since it is definetly bad coding that might cause troubles.
History
Updated by Wilson Snyder almost 3 years ago
- Subject changed from Parser issue to Parser issue with empty ports
- Status changed from New to Resolved
- Assignee set to Wilson Snyder
- Estimated time set to 0.10
This is fixed in the GIT version and will be in the next release. Thanks.
Also available in: Atom
![[logo]](/img/veripool_small.png)