Remove redundant commas
Bug #1477142 reported by
liuyan
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Glance |
Invalid
|
Low
|
Pankaj Mishra |
Bug Description
In glance source, there is redundant comma in front of parenthesis.
For example in below:
def _get_sort_key(self, req):
"""Parse a sort key query param from the request object."""
sort_key = req.params.
if sort_key is not None and sort_key not in SUPPORTED_
_keys = ', '.join(
msg = _("Unsupported sort_key. Acceptable values: %s") % (_keys,)
raise exc.HTTPBadRequ
return sort_key
in line six, the commas behind "_keys" is redundant, so we should remove it .
Changed in glance: | |
assignee: | nobody → liuyan (lykobe) |
Changed in glance: | |
status: | New → In Progress |
Changed in glance: | |
status: | Invalid → In Progress |
Changed in glance: | |
importance: | Undecided → Low |
Changed in glance: | |
assignee: | liuyan (lykobe) → Pankaj Mishra (pankaj-mishra) |
To post a comment you must log in.
This more or less boils down to future-proofing. If we need to expand the parameters included in these messages, it reduces the visual noise in the diff.