https://codereview.appspot.com/22120043/diff/40001/lily/parser.yy
File lily/parser.yy (right):

https://codereview.appspot.com/22120043/diff/40001/lily/parser.yy#newcode649
lily/parser.yy:649: | FRACTION
On 2013/11/09 17:12:52, Trevor Daniels wrote:
I don't like the mixed tabs and spaces here

There are no mixed tabs and spaces in the patch or its immediate
context.  A few lines later there is a spaces-only line, probably
introduced during a time where LilyPond globally configured spaces-only
mode for Emacs and this file did not yet contain a use-tabs override (as
the Bison file is indented in a tabs-style mode customary for such
files).

I don't think it is a good idea to reformat unrelated stuff in this
patch.

https://codereview.appspot.com/22120043/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to