Default --title on --message and clamp at 100 chars

Default --title on --message and clamp at 100 chars

[M-A, 2012-01-23 19:39:12]

It's more or less what chromium's wrapper tools are doing.

I thought about using "…" instead of "...". I'm just afraid of breaking non
utf-8 compatible workflows. (Note that it's not a bad idea to break broken
tools)

[Andi Albrecht, 2012-01-23 19:51:49]

LGTM

On 2012/01/23 19:39:12, M-A wrote:
> It's more or less what chromium's wrapper tools are doing.
> 
> I thought about using "…" instead of "...". I'm just afraid of breaking non
> utf-8 compatible workflows. (Note that it's not a bad idea to break broken
> tools)

+1 for breaking them - yes, I want to use Ümläüts :)

[Andi Albrecht, 2012-01-23 19:52:42]

sorry, I've had two comments - but just nits, so it's still a LGTM

http://codereview.appspot.com/5574050/diff/2001/upload.py
File upload.py (right):

http://codereview.appspot.com/5574050/diff/2001/upload.py#newcode2207
upload.py:2207: prompt = "New issue subject: "
What about moving the prompt definition closer to raw_input() again. AFAICT it
is only used for this purpose.

http://codereview.appspot.com/5574050/diff/2001/upload.py#newcode2249
upload.py:2249: title or message.split('\n', 1)[0].strip() or
raw_input(prompt)).strip()
Indent 2 spaces

[M-A, 2012-01-23 20:04:43]

Now with ÜTF-8 requirement.

http://codereview.appspot.com/5574050/diff/2001/upload.py
File upload.py (right):

http://codereview.appspot.com/5574050/diff/2001/upload.py#newcode2249
upload.py:2249: title or message.split('\n', 1)[0].strip() or
raw_input(prompt)).strip()
On 2012/01/23 19:52:43, Andi Albrecht wrote:
> Indent 2 spaces

Why? It's like a function call, no?

http://codereview.appspot.com/5574050/diff/4002/upload.py
File upload.py (right):

http://codereview.appspot.com/5574050/diff/4002/upload.py#newcode2254
upload.py:2254: title = title[:99] + '…'
I'm actually shocked that it worked on the first time. Added encoding at the top
of the file.

[Andi Albrecht, 2012-01-23 20:12:12]

http://codereview.appspot.com/5574050/diff/2001/upload.py
File upload.py (right):

http://codereview.appspot.com/5574050/diff/2001/upload.py#newcode2249
upload.py:2249: title or message.split('\n', 1)[0].strip() or
raw_input(prompt)).strip()
I'm not sure about the correct indentation level here. 2 spaces seems to fit
fine with the rest of the file.

On 2012/01/23 20:04:43, M-A wrote:
> On 2012/01/23 19:52:43, Andi Albrecht wrote:
> > Indent 2 spaces
> 
> Why? It's like a function call, no?