The 'if (length)' check is seq_remove() is redundant because it is already asserted that length is not NULL.
Created attachment 120540 [details] [review]
seq_remove: remove redundant NULL check
I misunderstood when we talked earlier. This is correct.
Attachment 120540 [details] pushed as 6747efc - seq_remove: remove redundant NULL check