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.

Sign up for GitHub

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

Jump to bottom

case doesn't need brackets #120

Open

davidhesselbom opened this issue Sep 18, 2015 · 1 comment

Labels

Milestone

Contributor

davidhesselbom commented Sep 18, 2015

For example,

        match type {
            case TextureType monochrome =>
                this internalFormat = GL_R8
                this format = GL_RED
                this _bytesPerPixel = 1
            case TextureType rgba =>

works just fine, so magic should warn about unnecessary brackets in

        match type {
            case TextureType monochrome => {
                this internalFormat = GL_R8
                this format = GL_RED
                this _bytesPerPixel = 1
            }
            case TextureType rgba =>

and so on because brackets are ugly.

davidhesselbom added the enhancement label

davidhesselbom added this to the v0.2 milestone

Contributor Author

davidhesselbom commented Nov 18, 2015

Any ETA on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment