Stricter checks for configured, and default, bounded timings.

Author: dok-net

libraries/Servo/src/Servo.cpp

@@ -105,14 +105,13 @@ void Servo::write(int value)
     // assumed to be 0-180 degrees servo
     value = constrain(value, 0, 180);
     value = improved_map(value, 0, 180, _minUs, _maxUs);
-  } else if (value > _maxUs) {
-    value = _maxUs;
   }
   writeMicroseconds(value);
 }
 
 void Servo::writeMicroseconds(int value)
 {
+  value = constrain(value, _minUs, _maxUs);
   _valueUs = value;
   if (_attached) {
     _servoMap &= ~(1 << _pin);

libraries/Servo/src/Servo.h

@@ -46,8 +46,8 @@
 
 // the following are in us (microseconds)
 //
-#define DEFAULT_MIN_PULSE_WIDTH       800 // the shortest duty cycle sent to a servo  
-#define DEFAULT_MAX_PULSE_WIDTH      2190 // the longest duty cycle sent to a servo 
+#define DEFAULT_MIN_PULSE_WIDTH      1000 // uncalibrated default, the shortest duty cycle sent to a servo
+#define DEFAULT_MAX_PULSE_WIDTH      2000 // uncalibrated default, the longest duty cycle sent to a servo 
 #define DEFAULT_NEUTRAL_PULSE_WIDTH  1500 // default duty cycle when servo is attached
 #define REFRESH_INTERVAL            20000 // classic default period to refresh servos in microseconds 
 #define MAX_SERVOS                      9 // D0-D8