diff --git a/mercurial/cext/manifest.c b/mercurial/cext/manifest.c
--- a/mercurial/cext/manifest.c
+++ b/mercurial/cext/manifest.c
@@ -103,8 +103,6 @@ static PyObject *hashflags(line *l)
 {
 	char flag;
 	PyObject *hash = nodeof(l, &flag);
-	ssize_t hlen;
-	Py_ssize_t hplen, flen;
 	PyObject *flags;
 	PyObject *tup;
 
@@ -296,7 +294,6 @@ static PyObject *lmiter_iterentriesnext(
 	Py_ssize_t pl;
 	line *l;
 	char flag;
-	Py_ssize_t consumed;
 	PyObject *ret = NULL, *path = NULL, *hash = NULL, *flags = NULL;
 	l = lmiter_nextline((lmIter *)o);
 	if (!l) {
diff --git a/mercurial/cext/revlog.c b/mercurial/cext/revlog.c
--- a/mercurial/cext/revlog.c
+++ b/mercurial/cext/revlog.c
@@ -752,7 +752,6 @@ static int add_roots_get_min(indexObject
                              char phase)
 {
 	Py_ssize_t len = index_length(self);
-	PyObject *iter;
 	PyObject *item;
 	PyObject *iterator;
 	int rev, minrev = -1;
@@ -790,14 +789,12 @@ static PyObject *compute_phases_map_sets
 	static const char trackedphases[] = {1, 2, 32, 96};
 	PyObject *ret = NULL;
 	PyObject *roots = Py_None;
-	PyObject *idx = NULL;
 	PyObject *pyphase = NULL;
 	PyObject *pyrev = NULL;
 	PyObject *phaseroots = NULL;
 	PyObject *phasessize = NULL;
 	PyObject *phasesets[4] = {NULL, NULL, NULL, NULL};
 	Py_ssize_t len = index_length(self);
-	const char *currentphase;
 	char *phases = NULL;
 	int minphaserev = -1, rev, i;
 	const int numphases = (int)(sizeof(phasesets) / sizeof(phasesets[0]));
@@ -880,7 +877,7 @@ static PyObject *compute_phases_map_sets
 	phaseroots = _dict_new_presized(numphases);
 	if (phaseroots == NULL)
 		goto release;
-	for (int i = 0; i < numphases; ++i) {
+	for (i = 0; i < numphases; ++i) {
 		pyphase = PyInt_FromLong(trackedphases[i]);
 		if (pyphase == NULL)
 			goto release;