code review 6197061: compress/testdata: remove Byte Order Mark from the Tom ...

compress/testdata: remove Byte Order Mark from the Tom Sawyer data.

I'm not sure where the BOM came from, originally.
http://www.gutenberg.org/files/74/74.txt doesn't have it, although
a fresh download of that URL gives me "\r\n"s instead of plain "\n"s,
and the extra line "Character set encoding: ASCII". Maybe Project
Gutenberg has changed their server configuration since we added that
file to the Go repo.

Anyway, this change is just manually excising the BOM from the start
of the file, leaving pure ASCII.

[nigeltao, 2012-05-08 00:35:22]

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

I'd like you to review this change to
https://go.googlecode.com/hg/

[bradfitz, 2012-05-08 00:37:54]

LGTM

On Tue, May 8, 2012 at 10:35 AM, <nigeltao@golang.org> wrote:

> Reviewers: r,
>
> Message:
> Hello r@golang.org (cc: golang-dev@googlegroups.com, krasin@golang.org,
> rsc@golang.org),
>
> I'd like you to review this change to
> https://go.googlecode.com/hg/
>
>
> Description:
> compress/testdata: remove Byte Order Mark from the Tom Sawyer data.
>
> I'm not sure where the BOM came from, originally.
>
http://www.gutenberg.org/**files/74/74.txt<http://www.gutenberg.org/files/74/...
have it, although
> a fresh download of that URL gives me "\r\n"s instead of plain "\n"s,
> and the extra line "Character set encoding: ASCII". Maybe Project
> Gutenberg has changed their server configuration since we added that
> file to the Go repo.
>
> Anyway, this change is just manually excising the BOM from the start
> of the file, leaving pure ASCII.
>
> Please review this at
http://codereview.appspot.com/**6197061/<http://codereview.appspot.com/6197061/>
>
> Affected files:
>  M src/pkg/compress/testdata/**Mark.Twain-Tom.Sawyer.txt
>
>
> Index: src/pkg/compress/testdata/**Mark.Twain-Tom.Sawyer.txt
> ==============================**==============================**=======
> --- a/src/pkg/compress/testdata/**Mark.Twain-Tom.Sawyer.txt
> +++ b/src/pkg/compress/testdata/**Mark.Twain-Tom.Sawyer.txt
> @@ -1,4 +1,4 @@
> -The Project Gutenberg EBook of The Adventures of Tom Sawyer, Complete
> +The Project Gutenberg EBook of The Adventures of Tom Sawyer, Complete
>  by Mark Twain (Samuel Clemens)
>
>  This eBook is for the use of anyone anywhere at no cost and with
>
>
>

[nigeltao, 2012-05-08 00:41:57]

*** Submitted as http://code.google.com/p/go/source/detail?r=8db1b9223486 ***

compress/testdata: remove Byte Order Mark from the Tom Sawyer data.

I'm not sure where the BOM came from, originally.
http://www.gutenberg.org/files/74/74.txt doesn't have it, although
a fresh download of that URL gives me "\r\n"s instead of plain "\n"s,
and the extra line "Character set encoding: ASCII". Maybe Project
Gutenberg has changed their server configuration since we added that
file to the Go repo.

Anyway, this change is just manually excising the BOM from the start
of the file, leaving pure ASCII.

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