Only stripping trailing slashes is clearly an improvement on always removing whatever the last character is and this should clearly land.
However, doing url manipulation with basic string functions, even simple stuff like this, makes me nervous. I'm thinking particularly of this change:
<https://code.launchpad.net/~gz/bzr/root_drive_file_url_841322/+merge/74034>
self.base = base - self._segment_parameters = urlutils.split_segment_parameters( - base.rstrip("/"))[1] + self._segment_parameters = urlutils.split_segment_parameters(base)[1]
Which was needed to avoid some very surprising breakage.
« Back to merge proposal
Only stripping trailing slashes is clearly an improvement on always removing whatever the last character is and this should clearly land.
However, doing url manipulation with basic string functions, even simple stuff like this, makes me nervous. I'm thinking particularly of this change:
<https:/ /code.launchpad .net/~gz/ bzr/root_ drive_file_ url_841322/ +merge/ 74034>
self.base = base parameters = urlutils. split_segment_ parameters( "/"))[1] parameters = urlutils. split_segment_ parameters( base)[1]
- self._segment_
- base.rstrip(
+ self._segment_
Which was needed to avoid some very surprising breakage.