Add some typecast for the closure-compiler

src/ol/array.js

@@ -126,15 +126,13 @@ export function reverseSubArray(arr, begin, end) {
 
 /**
  * @param {Array.<VALUE>} arr The array to modify.
- * @param {Array.<VALUE>|VALUE} data The elements or arrays of elements
+ * @param {!Array.<VALUE>|VALUE} data The elements or arrays of elements to add to arr.
- *     to add to arr.
  * @template VALUE
  */
 export function extend(arr, data) {
-  let i;
   const extension = Array.isArray(data) ? data : [data];
   const length = extension.length;
-  for (i = 0; i < length; i++) {
+  for (let i = 0; i < length; i++) {
     arr[arr.length] = extension[i];
   }
 }
@@ -160,7 +158,7 @@ export function remove(arr, obj) {
  * @param {Array.<VALUE>} arr The array to search in.
  * @param {function(VALUE, number, ?) : boolean} func The function to compare.
  * @template VALUE
- * @return {VALUE} The element found.
+ * @return {VALUE|null} The element found or null.
  */
 export function find(arr, func) {
   const length = arr.length >>> 0;

src/ol/xml.js

@@ -420,11 +420,10 @@ export function parseNode(parsersNS, node, objectStack, opt_this) {
  * @return {T} Object.
  * @template T
  */
-export function pushParseAndPop(
+export function pushParseAndPop(object, parsersNS, node, objectStack, opt_this) {
-  object, parsersNS, node, objectStack, opt_this) {
   objectStack.push(object);
   parseNode(parsersNS, node, objectStack, opt_this);
-  return objectStack.pop();
+  return /** @type {T} */ (objectStack.pop());
 }
 
 
@@ -491,10 +490,8 @@ export function serialize(
  * @return {O|undefined} Object.
  * @template O, T
  */
-export function pushSerializeAndPop(object,
+export function pushSerializeAndPop(object, serializersNS, nodeFactory, values, objectStack, opt_keys, opt_this) {
-  serializersNS, nodeFactory, values, objectStack, opt_keys, opt_this) {
   objectStack.push(object);
-  serialize(
+  serialize(serializersNS, nodeFactory, values, objectStack, opt_keys, opt_this);
-    serializersNS, nodeFactory, values, objectStack, opt_keys, opt_this);
+  return /** @type {O|undefined} */ (objectStack.pop());
-  return objectStack.pop();
 }