magic number - "-1" - is this an error code? an uninitialized value? what does it mean. make it const.