Various fixes for preproc

[sampottinger]

Addresses some small misc issues in preproc…

• Closes size(863,863/2) not working as first line of setup() function in a simple sketch #602. It will not rewrite if there is a method call or call with variable, passing up to compiler to determine if it is still valid. This is required because of someone overloading size or implementing their own (see Problem with function size(int arg, int arg) in Class #317).
• Most of Space oddities in .java code generated by preprocessor #607. See below.

[sampottinger]

@benfry - addressed #602 but looking at #607

[sampottinger]

@benfry - I was able to get most of this sorted except for indent. The problem is I have to keep the EDT and Javac in sync with each other when edits are made by preproc so errors and warnings go to the right place when they try to find token locations. However, newlines are not tokens on their own in Antlr. So, doing that indent will take a little more thought to ensure I don’t introduce any new bugs.

I’d recommend merging this without the indent fix and addressing indent in a separate PR.

[benfry]

Ok, the indent fix is more about avoiding the extra space before each public seen here:

I'm far less worried about adding indents (especially if ANTLR isn't giving you newlines, I totally get that), but we shouldn't be adding extra spaces in there.

[sampottinger]

@benfry - Sorry for the confusion. I think the extra space in front was resolved!

[benfry]

Perfect, thanks!

[benfry]

And #519 is still open, right? (Totally fine, just wrapping up the release here.)

[sampottinger]

Yes #519 is still open. :(

[benfry]

No problem, thanks again.