Address potential races in recent event changes

src/cswinrt/code_writers.h

@@ -6488,11 +6488,12 @@ bind<write_type_name>(type, typedef_name_type::CCW, true)
                 {
                     handler = (%) =>
                     {
-                        if (_state.del == null)
+                        % localDel = _state.del;
+                        if (localDel == null)
                         {
                             return %;
                         }
-                        %_state.del.Invoke(%);
+                        %localDel.Invoke(%);
                     };
                 }
                 return handler;
@@ -6506,6 +6507,7 @@ bind<write_type_name>(type, typedef_name_type::CCW, true)
                     eventTypeCode,
                     bind<write_event_source_type_name>(eventTypeSemantics), 
                     bind<write_event_invoke_params>(invokeMethodSig),
+                    eventTypeCode,
                     bind<write_event_invoke_return_default>(invokeMethodSig),
                     bind<write_event_invoke_return>(invokeMethodSig),
                     bind<write_event_invoke_args>(invokeMethodSig));

src/cswinrt/strings/WinRT.cs

@@ -370,7 +370,10 @@ public void Subscribe(TDelegate del)
         {
             lock (this)
             {
-                if (_state.del is null)
+                bool registerHandler = _state.del is null;
+
+                _state.del = (TDelegate)global::System.Delegate.Combine(_state.del, del);
+                if (registerHandler)
                 {
                     var marshaler = CreateMarshaler((TDelegate)EventInvoke);
                     try
@@ -385,7 +388,6 @@ public void Subscribe(TDelegate del)
                         DisposeMarshaler(marshaler);
                     }
                 }
-                _state.del = (TDelegate)global::System.Delegate.Combine(_state.del, del);
             }
         }
 
@@ -445,10 +447,10 @@ private class Cache
             }
 
             private IWeakReference target;
-            private ConcurrentDictionary<int, EventSource<TDelegate>.State> states = new ConcurrentDictionary<int, EventSource<TDelegate>.State>();
+            private readonly ConcurrentDictionary<int, EventSource<TDelegate>.State> states = new ConcurrentDictionary<int, EventSource<TDelegate>.State>();
 
-            private static ReaderWriterLockSlim cachesLock = new ReaderWriterLockSlim();
-            private static ConcurrentDictionary<IntPtr, Cache> caches = new ConcurrentDictionary<IntPtr, Cache>();
+            private static readonly ReaderWriterLockSlim cachesLock = new ReaderWriterLockSlim();
+            private static readonly ConcurrentDictionary<IntPtr, Cache> caches = new ConcurrentDictionary<IntPtr, Cache>();
 
             private Cache Update(IWeakReference target, EventSource<TDelegate> source, int index)
             {
@@ -579,8 +581,9 @@ override protected System.Delegate EventInvoke
                 {
                     handler = (System.Object obj, T e) =>
                     {
-                        if (_state.del != null)
-                            _state.del.Invoke(obj, e);
+                        System.EventHandler<T> delLocal = _state.del;
+                        if (delLocal != null)
+                            delLocal.Invoke(obj, e);
                     };
                 }
                 return handler;