code review 1261043: Freetype-Go: Paint now takes an explicit done bool inst...

Freetype-Go: Paint now takes an explicit done bool instead of using a
magic zero value to indicate end of painting.

This means that Painter implementations can assume thant the Spans' Y
values are monotonic.

[nigeltao, 2010-05-21 19:55:07]

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

I'd like you to review this change.

[rsc, 2010-05-21 21:46:59]

LGTM

Another way to do this would have been
to define that Paint(nil) means "done".
Either is fine.

Russ

[nigeltao, 2010-05-21 22:09:31]

*** Submitted as
http://code.google.com/p/freetype-go/source/detail?r=759756f8db37 ***

Freetype-Go: Paint now takes an explicit done bool instead of using a
magic zero value to indicate end of painting.

This means that Painter implementations can assume thant the Spans' Y
values are monotonic.

R=rsc
CC=golang-dev, rog
http://codereview.appspot.com/1261043