case var

[Simn]

See #6207 for discussion

[ncannasse]

Good for me.

…

On Mon, Sep 25, 2017 at 11:36 AM, Simon Krajewski ***@***.***> wrote: See #6207 <#6207> for discussion ------------------------------ You can view, comment on, or merge this pull request online at: #6608 Commit Summary - [matcher] allow and encourage `case var <ident>` (see #6207) - [std] add `case var` everywhere - [matcher] give similarity warnings for plain idents - fix tests File Changes - *M* src/syntax/parser.mly <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-0> (6) - *M* src/typing/matcher.ml <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-1> (40) - *M* src/typing/type.ml <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-2> (9) - *M* std/StringTools.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-3> (4) - *M* std/cs/_std/sys/io/Process.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-4> (2) - *M* std/haxe/io/Path.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-5> (2) - *M* std/haxe/macro/TypeTools.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-6> (2) - *M* std/haxe/rtti/Rtti.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-7> (4) - *M* std/java/_std/sys/io/Process.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-8> (4) - *M* std/java/internal/Exceptions.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-9> (4) - *M* std/php7/_std/StringTools.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-10> (2) - *M* std/python/_std/Sys.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-11> (4) - *M* tests/misc/projects/Issue4907/compile-fail.hxml.stderr <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-12> (3) - *M* tests/misc/projects/Issue5268/Main.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-13> (2) - *M* tests/unit/src/unit/TestMatch.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-14> (10) - *M* tests/unit/src/unit/issues/Issue2914.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-15> (8) - *M* tests/unit/src/unit/issues/Issue3031.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-16> (2) - *M* tests/unit/src/unit/issues/Issue3150.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-17> (2) - *M* tests/unit/src/unit/issues/Issue4677.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-18> (2) - *M* tests/unit/src/unit/issues/Issue5392.hx <https://github.com/HaxeFoundation/haxe/pull/6608/files#diff-19> (2) Patch Links: - https://github.com/HaxeFoundation/haxe/pull/6608.patch - https://github.com/HaxeFoundation/haxe/pull/6608.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#6608>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA-bwNXmkU3hCXVGPfJhRu8f5iqyg3S2ks5sl3QCgaJpZM4PicxS> .