Russ Egan

Results 35 comments of Russ Egan

Do you still want to maintain compat with

There are several tests which expected the old behavior. There are a few categories: 1. Several tests use example structs with no tags, but expect them to get filled anyway....

Sorry, we ending up using a different library for this (creasty/defaults)

btw, I needed this for my own project only because I preferred not to maintain a dedicated thread and run loop for the event delivery.

maybe add a slice or map to `decoder` to track used keys, then compare to original values at the end of Decoder.Decode()

I've run into a similar case in tests when using assertion frameworks: ``` db, err := OpenDB() require.NoError(t, err) defer db.Close() ``` Currently, wsl wants a line before defer. Unfortunately,...

Another defer variation: ``` tx := BeginTx(db) defer func() { EndTx(tx, err) }() ``` wsl wants a line before the defer statement. In this case, EndTx is being wrapped in...

Re: the second case... n := 1 if bool { fn(n) } probably works, but I don't think... n := 1 defer fn() { fn2(n) }() ...will work, where `n`...

Sorry, that was a type. Declaring and calling an anonymous function doesn't make sense, unless you are deferring the call: ```golang n := 1 defer func() { fn2(n) }() ```...

That seemed pretty quick to me! Thanks for the help!