From 61c61adea1662d4a1bb219f648f90ddeb04e3379 Mon Sep 17 00:00:00 2001
From: Julien Fontanet <julien.fontanet@isonoe.net>
Date: Thu, 21 Jan 2016 15:42:34 +0100
Subject: [PATCH] Minor code dedup.

---
 packages/xo-collection/src/collection.js | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/packages/xo-collection/src/collection.js b/packages/xo-collection/src/collection.js
index 9875b4ff5..2a8290481 100644
--- a/packages/xo-collection/src/collection.js
+++ b/packages/xo-collection/src/collection.js
@@ -113,20 +113,14 @@ export default class Collection extends EventEmitter {
   }
 
   clear () {
-    forEach(this._items, (_, key) => {
-      delete this._items[key]
-      this._size--
-      this._touch(ACTION_REMOVE, key)
-    })
+    forEach(this._items, (_, key) => this._remove(key))
   }
 
   remove (keyOrObjectWithId) {
     const [key] = this._resolveItem(keyOrObjectWithId)
     this._assertHas(key)
 
-    delete this._items[key]
-    this._size--
-    this._touch(ACTION_REMOVE, key)
+    this._remove(key)
   }
 
   set (keyOrObjectWithId, valueIfKey = undefined) {
@@ -157,9 +151,7 @@ export default class Collection extends EventEmitter {
     const [key] = this._resolveItem(keyOrObjectWithId)
 
     if (this.has(key)) {
-      delete this._items[key]
-      this._size--
-      this._touch(ACTION_REMOVE, key)
+      this._remove(key)
     }
   }
 
@@ -324,6 +316,12 @@ export default class Collection extends EventEmitter {
     return typeof key === 'number' || typeof key === 'string'
   }
 
+  _remove (key) {
+    delete this._items[key]
+    this._size--
+    this._touch(ACTION_REMOVE)
+  }
+
   _resolveItem (keyOrObjectWithId, valueIfKey = undefined) {
     if (valueIfKey !== undefined) {
       this._assertValidKey(keyOrObjectWithId)