Undefined variable 'o' in Python encoder

[danielmewes]

I'm not sure if this line is ever hit, but I'm pretty sure that the o in https://github.com/rethinkdb/rethinkdb/blob/next/drivers/python/rethinkdb/ast.py#L701 should be obj.

[asakatida]

It seems to never be hit in my usage. I'm updating my latest PR still.

[danielmewes]

Thanks @grandquista! I also didn't hit this in practice. Just noticed it when looking at the code.

(@Tryneus is currently out of office, but he'll be back next week to look at the PR. So we should be able to get it all in on time for 2.2)

[asakatida]

r.expr filters out any object of type that would trigger that line. I was looking at removing recursion from r.expr and that change would definitely trigger the line in some use cases. The line throws in either case. Fixing it just means the right error is thrown.