various incorrect capture by ref ranges

[travisstaloch]

Zig Version

0.12.0-dev.2158+4f2009de1

Steps to Reproduce and Observed Behavior

I noticed while using zls a few pointer captures which don't account for capturing by reference. All of the following code snippets incorrectly highlight the asterisk instead of the identifier:

    switch (undefined) {
        .a => |*ident| {},
        //     ^ unused capture
        //      ^^^^^ range should be here
    }

    if (undefined) |*foo| {} else |err| {}
    //              ^ unused capture

    while (undefined) |*foo| {} else |err| {}
    //                 ^ unused capture

   // EDITED here, removed incorrect by ref capture

I have a PR in the works which I'll submit soon.

Thanks to @Techatrix for helping me discover some of these in the zls discord. I originally only noticed that the switch pointer capture was off and he helped spot the others.

Expected Behavior

correct ranges for unused captures

[travisstaloch]

I now realize that the 4th snippet shouldn't actually be allowed (by ref error capture) so I've edited it out and will open a new issue for that. After further discussion w/ @Techatrix, the following shouldn't be allowed and so I'm removing it from my PR.

    err_union catch |err| switch (err) {
        error.X => |*foo| {},
    };

[travisstaloch]

thanks @Techatrix for the guidance! i wouldn't have thought to do any of this w/out your advice.