Since the reception is a bit weak, I think we can postpone this decision for after 2.3 release. Maybe even for 3.0 which will include namespaces, and as such requires a major restructuring anyway.
Regardless, I wish to improve the standard for the next release and find a way to check them automatically. Therefore, I'm working on a ruleset for PHP_CodeSniffer [1] which we can add to our build script. I'll open an issue to work on this. Regards, Ivan [1] http://pear.php.net/package/PHP_CodeSniffer On 13 June 2012 13:09, Christian Grobmeier <grobme...@gmail.com> wrote: > ugh inline brackets?? > This is pretty much C.... > > well if anybody else wants this... I am a huge fan of inline brackets > but of course I will follow the majority. > > Cheers > > On Wed, Jun 13, 2012 at 10:04 AM, Ivan Habunek <ivan.habu...@gmail.com> wrote: >> The PSR coding standards [1] have recently been approved by the PHP >> Framework Interop Group (ex PHP Standards Working Group) [2]. >> >> I have read the standards, they seem well constructed, and they make >> sense to me. >> >> PSR-0 - Deals with namespaces, so it does not affect the project at this >> point. >> PSR-1 - Sets basic coding standards. log4php code is already compliant >> with this standard. >> PSR-2 - Extends PSR-1 with much more detailed rules. log4php is >> currently not compliant. >> >> There are not that many changes we need to make to be compliant with >> PSR-2. The larger ones would be: >> - indent using 4 spaces (currently tabs) >> - opening braces for classes and methods go on the next line (currently >> inline) >> - lines must end with unix LF (currently undefined - mixed LF and CR/LF) >> >> I think a decent coding standard would be good for this project, and >> make any contributions easier. Therefore I propose that we adopt these >> standards, starting with the next release (2.3.0). I also volunteer to >> do the work. >> >> What say you? >> >> Regards, >> Ivan >> >> [1] https://github.com/php-fig/fig-standards/tree/master/accepted >> [2] https://groups.google.com/forum/#!forum/php-standards > > > > -- > http://www.grobmeier.de > https://www.timeandbill.de