avoid the set/get syndrome

Author: monkstone

processing_app/library/glvideo/Rakefile

@@ -0,0 +1,30 @@
+# Simple demo Rakefile to autorun samples in current directory
+# adjust path to jruby executable, and or opts as required
+
+SAMPLES_DIR = './'
+
+desc 'run demo'
+task default: [:demo]
+
+desc 'demo'
+task :demo do
+  samples_list.shuffle.each { |sample| run_sample sample }
+end
+
+def samples_list
+  files = []
+  Dir.chdir(SAMPLES_DIR)
+  Dir.glob('*.rb').each do |file|
+    files << File.join(SAMPLES_DIR, file)
+  end
+  return files
+end
+
+def run_sample(sample_name)
+  puts "Running #{sample_name}...quit to run next sample"
+  open("|jruby #{sample_name}", 'r') do |io|
+    while l = io.gets
+      puts(l.chop)
+    end
+  end
+end

processing_app/library/glvideo/library/corners/corners.rb

@@ -20,15 +20,15 @@ def initialize(width, height, ws, wh)
     @idx = -1
   end
 
-  def set_corner(mx, my)
+  def change_selected(mx, my)
     self[idx] = Vect.new(mx, my)
   end
 
   def selected?
     idx != -1
   end
 
-  def set_index(sel)
+  def select_corner(sel)
     @idx = sel
   end
 end

processing_app/library/glvideo/video_mapping.rb

@@ -10,6 +10,7 @@ class VideoMapping < Propane::App
 
   attr_reader :sources, :sel, :video, :corners, :quads, :last_mouse_move
   RES = 5	# number of subdivisions (e.g. 5 x 5)
+  NONE = -1
 
   def setup
     @last_mouse_move = 0
@@ -28,7 +29,7 @@ def draw
     video.read if sel.respond_to?(:read) && video.available
     # regenerate mesh if we're dragging a corner
     if corners.selected? && (pmouse_x != mouse_x || pmouse_y != mouse_y)
-      corners.set_corner(mouse_x, mouse_y)
+      corners.change_selected(mouse_x, mouse_y)
       # this improves performance, but will be replaced by a
       # more elegant way in a future release
       @quads = []
@@ -50,15 +51,16 @@ def draw
 
   def mouse_pressed
     corners.each_with_index do |corner, i|
-      return corners.set_index(i) if dist(mouse_x, mouse_y, corner.x, corner.y) < 20
+      return corners.select_corner(i) if dist(mouse_x, mouse_y, corner.x, corner.y) < 20
     end
     # no corner? then switch texture
     @sel = sel.respond_to?(:loop) ? sources.image : sources.video
     @quads = create_mesh(sel, corners, RES)
   end
 
   def mouse_released
-    corners.set_index(-1)
+    # no corner selected
+    corners.select_corner(NONE)
   end
 
   def create_mesh(tex, corners, res)