X-Git-Url: http://git.code-monkey.de/?a=blobdiff_plain;f=bin%2Fembrace;h=b586afe5b4c12e206a7f2c092e05fee384355456;hb=f201a1dce85804a7d4acde2af2b63dea795a69cc;hp=c657e7d6e3fc24166a654b592505e88bba3d8eb3;hpb=51ce043e8007319aae5b7eccffb85a060bb94bda;p=embrace.git

diff --git a/bin/embrace b/bin/embrace
index c657e7d..b586afe 100755
--- a/bin/embrace
+++ b/bin/embrace
@@ -117,9 +117,9 @@ module Embrace
 			@label = Evas::Text.new(evas)
 
 			@objects = [@img, @label]
+			@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)
@@ -159,18 +159,6 @@ module Embrace
 			@img = @label = @alpha_anim = nil
 		end
 
-		def on_layer_set(layer)
-			@objects.each { |o| o.layer = layer }
-		end
-
-		def on_stack_above(other)
-			@objects.each { |o| o.stack_above = other }
-		end
-
-		def on_stack_below(other)
-			@objects.each { |o| o.stack_below = other }
-		end
-
 		def on_move(x, y)
 			@objects.each { |o| o.move(x, y) }
 
@@ -182,28 +170,6 @@ 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
@@ -217,20 +183,22 @@ module Embrace
 			@bg = Evas::Rectangle.new(evas)
 			@bg.set_color(0, 0, 0, 255)
 
-			@icons = FixedSizeArray.new(MAX_ICONS)
-			@about_to_add = []
+			add_member(@bg)
+
+			@icons = []
 			@animators = []
 
-			@lock_count = 0
+			@about_to_add = 0
+			@add_lock_count = 0
 		end
 
 		def each
-			@icons.each { |i| yield i unless i.nil? }
+			@icons.each { |i| yield i }
 		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
@@ -245,7 +213,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)
@@ -254,15 +222,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?
 
@@ -270,25 +234,26 @@ module Embrace
 		end
 
 		def delete_at(i)
+			Kernel.raise(ContainerLockedError) unless @about_to_add.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
@@ -303,18 +268,6 @@ module Embrace
 			@bg = nil
 		end
 
-		def on_layer_set(layer)
-			@bg.layer = layer
-		end
-
-		def on_stack_above(other)
-			@bg.stack_above = other
-		end
-
-		def on_stack_below(other)
-			@bg.stack_below = other
-		end
-
 		def on_move(x, y)
 			@bg.move(x, y)
 		end
@@ -325,11 +278,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
 
@@ -341,12 +294,12 @@ module Embrace
 		def initialize
 			super
 
-			self.title = "blah"
+			self.title = "Embrace"
 			self.borderless = true
 
 			@icon_dim = IO.read(ICON_FILE, 8, 16).unpack("NN")
 
-			self.on_resize { @container.resize(*geometry[2, 3]) }
+			on_resize { @container.resize(*geometry[2, 3]) }
 
 			@container = Container.new(evas)
 			@container.move(0, 0)