Prefix/suffix exceptions not exempted from case checking in instantiation_008 about vhdl-style-guide HOT 6 CLOSED

Evening @maltaisn ,

The prefix and suffix exceptions are applied at the same time. So the configuration you have would be checking the following:

  P_comp_S : component comp
    port map (
      a => a,
      b => b
    );

Since your first instantiation is missing the suffix _S it is considered a violation.
The second instance is missing the prefix _P so it is also considered a violation.

I scanned the documentation and there is nothing that covers using both prefix and suffix exceptions at the same time.

The question is, how would you like to proceed?

--Jeremy

from vhdl-style-guide.

I would have expected the check to succeed if either a prefix is present, a suffix, or one of both.

from vhdl-style-guide.

@maltaisn This is a comment from an outside of what you're working on, but I believe it wouldn't be impossible to create a rule that enforces one of both. There are two base rules (rules that are the foundation for many of the full set of VSG's built in rules) that manage prefix and suffix rules: vsg.rules.token_prefix.py and vsg.rules.token_suffix.py. While the standard way to create a rule usually involves extending your rule off of one of these base rules, in order to capture the functionality of both of these rules, you could create a unique rule that instantiates both these rules as objects, runs their analysis manually, and adds violations based on the violations present within those objects.

--Benito

from vhdl-style-guide.

Evening @maltaisn and @Benito-E ,

I reviewed the implementation and I believe the best course of action would be to update the prefix/suffix base rule to check for all permutations of prefix and suffix exceptions if both are defined. I should have considered those permutations when I first added the exceptions.

I have pushed an update for this to issue-1177 branch. When you get a chance could you check it out on your end and let me know if it is working for you?

Thanks,

--Jeremy

from vhdl-style-guide.

It works for me, thanks!

from vhdl-style-guide.

Awesome I will merge this to master.

from vhdl-style-guide.