code review 79320043: go.image/vp8, go.image/webp: new packages, copied from

go.image/vp8, go.image/webp: new packages, copied from
code.google.com/p/vp8-go

This is a straight copy, followed by import path updates.

[nigeltao, 2014-03-24 05:48:55]

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

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

[gobot, 2014-03-24 05:51:49]

Replacing golang-dev with golang-codereviews.

To the author of this CL: 

If you are using 'hg mail -r golang-dev' to mail the CL, use simply 'hg mail'
instead.

If you did not name golang-dev explicitly and it was still added to the CL,
it means your working copy of the repo has a stale codereview.cfg
(or lib/codereview/codereview.cfg).
Please run 'hg sync' to update your client to the most recent codereview.cfg.
If the most recent codereview.cfg has defaultcc set to golang-dev instead of
golang-codereviews, please send a CL correcting it.

Thanks very much.

[aam, 2014-03-24 05:56:18]

this line should change at some point:

On Sun, Mar 23, 2014 at 11:48 PM,  <nigeltao@golang.org> wrote:
> Reviewers: r,
>
> Message:
> Hello r@golang.org (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change to
> https://code.google.com/p/go.image
>
>
> Description:
> go.image/vp8, go.image/webp: new packages, copied from
> code.google.com/p/vp8-go
>
> This is a straight copy, followed by import path updates.
>
> Please review this at https://codereview.appspot.com/79320043/
>
> Affected files (+2321, -0 lines):
>   A vp8/decode.go
>   A vp8/idct.go
>   A vp8/partition.go
>   A vp8/pred.go
>   A vp8/predfunc.go
>   A vp8/quant.go
>   A vp8/reconstruct.go
>   A vp8/token.go
>   A webp/decode.go
>
>
> --
>
> ---You received this message because you are subscribed to the Google Groups
> "golang-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-dev+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

[aam, 2014-03-24 05:58:09]

foiled by shift+enter, sorry. this is what i referenced:

> goinstall code.google.com/p/vp8-go/webp

it is from the summary at code.google.com/p/vp8-go/

[nigeltao, 2014-03-24 06:20:42]

On Mon, Mar 24, 2014 at 4:58 PM, andrey mirtchovski
<mirtchovski@gmail.com> wrote:
> foiled by shift+enter, sorry. this is what i referenced:
>
>> goinstall code.google.com/p/vp8-go/webp
>
> it is from the summary at code.google.com/p/vp8-go/

Yes, winding down the old code.google.com project will be a separate action.

[oleku, 2014-03-24 07:34:25]

Nice .. any plans for encoder ? 

On Monday, March 24, 2014 7:20:41 AM UTC+1, Nigel Tao wrote:
>
> On Mon, Mar 24, 2014 at 4:58 PM, andrey mirtchovski 
> <mirtc...@gmail.com <javascript:>> wrote: 
> > foiled by shift+enter, sorry. this is what i referenced: 
> > 
> >> goinstall code.google.com/p/vp8-go/webp 
> > 
> > it is from the summary at code.google.com/p/vp8-go/ 
>
> Yes, winding down the old code.google.com project will be a separate 
> action. 
>

[r, 2014-03-24 09:12:38]

LGTM but should you change the copyright string before checkin?

[nigeltao, 2014-03-25 02:29:59]

On Mon, Mar 24, 2014 at 6:34 PM, Oleku Konko <oleku.konko@gmail.com> wrote:
> Nice .. any plans for encoder ?

I'd like to see one, but haven't had the time to do it myself.

[nigeltao, 2014-03-25 02:30:08]

*** Submitted as
https://code.google.com/p/go/source/detail?r=dc5d8934d75b&repo=image ***

go.image/vp8, go.image/webp: new packages, copied from
code.google.com/p/vp8-go

This is a straight copy, followed by import path updates.

LGTM=r
R=r, mirtchovski, oleku.konko
CC=golang-codereviews
https://codereview.appspot.com/79320043