code review 5505067: freetype: import path gofixes.

freetype: import path gofixes.

Also update freetype/truetype for s/int/rune/.

[nigeltao, 2011-12-22 23:37:26]

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

I'd like you to review this change to
https://code.google.com/p/freetype-go

[bradfitz, 2011-12-22 23:40:28]

LGTM at least

On Thu, Dec 22, 2011 at 3:37 PM, <nigeltao@golang.org> wrote:

> Reviewers: rsc,
>
> Message:
> Hello rsc@golang.org (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change to
> https://code.google.com/p/**freetype-go<https://code.google.com/p/freetype-go>
>
>
> Description:
> freetype: import path gofixes.
>
> Also update freetype/truetype for s/int/rune/.
>
> Please review this at
http://codereview.appspot.com/**5505067/<http://codereview.appspot.com/5505067/>
>
> Affected files:
>  M example/freetype/main.go
>  M example/gamma/main.go
>  M example/raster/main.go
>  M example/round/main.go
>  M example/truetype/main.go
>  M freetype/Makefile
>  M freetype/freetype.go
>  M freetype/raster/Makefile
>  M freetype/truetype/Makefile
>  M freetype/truetype/truetype.go
>
>
> Index: example/freetype/main.go
> ==============================**==============================**=======
> --- a/example/freetype/main.go
> +++ b/example/freetype/main.go
> @@ -9,7 +9,6 @@
>        "bufio"
>        "flag"
>        "fmt"
> -      
"freetype-go.googlecode.com/**hg/freetype<http://freetype-go.googlecode.com/hg/freetype>
> "
>        "image"
>        "image/color"
>        "image/draw"
> @@ -17,6 +16,8 @@
>        "io/ioutil"
>        "log"
>        "os"
> +
> +      
"code.google.com/p/freetype-**go/freetype<http://code.google.com/p/freetype-go/freetype>
> "
>  )
>
>  var (
> Index: example/gamma/main.go
> ==============================**==============================**=======
> --- a/example/gamma/main.go
> +++ b/example/gamma/main.go
> @@ -14,7 +14,7 @@
>        "log"
>        "os"
>
> -      
"freetype-go.googlecode.com/**hg/freetype/raster<http://freetype-go.googlecode.com/hg/freetype/raster>
> "
> +      
"code.google.com/p/freetype-**go/freetype/raster<http://code.google.com/p/freetype-go/freetype/raster>
> "
>  )
>
>  func p(x, y int) raster.Point {
> Index: example/raster/main.go
> ==============================**==============================**=======
> --- a/example/raster/main.go
> +++ b/example/raster/main.go
> @@ -15,7 +15,7 @@
>        "log"
>        "os"
>
> -      
"freetype-go.googlecode.com/**hg/freetype/raster<http://freetype-go.googlecode.com/hg/freetype/raster>
> "
> +      
"code.google.com/p/freetype-**go/freetype/raster<http://code.google.com/p/freetype-go/freetype/raster>
> "
>  )
>
>  type node struct {
> Index: example/round/main.go
> ==============================**==============================**=======
> --- a/example/round/main.go
> +++ b/example/round/main.go
> @@ -21,7 +21,7 @@
>        "math"
>        "os"
>
> -      
"freetype-go.googlecode.com/**hg/freetype/raster<http://freetype-go.googlecode.com/hg/freetype/raster>
> "
> +      
"code.google.com/p/freetype-**go/freetype/raster<http://code.google.com/p/freetype-go/freetype/raster>
> "
>  )
>
>  func main() {
> Index: example/truetype/main.go
> ==============================**==============================**=======
> --- a/example/truetype/main.go
> +++ b/example/truetype/main.go
> @@ -11,7 +11,7 @@
>        "io/ioutil"
>        "log"
>
> -      
"freetype-go.googlecode.com/**hg/freetype/truetype<http://freetype-go.googlecode.com/hg/freetype/truetype>
> "
> +      
"code.google.com/p/freetype-**go/freetype/truetype<http://code.google.com/p/freetype-go/freetype/truetype>
> "
>  )
>
>  var fontfile = flag.String("fontfile", "../../luxi-fonts/luxisr.ttf",
> "filename of the ttf font")
> Index: freetype/Makefile
> ==============================**==============================**=======
> --- a/freetype/Makefile
> +++ b/freetype/Makefile
> @@ -5,7 +5,7 @@
>
>  include $(GOROOT)/src/Make.inc
>
>
-TARG=freetype-go.googlecode.**com/hg/freetype<http://freetype-go.googlecode.com/hg/freetype>
>
+TARG=code.google.com/p/**freetype-go/freetype<http://code.google.com/p/freetype-go/freetype>
>  GOFILES=\
>        freetype.go\
>
> Index: freetype/freetype.go
> ==============================**==============================**=======
> --- a/freetype/freetype.go
> +++ b/freetype/freetype.go
> @@ -10,10 +10,11 @@
>
>  import (
>        "errors"
> -      
"freetype-go.googlecode.com/**hg/freetype/raster<http://freetype-go.googlecode.com/hg/freetype/raster>
> "
> -      
"freetype-go.googlecode.com/**hg/freetype/truetype<http://freetype-go.googlecode.com/hg/freetype/truetype>
> "
>        "image"
>        "image/draw"
> +
> +      
"code.google.com/p/freetype-**go/freetype/raster<http://code.google.com/p/freetype-go/freetype/raster>
> "
> +      
"code.google.com/p/freetype-**go/freetype/truetype<http://code.google.com/p/freetype-go/freetype/truetype>
> "
>  )
>
>  // These constants determine the size of the glyph cache. The cache is
> keyed
> Index: freetype/raster/Makefile
> ==============================**==============================**=======
> --- a/freetype/raster/Makefile
> +++ b/freetype/raster/Makefile
> @@ -5,7 +5,7 @@
>
>  include $(GOROOT)/src/Make.inc
>
>
-TARG=freetype-go.googlecode.**com/hg/freetype/raster<http://freetype-go.googlecode.com/hg/freetype/raster>
>
+TARG=code.google.com/p/**freetype-go/freetype/raster<http://code.google.com/p/freetype-go/freetype/raster>
>  GOFILES=\
>        geom.go\
>        paint.go\
> Index: freetype/truetype/Makefile
> ==============================**==============================**=======
> --- a/freetype/truetype/Makefile
> +++ b/freetype/truetype/Makefile
> @@ -5,7 +5,7 @@
>
>  include $(GOROOT)/src/Make.inc
>
>
-TARG=freetype-go.googlecode.**com/hg/freetype/truetype<http://freetype-go.googlecode.com/hg/freetype/truetype>
>
+TARG=code.google.com/p/**freetype-go/freetype/truetype<http://code.google.com/p/freetype-go/freetype/truetype>
>  GOFILES=\
>        truetype.go\
>
> Index: freetype/truetype/truetype.go
> ==============================**==============================**=======
> --- a/freetype/truetype/truetype.**go
> +++ b/freetype/truetype/truetype.**go
> @@ -17,7 +17,7 @@
>        "fmt"
>  )
>
> -// An Index is a Font's index of a Unicode code point.
> +// An Index is a Font's index of a rune.
>  type Index uint16
>
>  // A Bounds holds the co-ordinate range of one or more glyphs.
> @@ -287,9 +287,9 @@
>        return f.unitsPerEm
>  }
>
> -// Index returns a Font's index for the given Unicode code point.
> -func (f *Font) Index(codePoint int) Index {
> -       c := uint16(codePoint)
> +// Index returns a Font's index for the given rune.
> +func (f *Font) Index(x rune) Index {
> +       c := uint16(x)
>        n := len(f.cm)
>        for i := 0; i < n; i++ {
>                if f.cm[i].start <= c && c <= f.cm[i].end {
>
>
>

[nigeltao, 2011-12-22 23:56:03]

*** Submitted as c9c9c863abe0 ***

freetype: import path gofixes.

Also update freetype/truetype for s/int/rune/.

R=rsc, bradfitz
CC=golang-dev
http://codereview.appspot.com/5505067