On 2012/04/14 15:21:26, Milimetr88 wrote:
>
I didn't expected so long reply. The whole patch is about adding some
comments to the code, splitting *one* function in two

But you split the function in two at an arbitrary point in the middle,
package all of its local variables from the first half into
a class structure, and then pass this class structure into the second
half of the function.

Where is the point?  Is either half of the function used for some other
purpose as well?  _Could_ it even be used for some other purpose?  What
is the purpose of creating this function border and parameter-passing
API?

http://codereview.appspot.com/5975054/

_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to