Issue 4779: Accept isolated durations as music function arguments

Issue 4779: Accept isolated durations as music function arguments


Also contains commits:

Use new arguments of make_duration


Let make_duration accept preexisting durations and factor


Let make_music_from_simple accept numbers/durations


parser.yy: factor out "multipliers" from "multiplied_duration"


parser.yy: Remove redundant duration_length

[git, 2016-02-28 07:26:06]

Is it OK to use tabs for indentation?
It looks like whenever the original code did use them the modified code also
uses them, even for subsequently added lines, while any *new* lines that don't
refer to existing code are formatted correctly using spaces.

(I can't comment on the code itself, though)

[dak, 2016-02-28 08:57:33]

On 2016/02/28 07:26:06, git wrote:
> Is it OK to use tabs for indentation?
> It looks like whenever the original code did use them the modified code also
> uses them, even for subsequently added lines, while any *new* lines that don't
> refer to existing code are formatted correctly using spaces.
> 
> (I can't comment on the code itself, though)

See the comment at the start of lily/parser.yy.

/* Mode and indentation are at best a rough approximation based on TAB
 * formatting (reasonable for compatibility with unspecific editor
 * modes as Bison modes are hard to find) and need manual correction
 * frequently.  Without a reasonably dependable way of formatting a
 * Bison file sensibly, there is little point in trying to fix the
 * inconsistent state of indentation.
 */

However, it does not make sense that significant additions don't use TAB here. 
It is my guess that they stem from copy&paste operations or some over-eager
magic at work: perhaps Emacs copies the indentation style from surrounding
passages?

I'll take a look and see whether the current change can be considered the
minimal amount of extra mess, but won't put up a new review just for the
spacing.

[dak, 2016-02-28 09:37:17]

On 2016/02/28 08:57:33, dak wrote:
> On 2016/02/28 07:26:06, git wrote:
> > Is it OK to use tabs for indentation?
> > It looks like whenever the original code did use them the modified code also
> > uses them, even for subsequently added lines, while any *new* lines that
don't
> > refer to existing code are formatted correctly using spaces.

> I'll take a look and see whether the current change can be considered the
> minimal amount of extra mess, but won't put up a new review just for the
> spacing.

Ok, I have no idea how Rietveld chooses to highlight tabs, but every single
changed _or_ added line uses TABS consistently and exclusively for indentation. 
The raw diff should show this.  No idea why the HTML diff sometimes does, and
sometimes doesn't.

So there is nothing I need to change in that respect.

[HoangMinh, 2016-02-28 10:06:18]

On 2016/02/28 09:37:17, dak wrote:
> <font><font>Ngày 2016/02/28 08:57:33, dak đã
viết:</font></font><font></font><font><font>
> > Bật 2016/02/28 07:26:06, git đã viết:</font></font><font></font><font><font>
> >> Is it OK để sử dụng các tab để thụt đầu
dòng?</font></font><font></font><font><font>
> >> Có vẻ như bất cứ khi nào các mã ban đầu đã sử dụng chúng mã đổi
cũng</font></font><font></font><font><font>
> >> Sử dụng chúng, ngay cả đối với các dòng sau đó thêm vào, trong khi bất kỳ *
đường * mới</font></font><font></font><font><font>
> đừng</font></font><font></font><font><font>
> >> Tham khảo bộ luật hiện được cấu trúc định dạng sử dụng một cách chính xác
không gian.</font></font><font></font>
> <font></font><font><font>
> > Tôi sẽ có một cái nhìn và xem liệu các thay đổi hiện nay có thể được coi
là</font></font><font></font><font><font>
> > Số tiền tối thiểu của rối thêm, nhưng sẽ không đưa ra một đánh giá mới chỉ
cho</font></font><font></font><font><font>
> > Khoảng cách.</font></font><font></font>
> <font></font><font><font>
> Ok, tôi không có ý tưởng như thế nào Rietveld chọn để làm nổi bật các tab,
nhưng mỗi đơn</font></font><font></font><font><font>
> thay đổi thêm dòng _or_ sử dụng TABS thống nhất và độc quyền cho indentation.
</font></font><font></font><font><font>
> Diff liệu cần hiển thị này. </font><font>Không có ý tưởng tại sao diff HTML
đôi khi không, và</font></font><font></font><font><font>
> đôi khi không.</font></font><font></font>
> <font></font><font><font>
> Vì vậy, không có gì là tôi cần phải thay đổi trong đó tôn trọng.</font></font>