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

Issue 239930043: Add French-specific note names (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 11 months ago by Valentin Villenave
Modified:
8 years, 1 month ago
Reviewers:
Keith, pacovila, dak, t.daniels
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Add French-specific note names French users may (and do) want to enter the D pitch as "ré" instead of Italian-like "re". This also brings the -x double-sharp suffix to French input language (an unrelated but welcome addition).

Patch Set 1 #

Total comments: 4

Patch Set 2 : Add -x suffix in italiano; "ré" comes before "re" in French #

Patch Set 3 : Add convert rules #

Total comments: 6
Unified diffs Side-by-side diffs Delta from patch set Stats (+172 lines, -13 lines) Patch
M Documentation/changes.tely View 1 2 2 chunks +21 lines, -1 line 3 comments Download
M Documentation/notation/pitches.itely View 1 2 4 chunks +14 lines, -8 lines 0 comments Download
M input/regression/note-names.ly View 1 2 2 chunks +4 lines, -3 lines 0 comments Download
M python/convertrules.py View 1 2 1 chunk +10 lines, -0 lines 1 comment Download
M python/musicexp.py View 2 chunks +5 lines, -0 lines 0 comments Download
M scm/define-note-names.scm View 1 2 9 chunks +118 lines, -1 line 2 comments Download

Messages

Total messages: 22
Valentin Villenave
Greetings everybody, here’s an itch I’ve been wanting to scratch for the past decade or ...
8 years, 11 months ago (2015-05-17 08:23:05 UTC) #1
dak
v.villenave@gmail.com writes: > Reviewers: , > > Message: > Greetings everybody, > here’s an itch ...
8 years, 11 months ago (2015-05-17 08:39:00 UTC) #2
Valentin Villenave
On 2015/05/17 08:39:00, dak wrote: > Well, with notenames like "re" it should have been ...
8 years, 11 months ago (2015-05-17 09:44:52 UTC) #3
dak
On 2015/05/17 09:44:52, Valentin Villenave wrote: > On 2015/05/17 08:39:00, dak wrote: > > Well, ...
8 years, 11 months ago (2015-05-17 10:38:24 UTC) #4
dak
https://codereview.appspot.com/239930043/diff/1/scm/define-note-names.scm File scm/define-note-names.scm (right): https://codereview.appspot.com/239930043/diff/1/scm/define-note-names.scm#newcode566 scm/define-note-names.scm:566: (rébb . ,(ly:make-pitch -1 1 DOUBLE-FLAT)) It would seem ...
8 years, 11 months ago (2015-05-18 11:07:01 UTC) #5
Valentin Villenave
Add -x suffix in italiano; "ré" comes before "re" in French
8 years, 11 months ago (2015-05-18 11:37:32 UTC) #6
dak
https://codereview.appspot.com/239930043/diff/1/Documentation/notation/pitches.itely File Documentation/notation/pitches.itely (right): https://codereview.appspot.com/239930043/diff/1/Documentation/notation/pitches.itely#newcode492 Documentation/notation/pitches.itely:492: @item @code{francais} or @code{français} Please don't introduce "francais". If ...
8 years, 11 months ago (2015-05-18 12:09:18 UTC) #7
Valentin Villenave
https://codereview.appspot.com/239930043/diff/1/Documentation/notation/pitches.itely File Documentation/notation/pitches.itely (right): https://codereview.appspot.com/239930043/diff/1/Documentation/notation/pitches.itely#newcode492 Documentation/notation/pitches.itely:492: @item @code{francais} or @code{français} On 2015/05/18 12:09:18, dak wrote: ...
8 years, 11 months ago (2015-05-18 17:08:20 UTC) #8
dak
On 2015/05/18 17:08:20, Valentin Villenave wrote: > https://codereview.appspot.com/239930043/diff/1/Documentation/notation/pitches.itely > File Documentation/notation/pitches.itely (right): > > https://codereview.appspot.com/239930043/diff/1/Documentation/notation/pitches.itely#newcode492 ...
8 years, 11 months ago (2015-05-18 17:38:22 UTC) #9
Valentin Villenave
Add convert rules
8 years, 11 months ago (2015-05-18 18:12:59 UTC) #10
Valentin Villenave
On 2015/05/18 17:38:22, dak wrote: > It does not have non-ASCII notenames, so you might ...
8 years, 11 months ago (2015-05-18 18:20:20 UTC) #11
dak
On 2015/05/18 18:20:20, Valentin Villenave wrote: > On 2015/05/18 17:38:22, dak wrote: > > It ...
8 years, 11 months ago (2015-05-18 19:09:56 UTC) #12
pacovila
El 18/05/2015 20:20, <v.villenave@gmail.com> escribió: > > On 2015/05/18 17:38:22, dak wrote: >> >> It ...
8 years, 11 months ago (2015-05-18 19:20:23 UTC) #13
Valentin Villenave
On Mon, May 18, 2015 at 9:20 PM, Francisco Vila <paconet.org@gmail.com> wrote: > Besides, this ...
8 years, 11 months ago (2015-05-18 19:41:04 UTC) #14
dak
Valentin Villenave <v.villenave@gmail.com> writes: > On Mon, May 18, 2015 at 9:20 PM, Francisco Vila ...
8 years, 11 months ago (2015-05-18 19:52:39 UTC) #15
Valentin Villenave
On Mon, May 18, 2015 at 9:52 PM, David Kastrup <dak@gnu.org> wrote: > We are ...
8 years, 11 months ago (2015-05-18 20:51:28 UTC) #16
dak
Valentin Villenave <v.villenave@gmail.com> writes: > So, my guess would be that we *could* start recommending ...
8 years, 11 months ago (2015-05-19 04:51:47 UTC) #17
t.daniels_treda.co.uk
Valentin Villenave wrote Monday, May 18, 2015 8:41 PM >> BTW dear lazylist: where do ...
8 years, 11 months ago (2015-05-19 21:12:37 UTC) #18
Valentin Villenave
On Tue, May 19, 2015 at 11:12 PM, Trevor Daniels <t.daniels@treda.co.uk> wrote: > on my ...
8 years, 11 months ago (2015-05-19 22:12:51 UTC) #19
pacovila
El 19/05/2015 23:12, "Trevor Daniels" <t.daniels@treda.co.uk> escribió: > > > Valentin Villenave wrote Monday, May ...
8 years, 11 months ago (2015-05-19 22:13:42 UTC) #20
dak
Valentin Villenave <v.villenave@gmail.com> writes: > On Tue, May 19, 2015 at 11:12 PM, Trevor Daniels ...
8 years, 11 months ago (2015-05-20 10:00:31 UTC) #21
Keith
8 years, 11 months ago (2015-05-25 21:29:11 UTC) #22
There seems to be a way to do this as a simple expansion of capabilities, so you
do need to change existing input files.  Have \language"français" accept 'ré',
and also 're' for backward compatibility with the former use of Italian names.

https://codereview.appspot.com/239930043/diff/40001/Documentation/changes.tely
File Documentation/changes.tely (right):

https://codereview.appspot.com/239930043/diff/40001/Documentation/changes.tel...
Documentation/changes.tely:70: of the @code{\language} command.
I suggest showing the complete @code{@w{\language "italiano"})

https://codereview.appspot.com/239930043/diff/40001/Documentation/changes.tel...
Documentation/changes.tely:74: generic Italian-derived syntax, the @var{d} pitch
may be
In English the pitch names are capital and upright: "the pitch D may be"

https://codereview.appspot.com/239930043/diff/40001/Documentation/changes.tel...
Documentation/changes.tely:81: deprecated in favor of @code{"español"} and
@code{"français"}
I think we should keep "espanol" because the Castañedas and ÓHaras and Müllers
of the world sometimes have to use systems that work best if we keep our text
files within ASCII.
There is no reason to deprecate "francais" because we do not currently have it.

https://codereview.appspot.com/239930043/diff/40001/python/convertrules.py
File python/convertrules.py (right):

https://codereview.appspot.com/239930043/diff/40001/python/convertrules.py#ne...
python/convertrules.py:3753: str = re.sub ('\\\\language\s*"francais"',
'\\\\language "français"', str)
We should not do an automatic conversion from ASCII to an expanded character
set.

https://codereview.appspot.com/239930043/diff/40001/scm/define-note-names.scm
File scm/define-note-names.scm (right):

https://codereview.appspot.com/239930043/diff/40001/scm/define-note-names.scm...
scm/define-note-names.scm:543: (francais . (
You can have here français

https://codereview.appspot.com/239930043/diff/40001/scm/define-note-names.scm...
scm/define-note-names.scm:1077: ;; add two native utf-8 aliases. Pairs obey
cp-like order: '(old new)
"add one native utf-8 alias"

The new français does not need an alias
Sign in to reply to this message.

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