Go back to using vector to store devices in a Hub

[libs/gui.git] / source / input / hub.cpp

diff --git a/source/input/hub.cpp b/source/input/hub.cpp
index 4a2d5533aafeb36bea172bee347e0bd4d25ae184..635ea8a6c935fe9485fee30c87dc55386a3f56ed 100644 (file)
--- a/source/input/hub.cpp
+++ b/source/input/hub.cpp
@@ -1,5 +1,6 @@
 #include <stdexcept>
 #include <sigc++/bind.h>
+#include <sigc++/bind_return.h>
 #include <msp/core/hash.h>
 #include <msp/core/maputils.h>
 #include "hub.h"
@@ -16,41 +17,49 @@ Hub::Hub()
 
 void Hub::attach(Device &dev)
 {
-       unsigned tag = hash32(dev.get_name(), 20);
-       while(devices.count(tag))
-               ++tag;
-
-       devices[tag] = &dev;
-       dev.signal_button_press.connect(sigc::bind(sigc::mem_fun(this, &Hub::button_press), tag));
-       dev.signal_button_release.connect(sigc::bind(sigc::mem_fun(this, &Hub::button_release), tag));
-       dev.signal_axis_motion.connect(sigc::bind(sigc::mem_fun(this, &Hub::axis_motion), tag));
+       unsigned index = devices.size();
+       devices.push_back(&dev);
+       dev.signal_button_press.connect(sigc::bind_return(sigc::bind(sigc::mem_fun(this, &Hub::button_press), index), false));
+       dev.signal_button_release.connect(sigc::bind_return(sigc::bind(sigc::mem_fun(this, &Hub::button_release), index), false));
+       dev.signal_axis_motion.connect(sigc::bind_return(sigc::bind(sigc::mem_fun(this, &Hub::axis_motion), index), false));
 }
 
 std::string Hub::get_button_name(unsigned btn) const
 {
-       const Device &dev = *get_item(devices, btn>>12);
-       return dev.get_name()+": "+dev.get_button_name(btn&0xFFF);
+       unsigned dev_index = btn>>8;
+       if(dev_index>=devices.size())
+               throw out_of_range("Hub::get_button_name");
+
+       const Device &dev = *devices[dev_index];
+       return dev.get_name()+": "+dev.get_button_name(btn&0xFF);
 }
 
 std::string Hub::get_axis_name(unsigned axis) const
 {
-       const Device &dev = *get_item(devices, axis>>12);
-       return dev.get_name()+": "+dev.get_axis_name(axis&0xFFF);
+       unsigned dev_index = axis>>8;
+       if(dev_index>=devices.size())
+               throw out_of_range("Hub::get_axis_name");
+
+       const Device &dev = *devices[dev_index];
+       return dev.get_name()+": "+dev.get_axis_name(axis&0xFF);
 }
 
-void Hub::button_press(unsigned btn, unsigned tag)
+void Hub::button_press(unsigned btn, unsigned index)
 {
-       set_button_state((tag<<12) | (btn&0xFFF), true, true);
+       if(btn<0x100)
+               set_button_state((index<<8) | btn, true, true);
 }
 
-void Hub::button_release(unsigned btn, unsigned tag)
+void Hub::button_release(unsigned btn, unsigned index)
 {
-       set_button_state((tag<<12) | (btn&0xFFF), false, true);
+       if(btn<0x100)
+               set_button_state((index<<8) | btn, false, true);
 }
 
-void Hub::axis_motion(unsigned axis, float value, float, unsigned tag)
+void Hub::axis_motion(unsigned axis, float value, float, unsigned index)
 {
-       set_axis_value((tag<<12) | (axis&0xFFF), value, true);
+       if(axis<0x100)
+               set_axis_value((index<<8) | axis, value, true);
 }
 
 } // namespace Input