Backslash continuations (Keystone)

Bug #940010 reported by Zhongyue Luo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Undecided
Zhongyue Luo

Bug Description

There are several places in Keystone where we make use of backslash continuations. However, this is discouraged by PEP8. Not all of these places are avoidable, but many of these are, through judicious use of parentheses. (In some places, the parentheses, brackets, or braces may even already exist!) We should clean these up throughout Keystone.

Zhongyue Luo (zyluo)
Changed in keystone:
assignee: nobody → Zhongyue Luo (lzyeval)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

Fix proposed to branch: master
Review: https://review.openstack.org/4482

Changed in keystone:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/4482
Committed: http://github.com/openstack/keystone/commit/036b99086c170418b4fa54bc51c1c632eb420506
Submitter: Jenkins
Branch: master

commit 036b99086c170418b4fa54bc51c1c632eb420506
Author: Zhongyue Luo <email address hidden>
Date: Fri Feb 24 10:38:45 2012 +0800

    Backslash continuations (Keystone)

    Fixes bug #940010

    Didn't fix SQLAlchemy queries.

    Module keystone/middleware/auth_token.py
    is fixed in https://review.openstack.org/#change,4385

    Change-Id: Idaad992b5ef493407a23f0f77110ed2ab3f2452e

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
milestone: none → essex-4
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: essex-4 → 2012.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.