From: David Kastrup <dak@gnu.org>
Date: Tue, 16 Aug 2011 17:46:33 +0000 (+0200)
Subject: context-def.cc: scm_memq before scm_delete_x is redundant, and its result is not... 
X-Git-Tag: release/2.15.9-1~9^2~56
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=1b197ba1aaae9b56d0d1e4979ead09e21619817c;p=lilypond.git

context-def.cc: scm_memq before scm_delete_x is redundant, and its result is not a valid C condition anyway and could have caused duplicate entries.
---

diff --git a/lily/context-def.cc b/lily/context-def.cc
index 5dd1684a83..e5f33c75c7 100644
--- a/lily/context-def.cc
+++ b/lily/context-def.cc
@@ -174,8 +174,7 @@ Context_def::get_accepted (SCM user_mod) const
   SCM def = get_default_child (user_mod);
   if (scm_is_symbol (def))
     {
-      if (scm_memq (def, acc))
-        acc = scm_delete_x (def, acc);
+      acc = scm_delete_x (def, acc);
       acc = scm_cons (def, acc);
     }