Status: Started
Owner: [email protected]
Labels: Type-Enhancement Patch-new

New issue 4135 by [email protected]: Patch: Replace mark_smob static member functions with non-static members
https://code.google.com/p/lilypond/issues/detail?id=4135

Replace mark_smob static member functions with non-static members

Since the mark functions generally need to access a number of members,
it is quite more convenient if the mark functions are proper non-static
member functions.


Also contains commit:

Don't use ASSERT_LIVE_IS_ALLOWED in simple smobs

When replacing mark_smob with a proper member function, use of
ASSERT_LIVE_IS_ALLOWED would require access to self_scm () or similar,
and simple smobs don't link back to their SCM cell.  This only
concerns Skyline and Context_mod, so the loss is not all that
large.

http://codereview.appspot.com/148230043

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

Reply via email to