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

Issue 288290043: Define French as a separate input language (Closed)

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

Description

Define French as a separate input language This patch gives French users the ability to enter the D pitch as "ré" instead of Italian-like "re". In addition, the French input language accepts double sharps entered using the -x suffix.

Patch Set 1 #

Total comments: 2

Patch Set 2 : Remove typo (fix doc build). #

Patch Set 3 : Use current LILYPOND_VERSION in regtest. #

Total comments: 6

Patch Set 4 : Remove "francais" #

Unified diffs Side-by-side diffs Delta from patch set Stats (+136 lines, -10 lines) Patch
M Documentation/changes.tely View 1 1 chunk +7 lines, -0 lines 0 comments Download
M Documentation/notation/pitches.itely View 1 2 3 4 chunks +10 lines, -4 lines 0 comments Download
M input/regression/note-names.ly View 1 2 2 chunks +4 lines, -3 lines 0 comments Download
M python/musicexp.py View 1 2 3 2 chunks +6 lines, -1 line 0 comments Download
M scm/define-note-names.scm View 1 2 3 2 chunks +109 lines, -2 lines 0 comments Download

Messages

Total messages: 18
simon.albrecht
https://codereview.appspot.com/288290043/diff/1/Documentation/notation/pitches.itely File Documentation/notation/pitches.itely (right): https://codereview.appspot.com/288290043/diff/1/Documentation/notation/pitches.itely#newcode492 Documentation/notation/pitches.itely:492: @multitable {@code{nederlands}} {do re/re mi fa sol la sib ...
8 years, 2 months ago (2016-02-22 20:46:57 UTC) #1
Valentin Villenave
On 2016/02/22 20:46:57, simon.albrecht wrote: > https://codereview.appspot.com/288290043/diff/1/Documentation/notation/pitches.itely > File Documentation/notation/pitches.itely (right): > > https://codereview.appspot.com/288290043/diff/1/Documentation/notation/pitches.itely#newcode492 > ...
8 years, 2 months ago (2016-02-22 22:30:46 UTC) #2
pkx166h
This won't get tested until there is a tracker. James
8 years, 2 months ago (2016-02-23 12:56:06 UTC) #3
pkx166h
On 2016/02/23 12:56:06, pkx166h wrote: > This won't get tested until there is a tracker. ...
8 years, 2 months ago (2016-02-23 12:58:18 UTC) #4
pkx166h
Fails make doc. https://codereview.appspot.com/288290043/diff/1/Documentation/changes.tely File Documentation/changes.tely (right): https://codereview.appspot.com/288290043/diff/1/Documentation/changes.tely#newcode70 Documentation/changes.tely:70: @end itemize This '@end itemize' breaks ...
8 years, 2 months ago (2016-02-23 14:36:41 UTC) #5
Valentin Villenave
Remove typo (fix doc build).
8 years, 2 months ago (2016-02-23 18:04:16 UTC) #6
Valentin Villenave
Use current LILYPOND_VERSION in regtest.
8 years, 2 months ago (2016-02-23 19:24:25 UTC) #7
dak
It seems to me (after glancing over it) like the last Rietveld review was just ...
8 years, 2 months ago (2016-02-28 12:18:29 UTC) #8
Valentin Villenave
On 2016/02/28 12:18:29, dak wrote: > It seems to me (after glancing over it) like ...
8 years, 1 month ago (2016-02-28 19:43:06 UTC) #9
Valentin Villenave
Remove "francais"
8 years, 1 month ago (2016-02-28 19:43:47 UTC) #10
dak
Thanks. I really think we are better off without "francais". I think the original motivation ...
8 years, 1 month ago (2016-02-28 21:01:50 UTC) #11
pacovila
2016-02-28 22:01 GMT+01:00 <dak@gnu.org>: > Thanks. I really think we are better off without "francais". ...
8 years, 1 month ago (2016-02-28 21:52:34 UTC) #12
dak
Francisco Vila <paconet.org@gmail.com> writes: > 2016-02-28 22:01 GMT+01:00 <dak@gnu.org>: >> Thanks. I really think we ...
8 years, 1 month ago (2016-02-28 22:12:19 UTC) #13
Valentin Villenave
On Sun, Feb 28, 2016 at 11:07 PM, David Kastrup <dak@gnu.org> wrote: > I haven't ...
8 years, 1 month ago (2016-02-28 22:52:52 UTC) #14
pacovila
El 28/02/2016 23:12, "David Kastrup" <dak@gnu.org> escribió: > > Francisco Vila <paconet.org@gmail.com> writes: > > ...
8 years, 1 month ago (2016-02-28 23:07:23 UTC) #15
dak
Valentin Villenave <v.villenave@gmail.com> writes: > On Sun, Feb 28, 2016 at 11:07 PM, David Kastrup ...
8 years, 1 month ago (2016-02-28 23:33:03 UTC) #16
Valentin Villenave
On Mon, Feb 29, 2016 at 12:32 AM, David Kastrup <dak@gnu.org> wrote: > Seriously? I ...
8 years, 1 month ago (2016-02-29 01:04:00 UTC) #17
dak
8 years, 1 month ago (2016-02-29 10:28:48 UTC) #18
Valentin Villenave <v.villenave@gmail.com> writes:

> On Mon, Feb 29, 2016 at 12:32 AM, David Kastrup <dak@gnu.org> wrote:
>> Seriously?  I mean, no German would think about leaving off umlauts from
>> a letter under any circumstance
>
> Well, in German it can alter the meaning (or at least distinguish
> between plural and singular). In French it’s a bit rarer. (Although
> when speaking out loud, nobody would say "Frankais" instead of
> "Français", which is what you get when you remove the cédille.)
>
> That being said, I still very much do take issue with the German
> reform that tends to replace "ich muß" with "ich muss" :-)
>
>> So one can forget "ç" in French?  Wow.  How did that come about?
>
> Well, I’m dealing with very computer-illiterate people: retired people
> who didn’t use a computer at work, younger people whose only dealings
> with written French come from hasty SMS texting...

Oh come on.  Particularly the computer illiterate will know how to work
a typewriter.  French typewriters are not reduced to ASCII (and even if
they are, you can overprint a comma).  And particularly with "hasty SMS
texting" the completions provided by T9 and its ilk tend to add the
diacritics on its own (as long as it provides the word at all, unlike
"suspekt -> rüsselü", and that still is diacritically enriched).

Well, I'm coming from a TeX tradition.  Donald Knuth actually related in
a talk that he picked a longer way to work explicitly in order to avoid
having a typographically awful road sign kill his mood for the day every
morning.

-- 
David Kastrup
Sign in to reply to this message.

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