LosManos/ST4bby — comment on commit f9c5da27d1b1f95cd86391fd2b28f39a0a4cbd5f
File:  line 0
By LosManos:

> ...method naming is clear...

I had to take a guess at your standards. I chose CreateXXX for string generating. I try to keep those helping methods static. I try to keep the methods in alphabetical order inside scopes like public and private; public a-z, private a-z.

> ...minor nits...
Yup. But it's like in a relationship; you sometimes have to talk about the minor nits before they grow big.

> ...should use StringBuilder...
http://www.codinghorror.com/blog/2009/01/the-sad-tragedy-of-micro-optimization-theater.html
http://geekswithblogs.net/BlackRabbitCoder/archive/2010/05/10/c-string-compares-and-concatenations.aspx
I don't mean to complain or contradict, just shed some light.
It is your show - you decide. I'll switch to string.Format. I didn't see no StringBuilder though. Planned for future?

> ...whitespace...
I like 4 spaces too, it seems to be standard today.
But the code was a mixture of tabs and spaces and 2 and 4 indentation so I soon quit spending time on it. The include file should be 4spaces though.

I have a problem - my editor (visualstudio/tangible) gives me tabs and I haven't found a way to make it spaces. How did you succeed? (well... obvious you didn't... :- ) I told Visualstudio to handle .tt files as csharp files and they have spaces; doesn't seem to work.