note-off ordering - fixes #6340

Evoral::Beats::operator>() rounds to (1.0 / PPQN), hardcoded 1/1920.0.

If the time difference between two events is smaller than 1/PPQN,
Beats::operator>()  and  Beats::operator<()  produce ambiguous results.
The same pair of values is both "less than" and "greater than" depending
which operator is used.

While it's fine for some cases to ignore the order of nearly concurent
events, the std::priority_queue must be strictly ordered.

Author: x42

libs/evoral/evoral/Sequence.hpp

@@ -134,19 +134,21 @@ class LIBEVORAL_API Sequence : virtual public ControlSet {
 		}
 	};
 
+#if 0 // NOT USED
 	struct LaterNoteComparator {
 		typedef const Note<Time>* value_type;
 		inline bool operator()(const boost::shared_ptr< const Note<Time> > a,
 		                       const boost::shared_ptr< const Note<Time> > b) const {
 			return a->time() > b->time();
 		}
 	};
+#endif
 
 	struct LaterNoteEndComparator {
 		typedef const Note<Time>* value_type;
 		inline bool operator()(const boost::shared_ptr< const Note<Time> > a,
 		                       const boost::shared_ptr< const Note<Time> > b) const {
-			return a->end_time() > b->end_time();
+			return a->end_time().to_double() > b->end_time().to_double();
 		}
 	};