Issue#1389: Whitespace rules for case-generate statements (#1390)

* Issue#1389: Expanded on case rules to check that they don't affect case-generates.

* Issue#1389: Added new case generate rules.

* Issue#1389: Added tests for new case generate rules.

* Issue#1389: Added documentation for new case generate rules.

* Issue#1389: Corrected test file reference.

* Issue#1389: Corrected code snippet.

* Issue#1389: Corrected code snippet.

* Issue#1389: Connected code snippets

* Issue#1389: Corrected code snippet.

* Issue#1389: Corrected code snippet

* Issue#1389: Updated rules to use simpler base rules where possible.

* Issue#1389: Updated rules to use simpler base rules where possible.

* Issue#1389: Corrected mistake in case_004.

Author: JHertz5

docs/case_generate_alternative_rules.rst

@@ -3,6 +3,56 @@
 Case Generate Alternative Rules
 -------------------------------
 
+case_generate_alternative_100
+#############################
+
+|phase_2| |error| |whitespace|
+
+This rule checks for a single space after the **when** keyword.
+
+|configuring_whitespace_rules_link|
+
+**Violation**
+
+.. code-block:: vhdl
+
+  case data generate
+
+    when   3 =>
+
+**Fix**
+
+.. code-block:: vhdl
+
+  case data generate
+
+    when 3 =>
+
+case_generate_alternative_101
+#############################
+
+|phase_2| |error| |whitespace|
+
+This rule checks for a single space before the **=>** operator.
+
+|configuring_whitespace_rules_link|
+
+**Violation**
+
+.. code-block:: vhdl
+
+  case data generate
+
+    when 3   =>
+
+**Fix**
+
+.. code-block:: vhdl
+
+  case data generate
+
+    when 3 =>
+
 case_generate_alternative_300
 #############################
 

docs/case_generate_statement_rules.rst

@@ -3,6 +3,49 @@
 Case Generate Statement Rules
 -----------------------------
 
+case_generate_statement_100
+###########################
+
+|phase_2| |error| |whitespace|
+
+This rule checks for a single space after the **case** keyword.
+
+|configuring_whitespace_rules_link|
+
+**Violation**
+
+.. code-block:: vhdl
+
+   case    data generate
+
+
+**Fix**
+
+.. code-block:: vhdl
+
+   case data generate
+
+case_generate_statement_101
+###########################
+
+|phase_2| |error| |whitespace|
+
+This rule checks for a single space before the **generate** keyword.
+
+|configuring_whitespace_rules_link|
+
+**Violation**
+
+.. code-block:: vhdl
+
+   case data    generate
+
+**Fix**
+
+.. code-block:: vhdl
+
+   case data generate
+
 case_generate_statement_400
 ###########################
 

docs/configuring_whitespace_rules.rst

@@ -136,6 +136,10 @@ Rules Enforcing Whitespace
 * `case_004 <case_rules.html#case-004>`_
 * `case_005 <case_rules.html#case-005>`_
 * `case_006 <case_rules.html#case-006>`_
+* `case_generate_alternative_100 <case_generate_alternative_rules.html#case-generate-alternative-100>`_
+* `case_generate_alternative_101 <case_generate_alternative_rules.html#case-generate-alternative-101>`_
+* `case_generate_statement_100 <case_generate_statement_rules.html#case-generate-statement-100>`_
+* `case_generate_statement_101 <case_generate_statement_rules.html#case-generate-statement-101>`_
 * `comment_004 <comment_rules.html#comment-004>`_
 * `component_002 <component_rules.html#component-002>`_
 * `component_007 <component_rules.html#component-007>`_

docs/rule_groups/whitespace_rule_group.rst

@@ -29,6 +29,10 @@ Rules Enforcing Whitespace Rule Group
 * `case_004 <../case_rules.html#case-004>`_
 * `case_005 <../case_rules.html#case-005>`_
 * `case_006 <../case_rules.html#case-006>`_
+* `case_generate_alternative_100 <../case_generate_alternative_rules.html#case-generate-alternative-100>`_
+* `case_generate_alternative_101 <../case_generate_alternative_rules.html#case-generate-alternative-101>`_
+* `case_generate_statement_100 <../case_generate_statement_rules.html#case-generate-statement-100>`_
+* `case_generate_statement_101 <../case_generate_statement_rules.html#case-generate-statement-101>`_
 * `comment_004 <../comment_rules.html#comment-004>`_
 * `comment_100 <../comment_rules.html#comment-100>`_
 * `component_002 <../component_rules.html#component-002>`_

tests/case/rule_002_test_input.fixed.vhd

@@ -33,4 +33,14 @@ begin
 
   end process PROC_2;
 
+  gen_label : case   boolean_1 generate
+
+   when STATE_1 =>
+
+     a <= b;
+     b <= c;
+     c <= d;
+
+  end generate;
+
 end architecture ARCH;

tests/case/rule_002_test_input.vhd

@@ -33,4 +33,14 @@ begin
 
   end process PROC_2;
 
+  gen_label : case   boolean_1 generate
+
+   when STATE_1 =>
+
+     a <= b;
+     b <= c;
+     c <= d;
+
+  end generate;
+
 end architecture ARCH;

tests/case/rule_003_test_input.fixed.vhd

@@ -33,4 +33,14 @@ begin
 
   end process PROC_2;
 
+  gen_label : case boolean_1    generate
+
+   when STATE_1 =>
+
+     a <= b;
+     b <= c;
+     c <= d;
+
+  end generate;
+
 end architecture ARCH;

tests/case/rule_003_test_input.vhd

@@ -33,4 +33,14 @@ begin
 
   end process PROC_2;
 
+  gen_label : case boolean_1    generate
+
+   when STATE_1 =>
+
+     a <= b;
+     b <= c;
+     c <= d;
+
+  end generate;
+
 end architecture ARCH;

tests/case/rule_004_test_input.fixed.vhd

@@ -33,4 +33,14 @@ begin
 
   end process PROC_2;
 
+  gen_label : case boolean_1 generate
+
+   when    STATE_1 =>
+
+     a <= b;
+     b <= c;
+     c <= d;
+
+  end generate;
+
 end architecture ARCH;

tests/case/rule_004_test_input.vhd

@@ -33,4 +33,14 @@ begin
 
   end process PROC_2;
 
+  gen_label : case boolean_1 generate
+
+   when    STATE_1 =>
+
+     a <= b;
+     b <= c;
+     c <= d;
+
+  end generate;
+
 end architecture ARCH;