Do nut use name annotation and avoid changing types

src/ol/Object.js

@@ -94,10 +94,10 @@ class BaseObject extends Observable {
     getUid(this);
 
     /**
-     * @name values_
      * @private
      * @type {!Object<string, *>}
      */
+    this.values_ = null;
 
     if (opt_values !== undefined) {
       this.setProperties(opt_values);

src/ol/events/Target.js

@@ -39,22 +39,22 @@ class Target extends Disposable {
     this.eventTarget_ = opt_target;
 
     /**
-     * @name pendingRemovals_
      * @private
-     * @type {!Object<string, number>}
+     * @type {Object<string, number>}
      */
+    this.pendingRemovals_ = null;
 
     /**
-     * @name dispatching_
      * @private
-     * @type {!Object<string, number>}
+     * @type {Object<string, number>}
      */
+    this.dispatching_ = null;
 
     /**
-     * @name listeners_
      * @private
-     * @type {!Object<string, Array<import("../events.js").Listener>>}
+     * @type {Object<string, Array<import("../events.js").Listener>>}
      */
+    this.listeners_ = null;
   }
 
   /**
@@ -65,10 +65,10 @@ class Target extends Disposable {
     if (!type || !listener) {
       return;
     }
-    let listeners = this.listeners_ || (this.listeners_ = {});
+    const listeners = this.listeners_ || (this.listeners_ = {});
-    listeners = listeners[type] || (listeners[type] = []);
+    const listenersForType = listeners[type] || (listeners[type] = []);
-    if (listeners.indexOf(listener) === -1) {
+    if (listenersForType.indexOf(listener) === -1) {
-      listeners.push(listener);
+      listenersForType.push(listener);
     }
   }