music-notation-swift / music-notation-core

Music Notation Library in Swift (Deprecated. See `music-notation` instead)
MIT License
115 stars 16 forks source link

Remove Note needs to take into account tie #31

Closed drumnkyle closed 8 years ago

drumnkyle commented 8 years ago

The Measure.removeNote needs to take into account a tie. It probably should also take into account a tie across another measure, but I'm not sure how that would be done without bringing the method into Staff, which I think is a bad idea. Something to investigate.

drumnkyle commented 8 years ago

Discussed with @migue48. We are going to make it throw an error if you try to remove a note that is part of a tie. So the user needs to remove the tie first.

Even though we can technically take care of it if it's within the measure we will still throw an error for consistency.