Added within functionality for range based inputs

Author: divanvisagie

lib/mixins/within.js

@@ -1,6 +1,6 @@
 var mixins = {
 
-  within: function(range, unit, callback) {
+  within: function(eventName, range, callback) {
     var upper;
 
     if (typeof range === "number") {
@@ -15,18 +15,7 @@ var mixins = {
       return;
     }
 
-    if (typeof unit === "function") {
-      callback = unit;
-      unit = "value";
-    }
-
-    if (typeof this[unit] === "undefined") {
-      return this;
-    }
-
-    // Use the continuous read event for high resolution
-    this.on("data", function() {
-      var value = this[unit] | 0;
+    this.on(eventName , function(value){
       if (value >= range[0] && value <= range[1]) {
         callback.call(this, null, value);
       }

lib/xbox.js

@@ -7,8 +7,10 @@ var HID = require('node-hid'),
   triggers = require('./triggers.json'),
   buttons = require('./buttons.json'),
   within = require('./mixins/within'),
+  __ = require('lodash');
   joysticks = require('./joysticks.json');
 
+
 var dead = 6000;
 
 function uint8Toint16(low, high) {
@@ -400,4 +402,6 @@ XboxController.prototype.rumble = function (left, right) {
   this.sendCommand([0x00, 0x00, 0x04, left, right], 'Xbox controller wont rumble');
 };
 
+__.mixin(XboxController.prototype, within);
+
 module.exports = XboxController;

tests/test.js

@@ -26,3 +26,8 @@ xbox.on('left:move', function (position) {
 xbox.on('right:move', function (position) {
   console.log('right:move', position)
 });
+
+
+xbox.within('righttrigger', [50,100], function(err, data){
+	console.log('rightttrigger within 50 and 100', err, data);
+});