LGTM On 8 April 2010 10:21, <jdpoirier@gmail.com> wrote: > Reviewers: rsc, > > Message: > ...
14 years, 11 months ago
(2010-04-08 00:25:13 UTC)
#2
LGTM
On 8 April 2010 10:21, <jdpoirier@gmail.com> wrote:
> Reviewers: rsc,
>
> Message:
> Hello rsc (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> typo in the first code example in the "Handling panics" section
>
> Please review this at http://codereview.appspot.com/881047/show
>
> Affected files:
> M doc/go_spec.html
>
>
> Index: doc/go_spec.html
> ===================================================================
> --- a/doc/go_spec.html
> +++ b/doc/go_spec.html
> @@ -4570,7 +4570,7 @@
> func f(hideErrors bool) {
> defer func() {
> if x := recover(); x != nil {
> - println("panicking with value", v)
> + println("panicking with value", x)
> if !hideErrors {
> panic(x) // go back to panicking
> }
>
>
>
>
> --
> To unsubscribe, reply using "remove me" as the subject.
>
Issue 881047: code review 881047: typo in the first code example in the "Handling panics"...
(Closed)
Created 14 years, 11 months ago by Joe Poirier
Modified 14 years, 11 months ago
Reviewers:
Base URL:
Comments: 0