[bug #29338] Crash using @synchronized on Windows

[Richard Frith-Macdonald]

Update of bug #29338 (project gnustep):

                  Status:                    None => Fixed                  
             Open/Closed:                    Open => In Test                

    _______________________________________________________

Follow-up Comment #4:

I looked into it, and it seems that the new sync.m just can't work
reliably/properly because it depends on unregistering classes (not supported
by the underlying runtime).
The workaround would be to simply leak the classes and never unregister them
... probably not acceptable.
This code also incorrectly assumes that it has the mutex allocated in
correctly aligned memory ... which is fixable if a little messy.

I replaced sync.m with Greg's original version which, while much slower,
should actually work reliably.  In future, we could use a hashtable to improve
performance somewhat if we want to.

I also modified the library to warn about bad/unsafe usage of
objc_disposeClassPair() and objc_registerClassPair()

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?29338>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/