[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Glyphs for Kievan Notation (issue 4951062)
From: |
lemzwerg |
Subject: |
Re: Glyphs for Kievan Notation (issue 4951062) |
Date: |
Mon, 19 Sep 2011 05:02:32 +0000 |
LGTM
http://codereview.appspot.com/4951062/diff/24001/mf/feta-kievan.mf
File mf/feta-kievan.mf (right):
http://codereview.appspot.com/4951062/diff/24001/mf/feta-kievan.mf#newcode52
mf/feta-kievan.mf:52: fill z1{dir 8.6} .. z2 .. z3
A minor thing: Please align this vertically (and at similar places
also); this is, the indentation here consists of a tab and some space
characters:
fill z1 .. z2 ..z3
& z4 .. z5 .. z6
...
Maybe you call this weird, but it helps reading the code, at least for
me :-)
http://codereview.appspot.com/4951062/
- Re: Glyphs for Kievan Notation (issue 4951062), (continued)
- Re: Glyphs for Kievan Notation (issue 4951062), Aleksandr Andreev, 2011/09/22
- Re: Glyphs for Kievan Notation (issue 4951062), Aleksandr Andreev, 2011/09/21
- Re: Glyphs for Kievan Notation (issue 4951062), Peekay Ex, 2011/09/21
- Re: Glyphs for Kievan Notation (issue 4951062), Janek Warchoł, 2011/09/21
- Re: Glyphs for Kievan Notation (issue 4951062), Aleksandr Andreev, 2011/09/21
- Re: Glyphs for Kievan Notation (issue 4951062), Reinhold Kainhofer, 2011/09/21
Re: Glyphs for Kievan Notation (issue 4951062), aleksandr . andreev, 2011/09/18
Re: Glyphs for Kievan Notation (issue 4951062),
lemzwerg <=
Re: Glyphs for Kievan Notation (issue 4951062), aleksandr . andreev, 2011/09/19
Re: Glyphs for Kievan Notation (issue 4951062), aleksandr . andreev, 2011/09/21
Re: Glyphs for Kievan Notation (issue 4951062), aleksandr . andreev, 2011/09/23
Re: Glyphs for Kievan Notation (issue 4951062), bordage . bertrand, 2011/09/24
Re: Glyphs for Kievan Notation (issue 4951062), aleksandr . andreev, 2011/09/27