Hi
Thanks for the report and the test case.
Yes, that comment is wrong. It didn't consider that if the page number
is too high, the resulting page will be empty even though the first page
isn't.
I propose fixing it with
Hi all,
I got this crash report from the server at k.sfconservancy.org over the
weekend, and the comment in the last line jumped out at me. It sounds
like there's a situation where the code assumes a list can't be empty,
but in fact it can be. The full details are below, except I've removed
