On Thu, 5 Mar 2026 04:17:30 GMT, Phil Race <[email protected]> wrote:

>> Where? In PrintJobDelegate what we had in the first version? But since we 
>> now always initialize the PageRange guess it will never be null and it has 
>> ran without that check since 1.1 so I guess we are ok..
>
> yes. What is the harm in avoiding an NPE ?

Not checking it actually allowed us to find this bug but I guess it's ok..Added 
it back..I see we check for null in PrintJobDelegate at l474 too so it's not 
without precedent

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/29874#discussion_r2887531579

Reply via email to