From b44c143d2a384f8a8d348cc0721de619d75cd175 Mon Sep 17 00:00:00 2001
From: Dan Eble <nine.fierce.ballads@gmail.com>
Date: Sat, 8 Aug 2015 12:21:08 -0400
Subject: [PATCH] Issue 4545: Use Grob::programming_error in Semi_tie:: and
 Tie::less

---
 lily/semi-tie.cc | 17 ++++++++++++-----
 lily/tie.cc      | 17 ++++++++++++-----
 2 files changed, 24 insertions(+), 10 deletions(-)

diff --git a/lily/semi-tie.cc b/lily/semi-tie.cc
index 0669c277f7..0d31a2e073 100644
--- a/lily/semi-tie.cc
+++ b/lily/semi-tie.cc
@@ -89,13 +89,20 @@ bool
 Semi_tie::less (Grob *g1, Grob *g2)
 {
   Item *i1 = dynamic_cast<Item *> (g1);
+  if (!i1)
+    {
+      g1->programming_error ("grob is not a semi-tie");
+      return false;
+    }
+
   Item *i2 = dynamic_cast<Item *> (g2);
-  if (i1 && i2) {
-    return get_position (i1) < get_position (i2);
-  }
+  if (!i2)
+    {
+      g2->programming_error ("grob is not a semi-tie");
+      return true;
+    }
 
-  programming_error ("grob is not a semi-tie");
-  return false;
+  return get_position (i1) < get_position (i2);
 }
 
 Item *
diff --git a/lily/tie.cc b/lily/tie.cc
index 849e859eb5..dcc53ad44e 100644
--- a/lily/tie.cc
+++ b/lily/tie.cc
@@ -45,13 +45,20 @@ bool
 Tie::less (Grob *g1, Grob *g2)
 {
   Spanner *s1 = dynamic_cast<Spanner *> (g1);
+  if (!s1)
+    {
+      g1->programming_error ("grob is not a tie");
+      return false;
+    }
+
   Spanner *s2 = dynamic_cast<Spanner *> (g2);
-  if (s1 && s2) {
-    return get_position (s1) < get_position (s2);
-  }
+  if (!s2)
+    {
+      g2->programming_error ("grob is not a tie");
+      return true;
+    }
 
-  programming_error ("grob is not a tie");
-  return false;
+  return get_position (s1) < get_position (s2);
 }
 
 void
-- 
2.39.5