You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
with the re_required_port rules interfaces get mistaken for inout, input, or output depending on which was the last used. This makes it impossible to use the rules at the same time especially if different regex is used for each. For example the module
with the regex rules
re_required_port_interface = "^[a-z]+[a-z0-9_]$"
re_required_port_inout = "^[iov?[A-Za-z]+[A-Za-z0-9]$"
re_required_port_input = "^[iv?[A-Za-z]+[A-Za-z0-9]$"
re_required_port_output = "^[ov?[A-Za-z]+[A-Za-z0-9]$"
throws an error on all interfaces in the module but it does it for the inout rule on the first one, input rule on the second one, output rule on the third one, and inout rule on the 4th one. If the port name for the interface is changed to not match the regex it still throws the interface rule error but it also throws the other error as well.
Looking through the source code it looks like it is because the rules for inout, input, and output have checks in place for if the previous port direction was an input, output, inout, or ref but not a check for if it is an interface. (see re_required_port_inout.rs lines 28-44 as an example compare with re_require_port_interface.rs lines 26-34). I'm not familiar enough with the source code yet to know for sure or to fix it but that is what I was able to find
The text was updated successfully, but these errors were encountered:
with the re_required_port rules interfaces get mistaken for inout, input, or output depending on which was the last used. This makes it impossible to use the rules at the same time especially if different regex is used for each. For example the module
with the regex rules
re_required_port_interface = "^[a-z]+[a-z0-9_]$"
re_required_port_inout = "^[iov?[A-Za-z]+[A-Za-z0-9]$"
re_required_port_input = "^[iv?[A-Za-z]+[A-Za-z0-9]$"
re_required_port_output = "^[ov?[A-Za-z]+[A-Za-z0-9]$"
throws an error on all interfaces in the module but it does it for the inout rule on the first one, input rule on the second one, output rule on the third one, and inout rule on the 4th one. If the port name for the interface is changed to not match the regex it still throws the interface rule error but it also throws the other error as well.
Looking through the source code it looks like it is because the rules for inout, input, and output have checks in place for if the previous port direction was an input, output, inout, or ref but not a check for if it is an interface. (see re_required_port_inout.rs lines 28-44 as an example compare with re_require_port_interface.rs lines 26-34). I'm not familiar enough with the source code yet to know for sure or to fix it but that is what I was able to find
The text was updated successfully, but these errors were encountered: