From: Rune Zedeler <rune@lilypond.org>
Date: Fri, 21 Dec 2007 22:56:03 +0000 (+0100)
Subject: clean-up pitch-interval.cc and ambitus-engraver.cc
X-Git-Tag: release/2.11.37-1~22^2
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=e2e022df5e6dc2d160b036e908305bf50d36882e;p=lilypond.git

clean-up pitch-interval.cc and ambitus-engraver.cc

add_pitch() returns Drul_array<bool> instead of Direction
---

diff --git a/lily/ambitus-engraver.cc b/lily/ambitus-engraver.cc
index 0cfbc58e98..e00ea133dd 100644
--- a/lily/ambitus-engraver.cc
+++ b/lily/ambitus-engraver.cc
@@ -122,11 +122,11 @@ Ambitus_engraver::acknowledge_note_head (Grob_info info)
   if (nr && nr->in_event_class ("note-event"))
     {
       Pitch pitch = *unsmob_pitch (nr->get_property ("pitch"));
-      if (pitch_interval_.is_empty())
-	causes_[UP] = causes_[DOWN] = nr;
-      Direction expand_dir = pitch_interval_.add_point (pitch);
-      if (expand_dir)
-	causes_[expand_dir] = nr;
+      Drul_array<bool> expands = pitch_interval_.add_point (pitch);
+      if (expands[UP])
+	causes_[UP] = nr;
+      if (expands[DOWN])
+	causes_[DOWN] = nr;
     }
 }
 
diff --git a/lily/include/pitch-interval.hh b/lily/include/pitch-interval.hh
index 478b58a7a8..04bd23a702 100644
--- a/lily/include/pitch-interval.hh
+++ b/lily/include/pitch-interval.hh
@@ -16,7 +16,7 @@ class Pitch_interval : public Drul_array<Pitch>
 public:
   Pitch_interval ();
   Pitch_interval (Pitch, Pitch);
-  Direction add_point (Pitch);
+  Drul_array<bool> add_point (Pitch);
   bool is_empty () const;
 };
 
@@ -25,7 +25,7 @@ class Pitch_lexicographic_interval : public Drul_array<Pitch>
 public:
   Pitch_lexicographic_interval ();
   Pitch_lexicographic_interval (Pitch, Pitch);
-  Direction add_point (Pitch);
+  Drul_array<bool> add_point (Pitch);
   bool is_empty () const;
 };
 
diff --git a/lily/pitch-interval.cc b/lily/pitch-interval.cc
index 3043bbff07..a7390aa282 100644
--- a/lily/pitch-interval.cc
+++ b/lily/pitch-interval.cc
@@ -28,26 +28,21 @@ Pitch_interval::is_empty () const
   return at (LEFT) > at (RIGHT);
 }
 
-Direction
+Drul_array<bool>
 Pitch_interval::add_point (Pitch p)
 {
-  if (is_empty())
-    {
-      at (LEFT) = at (RIGHT) = p;
-      return CENTER;
-    }
-  else if (at (LEFT).tone_pitch () > p.tone_pitch ())
+  Drul_array<bool> expansions(false,false);
+  if (at (LEFT).tone_pitch () > p.tone_pitch ())
     {
       at (LEFT) = p;
-      return LEFT;
+      expansions [LEFT] = true;
     }
-  else if (at (RIGHT).tone_pitch () < p.tone_pitch ())
+  if (at (RIGHT).tone_pitch () < p.tone_pitch ())
     {
       at (RIGHT) = p;
-      return RIGHT;
+      expansions [RIGHT] = true;
     }
-  else
-    return CENTER;
+  return expansions;
 }
 
 
@@ -69,24 +64,19 @@ Pitch_lexicographic_interval::is_empty () const
   return at (LEFT) > at (RIGHT);
 }
 
-Direction
+Drul_array<bool>
 Pitch_lexicographic_interval::add_point (Pitch p)
 {
-  if (is_empty())
-    {
-      at (LEFT) = at (RIGHT) = p;
-      return CENTER;
-    }
-  else if (at (LEFT) > p)
+  Drul_array<bool> expansions(false,false);
+  if (at (LEFT) > p)
     {
       at (LEFT) = p;
-      return LEFT;
+      expansions [LEFT] = true;
     }
-  else if (at (RIGHT) < p)
+  if (at (RIGHT) < p)
     {
       at (RIGHT) = p;
-      return RIGHT;
+      expansions [RIGHT] = true;
     }
-  else
-    return CENTER;
+  return expansions;
 }