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-04 01:14 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    
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes. If this is not your bug, you can add a comment by following this link. If this is your bug, but you forgot your password, you can retrieve your password here.
2010-03-16 11:12 UTC
Bug Type:
From: crempa at gmail dot com
New email:
PHP Version: Package Version: OS:


 [2007-02-04 01: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 }


 [2008-03-22 11: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 14: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 09: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 22:34 UTC] clbustos (Claudio Bustos)
This bug is not related with PEAR, but with allman/bsd style....
 [2010-03-16 11:12 UTC] clbustos (Claudio Bustos)
-Assigned To: +Assigned To: clbustos -Roadmap Versions: +Roadmap Versions: 1.0.0RC1