Instead of relying on a trick to make the constructor tactic parse, put

all the tactics using the constructor keyword in one entry. This has the
side-effect to also remove the other variant of constructor from the AST.

I also needed to hack around the "tauto" tactic to make it work, by
calling directly the ML tactic through a TacExtend node. This may be
generalized to get rid of the intermingled dependencies between this
tactic and the infamous Ltac quotation mechanism.

1 files changed, 0 insertions, 10 deletions

diff --git a/theories/Init/Notations.v b/theories/Init/Notations.v
index fbad368c2..5095329cd 100644
--- a/theories/Init/Notations.v
+++ b/theories/Init/Notations.v
@@ -90,13 +90,3 @@ Declare ML Module "g_class".
 Declare ML Module "g_eqdecide".
 Declare ML Module "g_rewrite".
 Declare ML Module "tauto".
-
-(** Small hack to overcome the fact that the (e)constructor tactics need to have
-    a proper parsing rule, because the variants with arguments conflicts
-    with it. *)
-Module CoreTactics.
-Declare ML Module "coretactics".
-End CoreTactics.
-
-Tactic Notation "constructor" := CoreTactics.constructor.
-Tactic Notation "econstructor" := CoreTactics.econstructor.