[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **labels**: --> Fixed_2_21_2 - **status**: Started --> Fixed - **Patch**: push --> - **Comment**: ~~~ commit 550d0a4def00786bf7c7373101dced4c68ad893d Merge: eeea9423b2 42eaa5cb51 Author: David Kastrup AuthorDate: Thu Apr 30 14:24:06 2020 +0200 Commit: David Kastrup CommitDate: Thu

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: countdown --> push - **Comment**: Patch counted down - please push --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Tue Apr 28, 2020 07:25 AM UTC

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

One more time on countdown I think - speak now or forever etc. (?) --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Sun Apr 26, 2020 08:44 AM UTC **Owner:**

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: review --> countdown - **Comment**: Patch on countdown for April 28th - seems to be some consensus here, if only to 'make a start'. --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Comment**: Leaving on Review (thanks for the clarification David). --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Sun Apr 19, 2020 08:19 AM UTC

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

We've had no movement (either way) on this for nearly a week. Should I be setting this back to needs_work or something else? --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

Sorry, I was a bit hasty with this. OK still looks like consensus needed somewhere. Leaving on Review --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Fri Apr

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: countdown --> review --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Fri Apr 17, 2020 08:35 AM UTC **Owner:** David Kastrup Refactor

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: review --> countdown - **Comment**: Patch on countdown for April 19th --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Wed Apr 15, 2020 02:51 AM

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

Leaving on Review as the discussion continues. --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Mon Apr 13, 2020 08:37 AM UTC **Owner:** David Kastrup

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: waiting --> review - **Comment**: Setting back to reivew as discussion continues. I'll check again on next countdown (unless of course a new patch is submitted). --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: push --> waiting - **Comment**: I think this should not be pushed. --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Mon Apr 13, 2020 07:34 AM

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: countdown --> push - **Comment**: I cannot tell if the discussions on Rietveld are holding this back or not (or are just general discussions on some internal design), I'll leave the decision to push obvious to you but it has been counted down as far as the 'time' goes. --- **

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: review --> countdown - **Comment**: Patch on countdown for Apri 13th --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Wed Apr 08, 2020 04:36 PM

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- **Patch**: new --> review - **Comment**: Passes make, make check and a full make doc. --- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Tue Apr 07, 2020

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

- Description has changed: Diff: --- old +++ new @@ -8,4 +8,28 @@ that the humongous mechanical change in the middle commit can be reviewed separately from the manual commits. +Commits are: + +Issue 5882/1: Grob::flush_extent_cache: refactor del_property call + +Issue 5882/2:

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5882 Refactor get/set_property to take the item as first argument

--- ** [issues:#5882] Refactor get/set_property to take the item as first argument** **Status:** Started **Created:** Tue Apr 07, 2020 02:14 PM UTC by David Kastrup **Last Updated:** Tue Apr 07, 2020 02:14 PM UTC **Owner:** nobody Refactor get/set_property to take the item as first argument