comparison mercurial/cext/revlog.c @ 38852:a3dacabd476b

index: don't allow index[len(index)] to mean nullid

Now everything else has been cleaned up and we can drop support for
getting the nullid from the end of the index (from *past* the end
actually, since we reduced the length in the previous patch).

Differential Revision: https://phab.mercurial-scm.org/D4023

38851:781b2720d2ac

	const char *c_node_id;
	const char *data;
	Py_ssize_t length = index_length(self) + 1;
	PyObject *entry;

	if (pos == -1 || pos == length - 1) {
		Py_INCREF(nullentry);
		return nullentry;
	}

	if (pos < 0 || pos >= length) {
		PyErr_SetString(PyExc_IndexError, "revlog index out of range");
		return NULL;
	}

	if (pos >= self->length - 1) {

38852:a3dacabd476b

	const char *c_node_id;
	const char *data;
	Py_ssize_t length = index_length(self) + 1;
	PyObject *entry;

	if (pos == -1) {
		Py_INCREF(nullentry);
		return nullentry;
	}

	if (pos < 0 || pos >= length - 1) {
		PyErr_SetString(PyExc_IndexError, "revlog index out of range");
		return NULL;
	}

	if (pos >= self->length - 1) {