PEP8 compliance fixes

[sascha_silbe]

Similar to #1108, the attached patch fixes most of the complains from pep8.py, but this time only for some of the files (mostly the ones changed in my own branch).

[sascha_silbe]

PEP8 compliance fixes for some of the files in sugar-toolkit

[erikos]

Thanks for the patch. I am just working on figuring out how much we should apply to the pep guidelines. Of course, I agree, that we should. But we would have to change a lot of code in all the modules, and we should agree we are all on the same page before.

For example: 2 lines before between classes...

[sascha_silbe]

Trim end of line on all source files

[sascha_silbe]

As a start I've now done only the EOL fixes (but for all files this time). I suppose that doesn't require a lot of discussion: Whitespace on end of line is redundant in virtually any source file and will get automatically removed by many editors, especially if you change anything on the same line. The latter point makes working with EOL-containing files hard if you want to back out single changes (because the whitespace changes will stay).

"make check" doesn't report anything it didn't report before (a lot of intltool complains).

[tomeu]

Replying to erikos:

Thanks for the patch. I am just working on figuring out how much we should apply to the pep guidelines. Of course, I agree, that we should. But we would have to change a lot of code in all the modules, and we should agree we are all on the same page before.

For example: 2 lines before between classes...

Yeah, I don't care about the particular rules that are enforced, for me what is important is that it's consistent across the whole codebase and that can be checked automatically.

[erikos]

Patch must be rebased - otherwise looks good.

[erikos]

pushed by sascha.