Seems like a great feature to me.
You should probably write Closes #6045 instead of Closes issue #6045 to link the issue and pull request.
See also: Issue-linking keywords

Oh, I thought Closes issue would work the same as Closes.
Thanks for pointing it out! I'll make the changes.
Glad you liked the feature.

Thanks @KartikSoneji for this helpful and complete patch. I'd be happy to incorporate this with two changes:

1. If you'd be willing to move the “friendly words” file to a file named friendly.txt in build/shared/lib, and load that on first use instead of making it a class, i.e. something like:

	if (friendlyWords == null) {
        friendlyWords = PApplet.loadStrings(Base.getLibFile("friendly.txt"));
    }

2. Make this a PR against the Processing 4 repo where current development is happening.

Thanks again…

Hi @benfry !

Thanks for reviewing the changes, and I am glad you liked them.
I'll start porting the patch to Processing 4.

Can you please explain the benefits of loading the list of words on first use as opposed to having everything self-contained in one .class file?
I assume it is to decouple the list of words from the program source, but the list is unlikely to change.
On the other hand, loading a file adds additional complexity and one more point of failure (what happens if it cannot be loaded?)

Implemented for 4.0 beta 9, arriving shortly.

This pull request has been automatically locked. Pull requests that have been closed are automatically locked 30 days after the last comment.