django-openstack-auth uses sets suboptimally in multiple places.
Bug #1506925 reported by
Kirill Zaitsev
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
django-openstack-auth |
Fix Released
|
Undecided
|
Kirill Zaitsev |
Bug Description
In many places throughout the code sets are constructed from list comprehensions. Since django-
Changed in django-openstack-auth: | |
assignee: | nobody → Kirill Zaitsev (kzaitsev) |
status: | New → In Progress |
To post a comment you must log in.
Reviewed: https:/ /review. openstack. org/216441 /git.openstack. org/cgit/ openstack/ django_ openstack_ auth/commit/ ?id=885cdd96c63 0fa225a2e7417f5 41ddfeb335bd89
Committed: https:/
Submitter: Jenkins
Branch: master
commit 885cdd96c630fa2 25a2e7417f541dd feb335bd89
Author: Kirill Zaitsev <email address hidden>
Date: Tue Aug 25 01:02:28 2015 +0300
Use set comprehension instead of converting lists to sets
This patch takes advantage of python set comprehensions
syntax instead of constructing a list and converting
it to set later.
Also takes advantage of .isdisjoint function,
that returns True if two sets have a null intersection.
Should slightly improve performance and readability.
Closes-Bug: #1506925 280d7570e782fd1 96ce716ac8a
Change-Id: Ia3d8b47efcf1b2