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

Bug #10019 switch indent
Submitted: 2007-02-03 20:14 UTC Modified: 2008-06-29 17:34 UTC
From: crempa at gmail dot com Assigned:
Status: Open Package: PHP_Beautifier (version 0.1.11)
PHP Version: 5.2.0 OS: Windows
Roadmaps: (Not assigned)    
Subscription  
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.
Password:
Status:
Package:
Bug Type:
Summary:
From: crempa at gmail dot com
New email:
PHP Version: Package Version: OS:

 

 [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....