Moved empty noCopy struct to the top of container structs. See @stemar94

 's comment at golang/go#12884 for details

args.go

@@ -36,11 +36,11 @@ var argsPool = &sync.Pool{
 //
 // Args instance MUST NOT be used from concurrently running goroutines.
 type Args struct {
+	noCopy noCopy
+
 	args  []argsKV
 	bufKV argsKV
 	buf   []byte
-
-	noCopy noCopy
 }
 
 type argsKV struct {

client.go

@@ -122,6 +122,8 @@ var defaultClient Client
 //
 // It is safe calling Client methods from concurrently running goroutines.
 type Client struct {
+	noCopy noCopy
+
 	// Client name. Used in User-Agent request header.
 	//
 	// Default client name is used if not set.
@@ -201,8 +203,6 @@ type Client struct {
 	mLock sync.Mutex
 	m     map[string]*HostClient
 	ms    map[string]*HostClient
-
-	noCopy noCopy
 }
 
 // Get appends url contents to dst and returns it as body.
@@ -407,6 +407,8 @@ type DialFunc func(addr string) (net.Conn, error)
 //
 // It is safe calling HostClient methods from concurrently running goroutines.
 type HostClient struct {
+	noCopy noCopy
+
 	// Comma-separated list of upstream HTTP server host addresses,
 	// which are passed to Dial in round-robin manner.
 	//
@@ -513,8 +515,6 @@ type HostClient struct {
 
 	readerPool sync.Pool
 	writerPool sync.Pool
-
-	noCopy noCopy
 }
 
 type clientConn struct {

cookie.go

@@ -47,6 +47,8 @@ var cookiePool = &sync.Pool{
 //
 // Cookie instance MUST NOT be used from concurrently running goroutines.
 type Cookie struct {
+	noCopy noCopy
+
 	key    []byte
 	value  []byte
 	expire time.Time
@@ -55,8 +57,6 @@ type Cookie struct {
 
 	bufKV argsKV
 	buf   []byte
-
-	noCopy noCopy
 }
 
 // CopyTo copies src cookie to c.

header.go

@@ -18,6 +18,8 @@ import (
 // ResponseHeader instance MUST NOT be used from concurrently running
 // goroutines.
 type ResponseHeader struct {
+	noCopy noCopy
+
 	disableNormalizing bool
 	noHTTP11           bool
 	connectionClose    bool
@@ -33,8 +35,6 @@ type ResponseHeader struct {
 	bufKV argsKV
 
 	cookies []argsKV
-
-	noCopy noCopy
 }
 
 // RequestHeader represents HTTP request header.
@@ -45,6 +45,8 @@ type ResponseHeader struct {
 // RequestHeader instance MUST NOT be used from concurrently running
 // goroutines.
 type RequestHeader struct {
+	noCopy noCopy
+
 	disableNormalizing bool
 	noHTTP11           bool
 	connectionClose    bool
@@ -70,8 +72,6 @@ type RequestHeader struct {
 	cookies []argsKV
 
 	rawHeaders []byte
-
-	noCopy noCopy
 }
 
 // SetContentRange sets 'Content-Range: bytes startPos-endPos/contentLength'

http.go

@@ -18,6 +18,8 @@ import (
 //
 // Request instance MUST NOT be used from concurrently running goroutines.
 type Request struct {
+	noCopy noCopy
+
 	// Request header
 	//
 	// Copying Header by value is forbidden. Use pointer to Header instead.
@@ -37,8 +39,6 @@ type Request struct {
 	// Group bool members in order to reduce Request object size.
 	parsedURI      bool
 	parsedPostArgs bool
-
-	noCopy noCopy
 }
 
 // Response represents HTTP response.
@@ -48,6 +48,8 @@ type Request struct {
 //
 // Response instance MUST NOT be used from concurrently running goroutines.
 type Response struct {
+	noCopy noCopy
+
 	// Response header
 	//
 	// Copying Header by value is forbidden. Use pointer to Header instead.
@@ -64,8 +66,6 @@ type Response struct {
 	// Response.Write() skips writing body if set to true.
 	// Use it for writing HEAD responses.
 	SkipBody bool
-
-	noCopy noCopy
 }
 
 // SetRequestURI sets RequestURI.

server.go

@@ -137,6 +137,8 @@ type RequestHandler func(ctx *RequestCtx)
 //
 // It is safe to call Server methods from concurrently running goroutines.
 type Server struct {
+	noCopy noCopy
+
 	// Handler for processing incoming requests.
 	Handler RequestHandler
 
@@ -270,8 +272,6 @@ type Server struct {
 	writerPool     sync.Pool
 	hijackConnPool sync.Pool
 	bytePool       sync.Pool
-
-	noCopy noCopy
 }
 
 // TimeoutHandler creates RequestHandler, which returns StatusRequestTimeout
@@ -345,6 +345,8 @@ func CompressHandlerLevel(h RequestHandler, level int) RequestHandler {
 // running goroutines. The only exception is TimeoutError*, which may be called
 // while other goroutines accessing RequestCtx.
 type RequestCtx struct {
+	noCopy noCopy
+
 	// Incoming request.
 	//
 	// Copying Request by value is forbidden. Use pointer to Request instead.
@@ -376,8 +378,6 @@ type RequestCtx struct {
 	timeoutTimer    *time.Timer
 
 	hijackHandler HijackHandler
-
-	noCopy noCopy
 }
 
 // HijackHandler must process the hijacked connection c.

uri.go

@@ -36,6 +36,8 @@ var uriPool = &sync.Pool{
 //
 // URI instance MUST NOT be used from concurrently running goroutines.
 type URI struct {
+	noCopy noCopy
+
 	pathOriginal []byte
 	scheme       []byte
 	path         []byte
@@ -50,8 +52,6 @@ type URI struct {
 	requestURI []byte
 
 	h *RequestHeader
-
-	noCopy noCopy
 }
 
 // CopyTo copies uri contents to dst.