Quadstep library - serial sending "error: incorrect value for motor_go"

Other Hardware Motors, Mechanics, Power and CNC
1

Hi there,
I am having a rare issue.
I a having a strange error message when I call the motor_go() function, which is "error: incorrect value for motor_go"

It seems that everything is working but I cant understand why this message appears. Can someone take a look in the library files so we found out what causes this problem? The serial command can be found in quadstep.cpp file. I took a look but had no luck.
I think I am using everythin correctly.

I also havent soldered anything on the quadstep board.

Please help!
Code:

/*
  SparkFun Electronics 2011
  Aaron Weiss, aaron at sparkfun dot com
  Beer-ware License: You can do whatever you want with this sketch.
                     If we meet someday, you can buy me a beer.

  QuadSetpper Example Sketch. For use with an Arduino Mega2560 and
  a 1.8 degree bipolar stepper motor.

  You must connect the STP pin for each motor as shown below.
  For rest of the pins, you choose where they go.

  Motor1 STP pin: mega pin 11
  Motor2 STP pin: meag pin 5
  Motor3 STP pin: meag pin 6
  Motor4 STP pin: mega pin 46

  Library Usage:

  motor_pins(x,y,z,l,m,n)
  x: motor channel number
  y: enable pin assignment
  z: direction pin assignment
  l: MS1 pin assignment
  m: MS2 pin assignment
  n: MS3 pin assignment

  motor_go(x,y,z,l)
  x: motor channel number
  y: step size: 1, 2, 4, 8, or 16
  z: increments for given step size
     for full step: 1 increment = 1.8deg
     for half step: 1 increment = 0.9deg and so on
     negaitve numbers rotate in the opposite direction
  l: torque/speed (0-10), 0 is high speed/low torque/low current
                          10 is low speed/high torque/high current (2.0A max)

  stall(x)
  x: motor channel number
*/

// include the motor library
#include <quadstep.h>

// create an instance of the class motor
quadstep quadstep;

void setup() {
  Serial.begin(9600);
  // assign the pin connections
  quadstep.motor_pins(1, A1, 36, A8, A9, A10); //ch 1
  quadstep.motor_pins(2, 10, 9, 8, 7, 4); //ch 2
  quadstep.motor_pins(3, 22, 23, 24, 25, 26); //ch 3
  quadstep.motor_pins(4, 27, 28, 29, 30, 31); //ch 4

}

//1.8deg = 1step
void loop() {

  // step motor 4 for 200 increments CW
  quadstep.motor_go(1, 8, 50, 2);
  delay(5000);
  // step motor 4 for 200 increments CCW
  quadstep.motor_go(1, 8, -50, 2);
  delay(2000);

  // holds the motor in one position with full torque/current
  //quadstep.stall(1);
  //delay(500);

  // step motor 4 for 200 increments CW
  quadstep.motor_go(2, 1, 30, 2);
  delay(2000);
  // step motor 4 for 200 increments CCW
  quadstep.motor_go(2, 1, -30, 2);
  delay(2000);

  // holds the motor in one position with full torque/current
  //quadstep.stall(1);
  //delay(500);


  // step motor 4 for 200 increments CW
  quadstep.motor_go(3, 16, 20, 2);
  delay(2000);
  // step motor 4 for 200 increments CCW
  quadstep.motor_go(3, 16, -20, 2);
  delay(2000);

  // holds the motor in one position with full torque/current
  // quadstep.stall(1);
  // delay(500);
}

Debug:

error: incorrect value for motor_go
error: incorrect value for motor_go
error: incorrect value for motor_go
error: incorrect value for motor_go
error: incorrect value for motor_go
error: incorrect value for motor_go
error: incorrect value for motor_go
error: incorrect value for motor_go
error: incorrect value for motor_go
error: incorrect value for motor_go
2

Its a bug in the code of the library - the consecutive tests for the motor
numbers in motor_go use "if" instead of "else if", so that the last "else"
that produces the error message fires whenever you call motor_go()
on a motor other than 4.

The code is extremely repetitious and wasteful of program space too...

Urhghgh!