|
Adg |
at May 23, 2013 at 2:31 am
|
⇧ |
| |
This looks ok to me now, but wait for brad.
https://codereview.appspot.com/9671043/diff/19002/google-api-go-generator/gen.goFile google-api-go-generator/gen.go (right):
https://codereview.appspot.com/9671043/diff/19002/google-api-go-generator/gen.go#newcode1152google-api-go-generator/gen.go:1152: pn("req.URL.Path =
strings.Replace(req.URL.Path, \"{%s}\", %s, 1)\n", arg.apiname,
arg.cleanExpr("c."))
On 2013/05/23 00:53:23, gocampoy wrote:On 2013/05/23 00:45:36, adg wrote:
AGAIN, please use `backtics` to avoid the needless quoting
Done
I was keeping it that way just to be consistent with the code around
it.
Look at the surrounding code. It uses backticks everywhere there are
quotes inside the strings.
https://codereview.appspot.com/9671043/