[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Re: #4834 Remove routing information from Grob_info

[Auto mailings of changes to Lily Issues]

Well, that's good. I liked the overall effect of the diffs, but it would have been a maintenance nightmare to sneak them in via a bug. I have a pretty good idea how to replicate their effect in a proper manner and might take a look eventually whether that's feasible. But for this refactoring issue, "no diffs" is certainly the correct outcome.

---

[issues:#4834] Remove routing information from Grob_info

Status: Started
Created: Wed Apr 27, 2016 06:12 AM UTC by pkx166h
Last Updated: Fri Apr 29, 2016 01:23 PM UTC
Owner: David Kastrup
Attachments:

• Selection_005.png (182.6 kB; image/png)
• Selection_006.png (204.8 kB; image/png)

Remove routing information from Grob_info

It passes information to acknowledgers that does not really belong there and instead
manages how information gets there. Moving this information into arguments for
the various kinds of announce_grob procedure is cleaner and allows to bounce
Grob_info through Scheme without the danger of information loss.

http://codereview.appspot.com/294330043

---

Sent from sourceforge.net because address@hidden 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.

------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto