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 b28882f6aad8733944f9002d4abfe6c308c1088f..635ea8a6c935fe9485fee30c87dc55386a3f56ed 100644 (file)
--- a/source/input/hub.cpp
+++ b/source/input/hub.cpp
@@ -17,41 +17,49 @@ Hub::Hub()
 
 void Hub::attach(Device &dev)
 {
-       unsigned tag = hash<20>(dev.get_name());
-       while(devices.count(tag))
-               ++tag;
-
-       devices[tag] = &dev;
-       dev.signal_button_press.connect(sigc::bind_return(sigc::bind(sigc::mem_fun(this, &Hub::button_press), tag), false));
-       dev.signal_button_release.connect(sigc::bind_return(sigc::bind(sigc::mem_fun(this, &Hub::button_release), tag), false));
-       dev.signal_axis_motion.connect(sigc::bind_return(sigc::bind(sigc::mem_fun(this, &Hub::axis_motion), tag), false));
+       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