[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Add regtest for issue 5181 (issue 327470043 by address@hidden)
From: |
thomasmorley65 |
Subject: |
Re: Add regtest for issue 5181 (issue 327470043 by address@hidden) |
Date: |
Mon, 25 Sep 2017 14:28:50 -0700 |
Very nice, i.e LGTM
Not sure whether this should be reflected by a reg-test, but currently
there is no case in it triggering the warning like:
warns = {
\override NoteHead.color = #red
c'1 d'
\revert NoteHead.color
}
{ \warns -1 e' }
And there are cases which will probably never work, like:
fails = {
d'-\tweak color #red
%% with or without \etc
\etc
}
{ \fails -1 e' }
Not sure if we should mention it in the docs at all, or in "Known issues
and warnings" or elsewhere ...?
https://codereview.appspot.com/327470043/