Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1994)

Issue 7501046: Harmonize point-and-click information for #xxx and $xxx (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 1 month ago by dak
Modified:
11 years ago
Reviewers:
janek
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Harmonize point-and-click information for #xxx and $xxx Both #xxx and $xxx retain any preexisting origin information when producing a music expression. However, \xxx always receives the origin information pertaining to the current location even if previous point-and-click information exists. This is consistent with the point-and-click information for music functions as a whole corresponding to the point of call, while the individual arguments, if separate music expressions, retain any preexisting origin.

Patch Set 1 #

Patch Set 2 : Correct several confusions about evaluation timing #

Patch Set 3 : Testing that this actually compiles would have been good #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -9 lines) Patch
M lily/lexer.ll View 1 2 3 chunks +23 lines, -5 lines 0 comments Download
M lily/parser.yy View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 9
dak
Correct several confusions about evaluation timing
11 years, 1 month ago (2013-03-12 14:23:39 UTC) #1
dak
Testing that this actually compiles would have been good
11 years, 1 month ago (2013-03-12 14:29:56 UTC) #2
janek
As usual, i have a dumb question about the commit message. What is the relation ...
11 years, 1 month ago (2013-03-12 22:12:34 UTC) #3
dak
On 2013/03/12 22:12:34, janek wrote: > As usual, i have a dumb question about the ...
11 years, 1 month ago (2013-03-13 06:06:46 UTC) #4
janek
On Wed, Mar 13, 2013 at 7:06 AM, <dak@gnu.org> wrote: > On 2013/03/12 22:12:34, janek ...
11 years, 1 month ago (2013-03-13 07:03:16 UTC) #5
dak
On 2013/03/13 07:03:16, janek wrote: > On Wed, Mar 13, 2013 at 7:06 AM, <mailto:dak@gnu.org> ...
11 years, 1 month ago (2013-03-13 08:11:52 UTC) #6
janek
On Wed, Mar 13, 2013 at 9:11 AM, <dak@gnu.org> wrote: > I don't get it. ...
11 years, 1 month ago (2013-03-13 15:44:20 UTC) #7
dak
On 2013/03/13 15:44:20, janek wrote: > On Wed, Mar 13, 2013 at 9:11 AM, <mailto:dak@gnu.org> ...
11 years, 1 month ago (2013-03-13 15:56:58 UTC) #8
janek
11 years, 1 month ago (2013-03-13 16:12:14 UTC) #9
On Wed, Mar 13, 2013 at 4:56 PM,  <dak@gnu.org> wrote:
> On 2013/03/13 15:44:20, janek wrote:
>> Ok, now i see what you mean.
>> I think my problem was that, because of "however", i had thought that
>> in the description you contrast some new behaviour with some other
>> behaviour that existed already.  As in "this patch makes $xxx and #xxx
>> do blah.  However, \xxx won't do blah - it continues to do foo as it
>> used to".
>
> Sigh.  "However," and "In contrast," mean _exactly_ the same.
> Besides, \xxx is indeed untouched in behavior and continues to do foo
> as it used to.  It is now special-cased (while it shared code and
> behavior with $ previously) but indeed, the behavior for it is exactly
> the same as before.

:/  I was afraid there was something i missed.  I apologize for being
so dimwitted.
Nevertheless, i think that using imperative for commit messages is a
good idea anyway.

Janek
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b