Hi Steve,

I agree with al Jan said, but wanted to clarify one of the comments a bit, 
in case it's not imediately obvious.
All throughout the bit of code I looked at I see things like:

err = incrval.SetValStr(tptoken, errstr, &incr)
    if nil != err {
        panic(fmt.Sprintf("YDB: Unexpected error with SetValStr(): %s", err
))
    }


This test will always fail, because nil can't be anything other than nil, so of 
course it is not equal to err.
The order is important. The subject of the test needs to be on the left side of 
the test, and the value you are checking
against needs to be on the right. 
Like:

err = incrval.SetValStr(tptoken, errstr, &incr)
   if err != nil {
     panic(fmt.Sprintf("YDB: Unexpected error with SetValStr(): %s", err))
   }
}


It may be helpful to take a look at https://golang.org/ref/spec.

Best of luck,
-K

-- 
You received this message because you are subscribed to the Google Groups 
"golang-nuts" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to golang-nuts+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/golang-nuts/06f03c07-7e5a-4072-8e48-0de20974004e%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to