Issue 5056/3: Don't overcheck arguments to \remove

Instead we delete engravers with delq! in order to be fast and reliable.

diff --git a/lily/context-def.cc b/lily/context-def.cc
index 5768ee5471989f94d42dc87fc13f6c1737e2489e..72ddf6d37b32e816b0bf585a2fede4f41b429915 100644 (file)
--- a/lily/context-def.cc
+++ b/lily/context-def.cc
@@ -291,11 +291,8 @@ Context_def::get_translator_names (SCM user_mod) const
 
       if (scm_is_eq (tag, ly_symbol2scm ("consists")))
         l1 = scm_cons (arg, l1);
-      else if (scm_is_eq (tag, ly_symbol2scm ("remove"))
-               && (scm_is_pair (arg)
-                   || ly_is_procedure (arg)
-                   || get_translator_creator (arg)))
-        l1 = scm_delete_x (arg, l1);
+      else if (scm_is_eq (tag, ly_symbol2scm ("remove")))
+        l1 = scm_delq_x (arg, l1);
     }
 
   return l1;