Add a test case for the issue #55
Add lookbehind to visibility fixer

Checks that the character preceeding class|interface|trait is not a $,
as you may have a variable called $class, $interface, or $trait in
non-oop code.
Add test assertion for visibility regex fix

Ensures that non-oop code is left alone, even if it contains varaibles
names $class, $interface, or $trait.
Fix grammar mistake
Change level of fixer "eof_ending" to PSR-2

As specified in PSR-2 Section 2.2 "Files".
See https://github.com/php-fig/fig-standards/blob/3e49c27/accepted/PSR-2-coding-style-guide.md#22-files
merged branch raphinesse/fix-eof_ending-level (PR #139)

Commits
-------

e57c92c Change level of fixer "eof_ending" to PSR-2
434b803 Fix grammar mistake

Discussion
----------

Change level of "eof_ending" fixer to PSR-2

Since the optimization performed by the `eof_ending` fixer is a [requirement of PSR-2](https://github.com/php-fig/fig-standards/blob/3e49c27/accepted/PSR-2-coding-style-guide.md#22-files) it should have the level PSR-2.

I had to combine the changes with the grammar fix to avoid hitting the line length recommendation in `README.md` :smirk:
Switched the deps to stable and updated them
merged branch stof/update_deps (PR #146)

This PR was merged into the master branch.

Commits
-------

0130f4e Switched the deps to stable and updated them

Discussion
----------

Switched the deps to stable and updated them
added a version to composer.json
updated license year
Added support for reference parameters to PhpdocParamsAlignmentFixer
Better ControlSpacesFixer multiline indentation.

Before this fix, ControlSpacesFixer lost the indentation of multiline
if -control statement for outer closing parenthesis.

Before:
        if(true === true
            && false === false
        ) {
Became:
        if (true === true
            && false === false
) {

Now:
        if (true === true
            && false === false
        ) {

Commit also comments the regular expression with inline comment, so it's
easier to understand.

In ControlSpacesFixer::fixControlsWithParenthesesAndSuffixBrace()
(%s)[^\S\n]*\([^\S\n]*([^()]*?|(?R))[^\S\n]*\)[^\S\n]*{
Became:
(%s)[^\S\n]*\([^\S\n]*([^()]*?|(?R))[^\S\n]*(\s*)\)[^\S\n]*{
Return non-zero status code if anything was fixed

This is so that commit hooks, or CI tools can easily fail.

fixes #158
Changed exit to just return
merged branch baldurrensch/issue_158 (PR #173)

This PR was merged into the master branch.

Commits
-------

951475f Changed exit to just return
85e4f1f Return non-zero status code if anything was fixed

Discussion
----------

Return non-zero status code if anything was fixed

This is so that commit hooks, or CI tools can easily fail.

fixes #158

---------------------------------------------------------------------------

by lyrixx at 2013-01-25T21:53:22Z

You should not exit by yourself, but return the return code.
See: https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Console/Application.php#L193 and https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Console/Application.php#L119

---------------------------------------------------------------------------

by baldurrensch at 2013-01-25T22:20:52Z

@lyrixx: Thanks for the comment. FIxed.

---------------------------------------------------------------------------

by stof at 2013-01-25T23:29:45Z

This is bad IMO. The goal of the fixer is to fix your code. So why would it be a failure when it does its job ?

---------------------------------------------------------------------------

by baldurrensch at 2013-01-25T23:43:26Z

It would make it a little easier to use in tests (CI, pre-commit hook). I would have no problem with adding a comand-line option `--check` to enable the return code. Please also see the discussion for the issue.

---------------------------------------------------------------------------

by lyrixx at 2013-01-26T11:05:55Z

@stof I agree with you. But in this case, it's better to be pragmatic and don't respect the purity of code.

You can also made an analogy with grep:

``` bash
>[~/dev/foo](patch-2) grep foooooo web/index.php
>[~/dev/foo](patch-2) echo $?
1
>[~/dev/foo](patch-2) grep php web/index.php
Some content ...
>[~/dev/foo](patch-2) echo $?
0
```

If grep find something, it return 0, If not it return 1.
If php-cs-fixer find something it return 0, else it return 1. (Yes the pr do the opposite)
fixed CS
Fix - trailing spaces after class or alias name
updated Symfony deps (closes #187)
Fix spelling mistake
merged branch localheinz/hotfix/spelling-mistake (PR #189)

This PR was squashed before being merged into the master branch (closes #189).

Discussion
----------

Fix spelling mistake

In `README.md` it should probably say `divided` rather than `devided`.

Commits
-------

8c506ec Fix spelling mistake