python indentation

python_indent.py

#  aligned
return str('%s://%s/%s%s' % (
    self.url_scheme,
    self.get_host(domain_part),
    posixpath.join(self.script_name[:-1].lstrip('/'),
                   url_quote(path_info.lstrip('/'), self.map.charset,
                             safe='/:|+')),
    suffix
))

#  hanging
return str('%s://%s/%s%s' % (
    self.url_scheme,
    self.get_host(domain_part),
    posixpath.join(self.script_name[:-1].lstrip('/'),
        url_quote(path_info.lstrip('/'), self.map.charset,
        safe='/:|+')),
    suffix
))

#  aligned (again)
return str('%s://%s/%s%s' % (
    self.url_scheme,
    self.get_host(domain_part),
    posixpath.join(
        self.script_name[:-1].lstrip('/'),
        url_quote(
            path_info.lstrip('/'),
            self.map.charset,
            safe='/:|+'
            )
        ),
    suffix
    )
)

# without thinking about it a whole lot, here's what I would have done
# (assuming, of course, breaking it up into multiple statements
# is against the spirit of the exercise :) )

return str('%s://%s/%s%s' % (
    self.url_scheme,
    self.get_host(domain_part),
    posixpath.join(
        self.script_name[:-1].lstrip('/'),
        url_quote(
            path_info.lstrip('/'),
            self.map.charset,
            safe='/:|+',
        )
    ),
    suffix
))

# seems closest to "aligned (again)" - changes annotated
return str('%s://%s/%s%s' % (
    self.url_scheme,
    self.get_host(domain_part),
    posixpath.join(
        self.script_name[:-1].lstrip('/'),
        url_quote(
            path_info.lstrip('/'),
            self.map.charset,
            safe='/:|+',  # trailing comma
        )  # match indentation of url_quote identifier
    ),
    suffix
))  # 'soft' closing paren matches the soft opening paren

+1 I like how you would have done it. I agree with the 'soft' closing parens also and would change mine to do the same.