Have Rietveld.get_description() consistently strip CR and whitespace.

Have Rietveld.get_description() consistently strip CR and whitespace.

This makes the commit description cleaner.

R=iannucci@chromium.org
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=199275

[iannucci, 2013-04-26 18:21:46]

https://codereview.chromium.org/14061008/diff/1/rietveld.py
File rietveld.py (right):

https://codereview.chromium.org/14061008/diff/1/rietveld.py#newcode89
rietveld.py:89: return '\n'.join(self.get('/%d/description' %
issue).strip().splitlines())
So if it ended with a line ending, it won't have any line ending any more?

https://codereview.chromium.org/14061008/diff/1/rietveld.py#newcode97
rietveld.py:97: data['description'] =
'\n'.join(data['description'].strip().splitlines())
Can description can be absent/None?

[M-A Ruel, 2013-04-26 18:23:53]

https://codereview.chromium.org/14061008/diff/1/rietveld.py
File rietveld.py (right):

https://codereview.chromium.org/14061008/diff/1/rietveld.py#newcode89
rietveld.py:89: return '\n'.join(self.get('/%d/description' %
issue).strip().splitlines())
On 2013/04/26 18:21:46, iannucci wrote:
> So if it ended with a line ending, it won't have any line ending any more?

Exact, which is fine.

https://codereview.chromium.org/14061008/diff/1/rietveld.py#newcode97
rietveld.py:97: data['description'] =
'\n'.join(data['description'].strip().splitlines())
On 2013/04/26 18:21:46, iannucci wrote:
> Can description can be absent/None?

Not in the real API.

[iannucci, 2013-05-09 21:27:51]

lgtm

[commit-bot: I haz the power, 2013-05-09 21:28:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/14061008/1

[commit-bot: I haz the power, 2013-05-09 21:30:47]

Change committed as 199275