From 8a9df9d15a8d3e42280fe39836800d99f76b17f7 Mon Sep 17 00:00:00 2001
From: Mayhem <stepien.nicolas@gmail.com>
Date: Sat, 20 Jul 2013 18:19:51 +0200
Subject: [PATCH] Tiny optimization to avoid calling QR.status() multiple times
 when closing the QR.

---
 src/Posting/QR.coffee | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/Posting/QR.coffee b/src/Posting/QR.coffee
index fdaa06e0c..8319392a2 100644
--- a/src/Posting/QR.coffee
+++ b/src/Posting/QR.coffee
@@ -74,8 +74,8 @@ QR =
     QR.cleanNotifications()
     d.activeElement.blur()
     $.rmClass QR.nodes.el, 'dump'
-    for i in QR.posts
-      QR.posts[0].rm()
+    for post in QR.posts.splice 0, QR.posts.length, new QR.post true
+      post.delete()
     QR.cooldown.auto = false
     QR.status()
   focusin: ->
@@ -494,15 +494,17 @@ QR =
       @select() if select
       @unlock()
     rm: ->
-      $.rm @nodes.el
+      @delete()
       index = QR.posts.indexOf @
       if QR.posts.length is 1
         new QR.post true
       else if @ is QR.selected
         (QR.posts[index-1] or QR.posts[index+1]).select()
       QR.posts.splice index, 1
-      URL.revokeObjectURL @URL
       QR.status()
+    delete: ->
+      $.rm @nodes.el
+      URL.revokeObjectURL @URL
     lock: (lock=true) ->
       @isLocked = lock
       return unless @ is QR.selected