Add support for cancelling the builder and add a descriptive name for the builder

[johnpeb]

No description provided.

[ralfstx]

Hi John,

thank you so much for your contribution. Your changes look good to me. Do you think you could come up with a unit test for cancelling the builder? Do you agree to contribute your changes under the terms of the Eclipse Public License?

[johnpeb]

Hi Ralf,
I agree to contribute these changes under the terms of the Eclipse Public License.

[ralfstx]

Hi John,

that's great, thanks. So there's only a unit test missing. We shouldn't
include any untested code. If you could add a test, this would be great. I
won't find the time to do it over next two or three weeks.

Thanks, Ralf

[johnpeb]

Hi Ralf,
We've been testing this code on our internal update site for a few months. It functions as designed. An outstanding issue is if it accidentally starts to process a long file (100K LOC concatenated file) then it's not possible to cancel and Eclipse still has to be killed. Unfortunately I won't be able to write a formal test until I have time to fix that issue.

[FagnerMartinsBrack]

TL;DR
I suppose he meant because of future regression issues.

[ralfstx]

I have no doubts that your code works as expected. We should have a unit test anyway. Nevermind, I'll add one.

As for the huge JS file, would you mind to open a new issue about that? We might define a (configurable) upper limit for the size of JS files to be checked.

[johnpeb]

Created issue #42

[rombert]

I'm really interested in this feature as well. Canceling a full workspace build is impossible right now and I need to kill Eclipse which is very inconvenient.