ergophobe - 10:38 pm on Oct 5, 2010 (gmt 0)
>>offering unsolicited advice
Why were you shown the code? Was it to review?
In an ideal world, we would all welcome and constructive comments, but not everyone does and even people who do, don't welcome them at all times.
Personally, I like to do so in phases and if I'm not in a "feedback phase" I will pretty much ignore your feedback - not intentionally per se, but I will have my own list of 1000 things to fix from the last feedback phase and when you bump that list up to 1010, it really isn't helping. When I get my list down to 20 things and you add 10, that becomes a lot more valuable for me.