A Solidity library to remove elements from an array.

ArrayUtil.sol

library ArrayUtil {

  /** Finds the index of a given value in an array. */
  function IndexOf(uint[] values, uint value) returns(uint) {
    uint i = 0;
    while (values[i] != value) {
      i++;
    }
    return i;
  }

  /** Removes the given value in an array. */
  function RemoveByValue(uint[] values, uint value) {
    uint i = IndexOf(value);
    RemoveByIndex(i);
  }

  /** Removes the value at the given index in an array. */
  function RemoveByIndex(uint[] values, uint i) {
    while (i<values.length-1) {
      values[i] = values[i+1];
      i++;
    }
    values.length--;
  }
}

your function is going to go in an infinite loop if the element to be deleted is not present

You should make a new variable length. It won't compile at least in the latest versions of solidity.

And you don't pass values(array on which to execute operation) anywhere.

You're not receiving the uint[] as a storage, I believe it will create a copy