[Wikitech-l] Coding conventions

2013-01-22 Thread Brad Jorsch
Our coding conventions for PHP are currently ambivalent on whether we should write if ( or if(. It's probably time to pick one. Discussion at https://www.mediawiki.org/wiki/Manual_talk:Coding_conventions/PHP#Control_structures ___ Wikitech-l mailing

Re: [Wikitech-l] Coding conventions

2013-01-22 Thread bawolff
On 2013-01-22 3:01 PM, Brad Jorsch bjor...@wikimedia.org wrote: Our coding conventions for PHP are currently ambivalent on whether we should write if ( or if(. It's probably time to pick one. Discussion at https://www.mediawiki.org/wiki/Manual_talk:Coding_conventions/PHP#Control_structures

Re: [Wikitech-l] Coding conventions

2013-01-22 Thread Tyler Romeo
Yeah, this has gotten pretty frustrating, because sometimes even in the same function different styles are used. We need to stick with one. I'd recommend using the spacey version, because we use the non-spacey version for function calls and definitions, so at least IMO it's a good way to

Re: [Wikitech-l] Coding conventions

2013-01-22 Thread Yuri Astrakhan
I feel we should keep in line with the function calls. There are already way too many spaces as it is, and there is never a problem telling control structures apart - they are always followed by an indented block and most editors highlight them. I think if(), while(), foreach(), for() should all

Re: [Wikitech-l] Coding conventions

2013-01-22 Thread Jeroen De Dauw
Hey, I hereby admit defeat. My thread was clearly not the ultimate bikeshed. Cheers -- Jeroen De Dauw http://www.bn2vs.com Don't panic. Don't be evil. -- ___ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org

Re: [Wikitech-l] Coding conventions

2013-01-22 Thread Tyler Romeo
Lmao. Honestly I don't care either way, and any attempt to make up reasons for one over the other is going to be insignificant. We just need to pick one and stick with it. If we can agree on one, I'll personally go through the entire codebase and change it. *--* *Tyler Romeo* Stevens Institute of

Re: [Wikitech-l] Coding conventions

2013-01-22 Thread Bawolff Bawolff
On 2013-01-22 6:05 PM, Jeroen De Dauw jeroended...@gmail.com wrote: Hey, I hereby admit defeat. My thread was clearly not the ultimate bikeshed. Cheers -- On this bikeshed, allowing both styles sounds perfectly acceptable to me. -bawolff ___