code review 982044: Freetype-Go: Add Luxi .ttf fonts, and the human-readabl...

Freetype-Go: Add Luxi .ttf fonts, and the human-readable .ttx equivalents.

All up this is about 10 MiB of data, about 90% of which is the .ttx XML
versions.

[nigeltao, 2010-04-27 12:30:04]

Hello r, rsc (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[r, 2010-04-27 16:35:02]

LGTM but rsc understands the licensing issues better so wait for him.

[rsc, 2010-04-27 17:08:58]

LGTM

What do you plan to use the ttx for?
Is it worth having both ttf and ttx when there's a tool to generate ttx?

[nigeltao_gnome, 2010-04-27 23:25:09]

On 28 April 2010 03:08, Russ Cox <rsc@golang.org> wrote:
> What do you plan to use the ttx for?
> Is it worth having both ttf and ttx when there's a tool to generate ttx?

Eventually it will be for unit tests, the same way
$GOROOT/src/pkg/image/png/testdata/pngsuite/*.sng is currently used.

[rsc, 2010-04-27 23:45:14]

LGTM


On Tue, Apr 27, 2010 at 16:25, Nigel Tao <nigel.tao.gnome@gmail.com> wrote:
> On 28 April 2010 03:08, Russ Cox <rsc@golang.org> wrote:
>> What do you plan to use the ttx for?
>> Is it worth having both ttf and ttx when there's a tool to generate ttx?
>
> Eventually it will be for unit tests, the same way
> $GOROOT/src/pkg/image/png/testdata/pngsuite/*.sng is currently used.
>

[nigeltao, 2010-04-28 12:29:47]

*** Abandoned ***

[nigeltao_gnome, 2010-04-28 12:32:10]

On 28 April 2010 22:29,  <nigeltao@golang.org> wrote:
> *** Abandoned ***
>
> http://codereview.appspot.com/982044/show

I think this change was too large for the codereview extension to
handle, even after I trimmed it down from 12 fonts to 3, so I hg
pushed it manually.