7 Jun
2023
7 Jun
'23
10:45 a.m.
On Wed Jun 7 10:20:27 2023 +0000, Alexandros Frantzis wrote:
changed this line in [version 3 of the diff](/wine/wine/-/merge_requests/2944/diffs?diff_id=50589&start_sha=cc2ba7ee7ea998f1b311d32b030bc46f3331c5e5#be6b1a6381414c8dae6f91e46942652a04a69f08_131_152)
I have kept the internal locking approach (more convenient as you say), and added a check in create to verify we do not already have `win_data`. To minimize the operations performed under lock, I went with a more optimistic concurrency approach, and the check is done after we have already created the new `win_data`.
I think NULL should not be accepted here.
Changed to `assert()`;