[julian-smith-artifex-com]

• Fix Document.delete_pages() raises ValueError: bad page number(s) on negative page numbers passed as two positional arguments #5240.
• Use % operator instead of loop to wrap -ve values.

[JorjMcKie]

Question: why do we need if start < 0: at all? start %= 360 will always deliver a value in the range [0, 360), no matter what the initial start is. Saves the value check.

[julian-smith-artifex-com]

This would change the behaviour of out-of-range +ve values - at the moment these are errors.

There's a benefit to wrapping -ve values (e.g. -1 for last item is convenient). But there is not a corresponding benefit for accepting out-of-range +ve values, and i think such values are likely to be a mistake that we should not hide.