Re: [Lilypond-auto] Issue 3531 in lilypond: Patch: replaced function argument 'string' by 'const string&' where it makes sense to avoid unnecessary copying of 'string' objects.

[lilypond]

Comment #6 on issue 3531 by address@hidden: Patch: replaced  
function argument 'string' by 'const string&' where it makes sense to avoid  
unnecessary copying of 'string' objects.
http://code.google.com/p/lilypond/issues/detail?id=3531

> although I am not completely sure what, if any
> significance the comment by Keith/Frederic in
> the Rietveld

It means that at one location, how the code was written before my patch  
could let think the programmer wanted to change the argument (by reference)  
but the argument was given by value (copy). However, I do not think this is  
the case. Nevertheless, I did not change the behaviour of the function so  
if there was a bug there, the bug remains. If there was no bug, it should  
still be OK after applying my patch.

--
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