use EvasObject#name to identify icons

[embrace.git] / bin / embrace

diff --git a/bin/embrace b/bin/embrace
index b82449c932e5558810d2aadc403ccd2e8a1e6f97..9772596e4280c26e21eeb6148f706d9049379765 100755 (executable)
--- a/bin/embrace
+++ b/bin/embrace
@@ -110,6 +110,8 @@ module Embrace
                def initialize(evas, label)
                        super(evas)
 
+                       self.name = label
+
                        @slot = nil
                        @alpha_anim = nil
 
@@ -120,22 +122,17 @@ module Embrace
                        @objects.each { |o| add_member(o) }
 
                        @img.set_color(255, 255, 255, 0)
-                       @label.set_color(255, 255, 255, 0)
                        @label.set_color(255, 0, 0, 0)
 
                        @img.set_file(ICON_FILE)
                        @img.set_fill(0, 0, *@img.get_size)
 
-                       @label.text = label
+                       @label.text = name
                        @label.set_font("VeraBd", 10)
 
                        resize(*@img.get_size)
                end
 
-               def label
-                       @label.text
-               end
-
                # smart callbacks
                def on_show
                        @objects.each { |o| o.show }
@@ -171,35 +168,11 @@ module Embrace
                end
        end
 
-       class FixedSizeArray < Array
-               def initialize(siz)
-                       super
-               end
-
-               def each
-                       super { |item| yield item unless item.nil? }
-               end
-
-               def delete_at(i)
-                       self[i] = nil
-               end
-
-               undef :push
-               undef :<<
-               undef :unshift
-
-               undef :pop
-               undef :shift
-               undef :delete
-       end
-
        class Container < Evas::Smart
                class ContainerError < StandardError; end
                class ContainerFullError < ContainerError; end
                class ContainerLockedError < ContainerError; end
 
-               include Enumerable
-
                def initialize(evas)
                        super
 
@@ -208,20 +181,16 @@ module Embrace
 
                        add_member(@bg)
 
-                       @icons = FixedSizeArray.new(MAX_ICONS)
-                       @about_to_add = []
+                       @icons = []
                        @animators = []
 
-                       @lock_count = 0
-               end
-
-               def each
-                       @icons.each { |i| yield i unless i.nil? }
+                       @about_to_add = 0
+                       @add_lock_count = 0
                end
 
                def <<(i)
                        Kernel.raise(ContainerFullError) if slots_left.zero?
-                       Kernel.raise(ContainerLockedError) if @lock_count > 0
+                       Kernel.raise(ContainerLockedError) unless @add_lock_count.zero?
 
                        i.move_relative(self, 0, 0)
                        i.slot = next_slot
@@ -236,7 +205,7 @@ module Embrace
 
                        movement = Main.instance.icon_height * (slots_left - 1)
 
-                       @about_to_add << i
+                       @about_to_add += 1
 
                        move_time = 0.25 * slots_left
                        @animators << MoveAnimator.new(move_time, movement, i)
@@ -245,15 +214,11 @@ module Embrace
                                @animators.delete(ani)
                                @icons[i.slot] = i
 
-                               # FIXME check whether we can always shift the array instead
-                               #puts "really added #{i.label} now (slot #{i.slot})"
-                               @about_to_add.delete(i)
+                               @about_to_add -= 1
                        end
                end
 
                def delete(icon)
-                       # icons that are placed above the one that's deleted need
-                       # to be moved
                        i = @icons.index(icon)
                        return (block_given? ? yield : nil) if i.nil?
 
@@ -261,25 +226,27 @@ module Embrace
                end
 
                def delete_at(i)
+                       Kernel.raise(ContainerLockedError) unless @about_to_add.zero?
+                       Kernel.raise(ContainerLockedError) unless @add_lock_count.zero?
+
+                       # icons that are placed above the one that's deleted need
+                       # to be moved
+                       ar = @icons[(i + 1)..-1]
+
                        @icons[i].delete
                        @icons.delete_at(i)
 
-                       ar = @icons[i..-1].reject { |i| i.nil? }
-                       return if ar.nil?
+                       return if ar.empty?
 
-                       @lock_count += 1
+                       @add_lock_count += 1
 
                        @animators << MoveAnimator.new(2, Main.instance.icon_height, *ar)
                        @animators.last.on_finished do |ani|
                                @animators.delete(ani)
-                               @lock_count -= 1
+                               @add_lock_count -= 1
                        end
                end
 
-               def length
-                       @icons.nitems
-               end
-
                # smart callbacks
                def on_show
                        @bg.show
@@ -304,11 +271,11 @@ module Embrace
 
                private
                def slots_left
-                       MAX_ICONS - @icons.nitems - @about_to_add.length
+                       MAX_ICONS - @icons.nitems - @about_to_add
                end
 
                def next_slot
-                       @icons.nitems + @about_to_add.length
+                       @icons.nitems + @about_to_add
                end
        end
 
@@ -383,7 +350,7 @@ module Embrace
                                lbl = md.captures.first
                        end
 
-                       found = @container.find { |i| i.label == lbl }
+                       found = evas.find_object(lbl)
 
                        begin
                                if ev.count.zero?