code review 1961046: freetype-go: update examples for draw.Rect being rename...

freetype-go: update examples for draw.Rect being renamed to image.Rect.

[nigeltao, 2010-08-26 08:00:33]

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

I'd like you to review this change.

[r, 2010-08-26 09:05:11]

Lgtm

On Thursday, August 26, 2010,  <nigeltao@golang.org> wrote:
> Reviewers: r,
>
> Message:
> Hello r (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> freetype-go: update examples for draw.Rect being renamed to image.Rect.
>
> Please review this at http://codereview.appspot.com/1961046/
>
> Affected files:
>   M example/freetype/main.go
>   M example/gamma/main.go
>   M example/raster/main.go
>
>
> Index: example/freetype/main.go
> ===================================================================
> --- a/example/freetype/main.go
> +++ b/example/freetype/main.go
> @@ -88,7 +88,7 @@
>                 ruler = image.RGBAColor{0x22, 0x22, 0x22, 0xff}
>         }
>         rgba := image.NewRGBA(640, 480)
> -       draw.Draw(rgba, draw.Rect(0, 0, 640, 480), bg, draw.ZP)
> +       draw.Draw(rgba, rgba.Bounds(), bg, image.ZP)
>         c := freetype.NewContext()
>         c.SetDPI(*dpi)
>         c.SetFont(font)
> Index: example/gamma/main.go
> ===================================================================
> --- a/example/gamma/main.go
> +++ b/example/gamma/main.go
> @@ -51,8 +51,8 @@
>                 h = 200
>         )
>         rgba := image.NewRGBA(w, h)
> -       draw.Draw(rgba, draw.Rect(0, 0, w, h/2), image.Black, draw.ZP)
> -       draw.Draw(rgba, draw.Rect(0, h/2, w, h), image.White, draw.ZP)
> +       draw.Draw(rgba, image.Rect(0, 0, w, h/2), image.Black, image.ZP)
> +       draw.Draw(rgba, image.Rect(0, h/2, w, h), image.White, image.ZP)
>         mask := image.NewAlpha(50, 50)
>         painter := raster.NewAlphaPainter(mask)
>         painter.Op = draw.Src
> @@ -61,9 +61,9 @@
>                 clear(mask)
>                 r.Rasterize(raster.NewGammaCorrectionPainter(painter, g))
>                 x, y := 50*i+25, 25
> -               draw.DrawMask(rgba, draw.Rect(x, y, x+50, y+50), image.White,
draw.ZP, mask, draw.ZP, draw.Over)
> +               draw.DrawMask(rgba, image.Rect(x, y, x+50, y+50), image.White,
image.ZP, mask, image.ZP, draw.Over)
>                 y += 100
> -               draw.DrawMask(rgba, draw.Rect(x, y, x+50, y+50), image.Black,
draw.ZP, mask, draw.ZP, draw.Over)
> +               draw.DrawMask(rgba, image.Rect(x, y, x+50, y+50), image.Black,
image.ZP, mask, image.ZP, draw.Over)
>         }
>
>         // Save that RGBA image to disk.
> Index: example/raster/main.go
> ===================================================================
> --- a/example/raster/main.go
> +++ b/example/raster/main.go
> @@ -148,8 +148,8 @@
>         // Draw the mask image (in gray) onto an RGBA image.
>         rgba := image.NewRGBA(w, h)
>         gray := image.ColorImage{image.AlphaColor{0x1f}}
> -       draw.Draw(rgba, draw.Rect(0, 0, w, h), image.Black, draw.ZP)
> -       draw.DrawMask(rgba, draw.Rect(0, 0, w, h), gray, draw.ZP, mask,
draw.ZP, draw.Over)
> +       draw.Draw(rgba, rgba.Bounds(), image.Black, image.ZP)
> +       draw.DrawMask(rgba, rgba.Bounds(), gray, image.ZP, mask, image.ZP,
draw.Over)
>         showNodes(rgba, outside)
>         showNodes(rgba, inside)
>
>
>
>

[nigeltao, 2010-08-27 00:50:53]

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

freetype-go: update examples for draw.Rect being renamed to image.Rect.

R=r
CC=golang-dev
http://codereview.appspot.com/1961046