- **summary**: Use more const Grobs in the Paper_column interface --> Constness 
of Grob methods
- Description has changed:

Diff:

~~~~

--- old
+++ new
@@ -1,3 +1 @@
-Use more const Grobs in the Paper_column interface
-
-http://codereview.appspot.com/346100043
+Grob::internal_get_property is advertised as a const method, but it came up in 
a [code review](http://codereview.appspot.com/346100043) that "caching of 
values is not really conceptually const."  So, at the very least, the const is 
misleading.  It would be good to change it for clarity, but it might require a 
chain of consequential changes.

~~~~

- **status**: Started --> Shelved
- **Patch**: review -->  
- **Type**: Enhancement --> Maintainability



---

** [issues:#5349] Constness of Grob methods**

**Status:** Shelved
**Created:** Mon Jun 18, 2018 11:56 PM UTC by Dan Eble
**Last Updated:** Tue Jun 19, 2018 12:27 PM UTC
**Owner:** Dan Eble


Grob::internal_get_property is advertised as a const method, but it came up in 
a [code review](http://codereview.appspot.com/346100043) that "caching of 
values is not really conceptually const."  So, at the very least, the const is 
misleading.  It would be good to change it for clarity, but it might require a 
chain of consequential changes.


---

Sent from sourceforge.net because [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
  • [Lilypond-... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto

Reply via email to