| 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 | ||
[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
}
Comments
[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