I'd be curious if anyone who has worked at Valve has any insights they would like to share.
It's worth pointing out that Linus Torvalds has a successful continuous feedback system -- and it is his 99% job to provide feedback. I only mean that he doesn't write as much of the Linux code any more, day-to-day, he mostly just reviews patches.
That's wonderful! I work under several layers of managers who are each more interested in doing hands-on work than management work, and it is truly awful. I have literally gone almost two years without substantive, actionable feedback. I have a great team, which has kept me with the company, but at this point I am actively looking to jump ship.
What's so bad about that? It actually seems like a pretty sensible and well argued bit of feedback - with some slightly colorful language but nothing that seems that extreme.
It's honestly rather long winded for something the compiler isn't going to give two hoots about. :)
When it comes to syntax, I personally think that the best policy is to pick one style, and stick with it. I have come to distrust people with passionate rants like this about syntax because, in my experience, in a few years there probably will be another syntax that falls in fashion, and all of a sudden this New Syntax that now is The New One True Way that everyone gets passionate about, and you have to be high to actually like Old Syntax or something.
What can happen in long term projects is syntax mishmash. Not terribly a big deal with comment formatting, but annoying for other things. Wonder if that variable you are trying to remember is camelCase, underscore_notation, or sHungarianNotation? Is the table you are joining to using Id as a key, TableId as a key, or Table_Id as a key? That's more annoying than any of the problems any syntax has.
So Linus's rant only would be useful to me if his opinion on comment formatting has been consistent over the years. :)
It would have been good if the kernel folks could have written or made use of an existing tool (e.g. GNU indent) that would ensure that the style is consistent. Inconsistent style hurts the eyes.
It's worth pointing out that Linus Torvalds has a successful continuous feedback system -- and it is his 99% job to provide feedback. I only mean that he doesn't write as much of the Linux code any more, day-to-day, he mostly just reviews patches.