On Wed, 3 Sep 2025 01:41:44 GMT, Phil Race <[email protected]> wrote: > I think this all bears a bit more studying as to which code to update for the > fix - one of these has to go (although knowsPageRange probably needs to exist > and return YES, else the peek won't happen if I correctly read the Apple docs)
We have to return `YES` to `knowsPageRange`. Printing doesn't work otherwise. I am still for simplifying the code if we're accepting what's currently suggested: 1. Get rid of `totalPages`. ([comment](https://github.com/openjdk/jdk/pull/11266#discussion_r2298979119)) 2. Always use NSIntegerMax and remove the fTotalPages field. ([comment](https://github.com/openjdk/jdk/pull/11266#discussion_r2298982204)) ------------- PR Comment: https://git.openjdk.org/jdk/pull/11266#issuecomment-3547871808
