Skip to content

Fix a bug where redisx didn't actually ignore fields in structs tagged with "-" #17

Closed
wants to merge 1 commit into from

2 participants

@wmoss
wmoss commented Feb 5, 2013

From testing, the ScanStruct function was not actually ignoring fields tagged with redis:"-". This seems to fix this, although I can't say I fully understand what's happening in the bottom half of the loop.

@garyburd
Owner
garyburd commented Feb 7, 2013

Fixed in 9c59bb9. I fixed both occurrences of the bug and added a test.

I added issue #18 to remind myself to do something about the code duplicated between the redis and redisx packages.

Please use the ScanStruct function in the redis package. This is the version of the function that I plan to retain after dealing with #18.

@garyburd garyburd closed this Feb 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.