Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WhitespaceAround fails on comment after switch #77

Closed
Gama11 opened this issue Feb 29, 2016 · 3 comments
Closed

WhitespaceAround fails on comment after switch #77

Gama11 opened this issue Feb 29, 2016 · 3 comments
Labels

Comments

@Gama11
Copy link
Member

Gama11 commented Feb 29, 2016

class Test
{
    function foo()
    {
        switch ("Test") // broken
        {
        }
    }
}
./source/Test.hx:1: character 0 : Error: Check WhitespaceAfter failed: bad token CommentLine( broken) != BrOpen
AlexHaxe added a commit to AlexHaxe/haxe-checkstyle that referenced this issue Mar 2, 2016
added support for annotations in conditional HaxeCheckstyle#78
@AlexHaxe
Copy link
Member

AlexHaxe commented Mar 2, 2016

I have added some support for comments, but I probably have missed a few places

@Gama11
Copy link
Member Author

Gama11 commented Mar 2, 2016

If this particular example now works, it's probably best to close and open a new issue for new cases?

Also, this should probably be added as a test?

@AlexHaxe
Copy link
Member

AlexHaxe commented Mar 2, 2016

yes, that's why I kept it open, I have to add a test case

@AlexHaxe AlexHaxe added the bug label Mar 2, 2016
@AlexHaxe AlexHaxe self-assigned this Mar 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants