Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.1.15

Bug #10019 switch indent
Submitted: 2007-02-03 20:14 UTC Modified: 2010-03-16 07:12 UTC
From: crempa at gmail dot com Assigned: clbustos
Status: Assigned Package: PHP_Beautifier (version 0.1.11)
PHP Version: 5.2.0 OS: Windows
Roadmaps: 1.0.0RC1    
Subscription  


 [2007-02-03 20:14 UTC] crempa at gmail dot com (Crempa)
Description: ------------ If I use IndentStyles filter with "allman/bsd" type than the content of switch is not indented. Expected result: ---------------- switch { //switch content } Actual result: -------------- switch { //switch content }

Comments

 [2008-03-22 07:47 UTC] doconnor (Daniel O'Connor)
As much as it looks ugly, I think that's the accepted allman/bsd switch() style :S
 [2008-06-23 09:26 UTC] clbustos (Claudio Bustos)
The "indent style" article on Wikipedia doesn't refers to switch statements. Someone has examples to make a decision? If the bug report is correct, I'll include the patch on 1.0.0RC1
 [2008-06-29 04:49 UTC] dufuz (Helgi Þormar Þorbjörnsson)
Both of the switch styles shown below are allowed in PEAR ... so it's up the the maintainer or even add a config option.
 [2008-06-29 17:34 UTC] clbustos (Claudio Bustos)
This bug is not related with PEAR, but with allman/bsd style....
 [2010-03-16 07:12 UTC] clbustos (Claudio Bustos)
-Assigned To: +Assigned To: clbustos -Roadmap Versions: +Roadmap Versions: 1.0.0RC1