Saving sketch with the same name as a class #196

[iamutkarshtiwari]

This patch is fix to the issue -> #196, #305

I have generated a list of all the possible reserved keywords( in Processing and Java language) in a _"reservedKeywords.txt"_ file in _build/shared/_ (which will be later copied to build/linux/work/ during setup build) folder through a script(https://github.com/iamutkarshtiwari/ReservedWordsGenerator). Whenever the user will save a sketch with a new name, _sanitizeName()_ will search for that name in 'reservedKeywords.txt" file. If the name exist in that file, it will prompt the user to save with any another filename but inbuilt class name.

[iamutkarshtiwari]

Changes made in _build.xml_ on lines-> _631, 778, 1064, 1068.
I also cleaned up a lot of trailing whitespaces in _build.xml file.

[iamutkarshtiwari]

You may test this patch by saving the sketch with any _Processing class names_ (like _PImage, PApplet, PSurfaceNone, etc) or with _reserved Keywords(like _abstract, int, long, boolean, etc_) in Java.

[scotthmurray]

@iamutkarshtiwari This is a needed feature. I stumbled on this the other day, unintentionally saving a sketch using a name of an existing class or method. I don't remember the specific outcome, but things went haywire. Thanks for looking at this!

[iamutkarshtiwari]

@alignedleft Did you test this patch?

[scotthmurray]

@iamutkarshtiwari No, that’s not really my role. I was just voicing support for the idea.

[GKFX]

@iamutkarshtiwari Some little criticisms:
Why are keywords from processing.app reserved? I tested a sketch called CopyAsHtmlAction (without applying your patch) and it runs fine.
There's a space missing before the bracket in Name '%s' is a not allowed(reserved keyword).
You've forgotten to close or try-with-resources your FileReader and BufferedReader.