Remove calls to the mk macro on final returns in parser.rs

[someguynamedmatt]

This urges a discussion:

• What do the Position attributes do in the Expr structs? They don't have a use currently, but I can't imagine they were thrown in there by Bebbington "just because".

• The solution proposed is probably more of a band-aid than an actual solution. At least until we can figure out what the start and end (Position) attrs are for.

[jasonwilliams]

nice one, just checked out and its working.
hmm im not sure either, was it so if there was an error processing an expression it would spit out the line/column number?
Maybe we should remove them if we don't need them for now?

[someguynamedmatt]

It might be that, but it also might be for more complex blocks. I tried to do a simple

if ( 1 === 1) {
   b = 'cat';
}

and it broke. That could have happened for any number of reasons, but I dunno as of yet.

Removing them might make sense. If we hit the "Oh crap, we need that" point, we can put them back in.

[jasonwilliams]

Whats your debugging set up at the moment? Are you just printing until it breaks or do you have some debugger up and running?

[someguynamedmatt]

println! until I narrow-down the issue. I'm primitive. What's your setup?

[jasonwilliams]

Same, vscode, been trying to get the debugger up and running, it works well but had an issue which is documented here:
vadimcn/codelldb#127