[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5284 Improve ASSIGN_EVENT_ONCE
- **labels**: --> Fixed_2_21_0 - Description has changed: Diff: --- old +++ new @@ -1,4 +1,3 @@ - http://codereview.appspot.com/338540043Improve ASSIGN_EVENT_ONCE * Really assign the event only once. - **status**: Started --> Fixed - **Patch**: push --> - **Comment**: commit 852123e930a9fde9b73293f5d2b2ba52d8ea397b (HEAD -> pushed/issue-5284-event-reassignment-warning, origin/staging) Author: Dan EbleDate: Fri Mar 2 21:45:26 2018 -0500 Issue 5284: improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. * Rephrase the warning so that it could be used more broadly than for simultaneous events. * Rephrase the warning so that it could be used with events of different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. --- ** [issues:#5284] Improve ASSIGN_EVENT_ONCE** **Status:** Fixed **Labels:** Fixed_2_21_0 **Created:** Sun Mar 04, 2018 09:10 PM UTC by Dan Eble **Last Updated:** Tue Mar 13, 2018 01:08 PM UTC **Owner:** Dan Eble http://codereview.appspot.com/338540043Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. * Rephrase the warning so that it could be used more broadly than for simultaneous events. * Rephrase the warning so that it could be used with events of different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. http://codereview.appspot.com/338540043 --- Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is subscribed to https://sourceforge.net/p/testlilyissues/issues/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot___ Testlilyissues-auto mailing list testlilyissues-a...@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5284 Improve ASSIGN_EVENT_ONCE
- Description has changed: Diff: --- old +++ new @@ -1,4 +1,5 @@ -Improve ASSIGN_EVENT_ONCE + +http://codereview.appspot.com/338540043Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. @@ -8,4 +9,5 @@ different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. + http://codereview.appspot.com/338540043 - **Patch**: countdown --> push - **Comment**: Patch counted down - please push. --- ** [issues:#5284] Improve ASSIGN_EVENT_ONCE** **Status:** Started **Created:** Sun Mar 04, 2018 09:10 PM UTC by Dan Eble **Last Updated:** Sat Mar 10, 2018 09:26 AM UTC **Owner:** Dan Eble http://codereview.appspot.com/338540043Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. * Rephrase the warning so that it could be used more broadly than for simultaneous events. * Rephrase the warning so that it could be used with events of different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. http://codereview.appspot.com/338540043 --- Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is subscribed to https://sourceforge.net/p/testlilyissues/issues/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot___ Testlilyissues-auto mailing list testlilyissues-a...@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5284 Improve ASSIGN_EVENT_ONCE
- **Patch**: review --> countdown - **Comment**: Patch on countdown for March 13th --- ** [issues:#5284] Improve ASSIGN_EVENT_ONCE** **Status:** Started **Created:** Sun Mar 04, 2018 09:10 PM UTC by Dan Eble **Last Updated:** Fri Mar 09, 2018 09:41 AM UTC **Owner:** Dan Eble Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. * Rephrase the warning so that it could be used more broadly than for simultaneous events. * Rephrase the warning so that it could be used with events of different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. http://codereview.appspot.com/338540043 --- Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is subscribed to https://sourceforge.net/p/testlilyissues/issues/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot___ Testlilyissues-auto mailing list testlilyissues-a...@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5284 Improve ASSIGN_EVENT_ONCE
- **Needs**: --> - **Patch**: new --> review - **Type**: --> Enhancement - **Comment**: Passes make, make check and a full make doc. --- ** [issues:#5284] Improve ASSIGN_EVENT_ONCE** **Status:** Started **Created:** Sun Mar 04, 2018 09:10 PM UTC by Dan Eble **Last Updated:** Wed Mar 07, 2018 11:58 PM UTC **Owner:** Dan Eble Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. * Rephrase the warning so that it could be used more broadly than for simultaneous events. * Rephrase the warning so that it could be used with events of different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. http://codereview.appspot.com/338540043 --- Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is subscribed to https://sourceforge.net/p/testlilyissues/issues/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot___ Testlilyissues-auto mailing list testlilyissues-a...@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5284 Improve ASSIGN_EVENT_ONCE
Rephrase warning http://codereview.appspot.com/338540043 --- ** [issues:#5284] Improve ASSIGN_EVENT_ONCE** **Status:** Started **Created:** Sun Mar 04, 2018 09:10 PM UTC by Dan Eble **Last Updated:** Wed Mar 07, 2018 11:58 PM UTC **Owner:** Dan Eble Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. * Rephrase the warning so that it could be used more broadly than for simultaneous events. * Rephrase the warning so that it could be used with events of different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. http://codereview.appspot.com/338540043 --- Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is subscribed to https://sourceforge.net/p/testlilyissues/issues/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot___ Testlilyissues-auto mailing list testlilyissues-a...@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5284 Improve ASSIGN_EVENT_ONCE
- **Patch**: review --> countdown - **Comment**: Patch on countdown for March 10th --- ** [issues:#5284] Improve ASSIGN_EVENT_ONCE** **Status:** Started **Created:** Sun Mar 04, 2018 09:10 PM UTC by Dan Eble **Last Updated:** Mon Mar 05, 2018 10:32 AM UTC **Owner:** Dan Eble Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. * Rephrase the warning so that it could be used more broadly than for simultaneous events. * Rephrase the warning so that it could be used with events of different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. http://codereview.appspot.com/338540043 --- Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is subscribed to https://sourceforge.net/p/testlilyissues/issues/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot___ Testlilyissues-auto mailing list testlilyissues-a...@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5284 Improve ASSIGN_EVENT_ONCE
- **summary**: Issue : improve ASSIGN_EVENT_ONCE --> Improve ASSIGN_EVENT_ONCE - Description has changed: Diff: --- old +++ new @@ -1,4 +1,4 @@ -Issue : improve ASSIGN_EVENT_ONCE +Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. - **Needs**: --> - **Patch**: new --> review - **Type**: --> Enhancement - **Comment**: Passes make, make check and a full make doc. --- ** [issues:#5284] Improve ASSIGN_EVENT_ONCE** **Status:** Started **Created:** Sun Mar 04, 2018 09:10 PM UTC by Dan Eble **Last Updated:** Mon Mar 05, 2018 12:36 AM UTC **Owner:** Dan Eble Improve ASSIGN_EVENT_ONCE * Really assign the event only once. * End an unnecessary relationship with translator listeners. * Rephrase the warning so that it could be used more broadly than for simultaneous events. * Rephrase the warning so that it could be used with events of different classes (e.g. abc-mark-event and xyz-mark-event). * Inline the common case that the event is assigned without warning. http://codereview.appspot.com/338540043 --- Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is subscribed to https://sourceforge.net/p/testlilyissues/issues/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.-- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot___ Testlilyissues-auto mailing list testlilyissues-a...@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto