On 07/13/2011 10:38 AM, Olaf van der Spek wrote:
<snip>
Of course. However, I already posted why I think the two things should
be changed. Without a response to those arguments, I don't see what
else I could do.
With all due respect, you only provided a reason for why one thing should be changed and that was purely an opinion:

string&foo;


 IMO it should be string&. The&  is part of the type, not part of the name.


My argument is that if you want to touch something that affects everyone like coding standards, that you should attempt to demonstrate real benefits for change (beyond IMO). You are asking people to do something - change what is currently working for everyone else on the project - *you* should explain what is in it for the people you are asking to do the work of changing. Of course, it is possible that you think simply conforming with your opinions is reward enough ; )

Again, you place the burden on others- "If you don't respond, I will assume that I can have my way" (as illustrated by your attempts to construe a lack of response as lack of objection), but if someone does respond you are likely to go on endlessly about what is ultimately a matter of opinion (and one should generally respect established precedent in such matters, IMO)

In regards to the parentheses, you illustrate your behavior for me:

return (x == MY_TYPE);


 Is there any need for those parentheses?

*That* is not a reason for change - that is demanding that people expend time and energy explaining why they don't conform to your way of doing things. If you do consider that to be an explanation - you don't understand why it is, therefore it should not be - then we have a larger communication problem than I had initially thought.

At the end of the day, this is in the hands of everyone, I am simply saying that you are being excessively pushy with your opinions without providing any demonstrable advantage for doing things your way.
Greetings,

Olaf


_______________________________________________
Mailing list: https://launchpad.net/~drizzle-discuss
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~drizzle-discuss
More help   : https://help.launchpad.net/ListHelp

Reply via email to