Regarding this code 
<https://code.google.com/p/web2py/source/detail?r=b6e00ef71689bb688f5313e3b49bbfffd339834c>
:

if isinstance(cond.second, set):
>     cond.second = list(cond.second)
> if isinstance(cond.second, (list, tuple)):
>    return base, ','.join("[value='%s']" % (v) for v in cond.second)


Why is it not simply this?:

if isinstance(cond.second, (set, list, tuple)):
>    return base, ','.join("[value='%s']" % (v) for v in cond.second)


Is there some reason to convert a set to a list first?  Perhaps it is a 
performance related issue?


-- 
Resources:
- http://web2py.com
- http://web2py.com/book (Documentation)
- http://github.com/web2py/web2py (Source code)
- https://code.google.com/p/web2py/issues/list (Report Issues)
--- 
You received this message because you are subscribed to the Google Groups 
"web2py-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to web2py+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to