Don't check for ops in reverseOps before removing it, just try it.

[bcroq]

No description provided.

[joernhees]

not so much a comment for this change, but for my understanding: is this to "compress" the reverseOps only?

[bcroq]

In add() there was a bug: it checks if (s, p, o, ctxId, 'remove') is in self.reverseOps and if it is found it removes (s, p, o, context, 'add')

Then, it is generally a bad idea to check if an element is in a list before removing it. Just try to remove and deal with the ValueError exception.

if x in l:
    l.remove(x)
else:
    l.add(y)

becomes:

try:
    l.remove(x)
except ValueError:
    l.add(y)

It saves an uneeded call to __contains__