LaurentCR / pylast

Automatically exported from code.google.com/p/pylast
Apache License 2.0
0 stars 0 forks source link

[PATCH] Whitespace fix #26

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What version of pyLast is this?
r214

Please provide any additional information below.

There are a lot of trailing whitespaces in the code. You can use 
":%s/\s\+$//gc" from VIM to remove them. I've also attached a patch.
You can use the following to see them in vim:
let python_space_errors=1
highlight WhitespaceEOL ctermbg=red guibg=red
match WhitespaceEOL /\s\+$/

Original issue reported on code.google.com by nikolas....@gmail.com on 19 Oct 2009 at 10:11

Attachments:

GoogleCodeExporter commented 9 years ago
Also using tabs instead of spaces?you can use the following to fix it:

:set ts=4
:retab

Original comment by nikolas....@gmail.com on 19 Oct 2009 at 10:27

GoogleCodeExporter commented 9 years ago
I am using tabs instead of spaces. It's more flexible. You can choose your own 
tab-width.

Original comment by amr.hassan on 20 Oct 2009 at 7:36

GoogleCodeExporter commented 9 years ago
Okay, I found that it is the convention to use spaces in python.
http://www.python.org/dev/peps/pep-0008/
I'll commit the changes asap.

Original comment by amr.hassan on 20 Oct 2009 at 8:05

GoogleCodeExporter commented 9 years ago
Ok, thanks for seeing this as an issue. 
I'm planning to use pylast for a full-featured quodlibet last.fm plugin. So I 
will 
submit some real patches if needed. 

Original comment by nikolas....@gmail.com on 20 Oct 2009 at 8:41