Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(422)

Issue 5541063: code review 5541063: net/http: rename ServeFile to ServeOSFile, add new ServeFile (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 3 months ago by adg
Modified:
12 years, 3 months ago
Reviewers:
rsc, r2
CC:
golang-dev
Visibility:
Public.

Description

net/http: rename ServeFile to ServeOSFile, add new ServeFile This new function takes an http.File and simply serves its contents, bypassing the redirect behavior (that can still be used through FileServer). ServeOSFile is now a mere helper that opens the named file from the local file system and invokes ServeFile. Users of the original ServeFile will need to switch to using FileServer (if they need the redirect behavior) or just use ServeOSFile. Fixes issue 2039.

Patch Set 1 #

Patch Set 2 : diff -r f690897afe60 https://go.googlecode.com/hg #

Unified diffs Side-by-side diffs Delta from patch set Stats (+87 lines, -71 lines) Patch
M src/pkg/net/http/fs.go View 1 4 chunks +81 lines, -65 lines 0 comments Download
M src/pkg/net/http/fs_test.go View 1 5 chunks +5 lines, -5 lines 0 comments Download
M src/pkg/net/http/serve_test.go View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6
adg
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://go.googlecode.com/hg
12 years, 3 months ago (2012-01-15 22:21:32 UTC) #1
rsc
Better name please.
12 years, 3 months ago (2012-01-16 20:13:01 UTC) #2
adg
Hello golang-dev@googlegroups.com, rsc@golang.org (cc: golang-dev@googlegroups.com), Please take another look.
12 years, 3 months ago (2012-01-16 23:11:36 UTC) #3
rsc
Sorry, but I can't stomach ServeOSFile.
12 years, 3 months ago (2012-01-16 23:15:18 UTC) #4
r2
I'm reminded of the first BSTJ Unix issue, from the era when AT&T's lawyers were ...
12 years, 3 months ago (2012-01-17 00:06:39 UTC) #5
adg
12 years, 3 months ago (2012-02-02 06:32:21 UTC) #6
*** Abandoned ***
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b