CoordinateSequence: Raise error on invalid ordinateIndex

[dbaston]

As discussed in #1244 (comment)

Mixed feelings about this, since the ability to set a Z value on an XY sequence without getting an error has existed since forever.

[dr-jts]

This is the right thing to do, to prevent incorrect code.

[mwtoews]

Maybe also add a test for XYM?

template<>
template<>
void object::test<25>()
{
    set_test_name("setOrdinate on XYM coordinate");

    /* TODO: use GEOSCoordSeq_createWithDimensions() instead */
    double buffer[3];
    cs_ = GEOSCoordSeq_copyFromBuffer(buffer, 1, 0, 1);

    ensure("setX", GEOSCoordSeq_setOrdinate(cs_, 0, 0, 1));
    ensure("setY", GEOSCoordSeq_setOrdinate(cs_, 0, 1, 2));
    ensure("setZ", !GEOSCoordSeq_setOrdinate(cs_, 0, 2, 3));
    ensure("setM", GEOSCoordSeq_setOrdinate(cs_, 0, 3, 4));

    double x, y, m;
    ensure(GEOSCoordSeq_copyToArrays(cs_, &x, &y, nullptr, &m));

    ensure_equals("X", x, 1);
    ensure_equals("Y", y, 2);
    ensure_equals("M", m, 4);
}