[don’t merge] Add locks for countlabels in cardslist

[phughk]

This is related to #11848

The diff introduces a read write lock to the code for the countlist.

The count list is not accessed as reads only write locks in the diff.

This should prevent the error from occuring as far as I can see.

[JayDi85]

Is it reproduce able in every try or it's a random error?

As info:

Swing (GUI library) thread process all GUI related code in single thread by messages/events queue. So any modification must be done in same thread (or be a thread safe, synchronized, etc).

There are two tools for it:

• ThreadUtils.ensureRunInGUISwingThread() -- make sure your code running in swing thread (it allows to find some bad use cases);
• SwingUtilities.invokeLater(() -> {xxx}) -- execute code in swing thread. All GUI modification code must be called from it. By default any panels and other GUI related methods run in swing thread already. But some third party events must be not (example: game events from a server);

So maybe it need more research and fix of the original code/event processing. Original error starts here due stack (double clicks):

[phughk]

Is it reproduce able in every try or it's a random error?

I had 3 drafts fail when I was "playing seriously" and 1 succeed when I was just picking first card available and random deck from drafted.

There are two tools for it:

Ah, so actually you mean this is a threading issue? As in the threads should not be outside the GUI thread and throw exceptions? Can try fixing that thank you

[phughk]

Ticket status: needs investigation as per @.JayDi85 's comment