On Fri, 19 Apr 2024 05:10:46 GMT, Tejesh R <[email protected]> wrote: > I wasn't able to make out the changes with comparison since restructuring is > also done here. Just like you are explaining here would help in understanding > the changes for review.
Putting all access to `pageLoader` into a `synchronized` block solves the thread-safety problem. At the same time, we can improve the readability of the method as a whole. Restructuring helps make the intent clearer, at least I'm aiming to do so. I agree looking at the diffs here doesn't give you much insight, but going through the original and updated code gives a clearer picture. ------------- PR Comment: https://git.openjdk.org/jdk/pull/18670#issuecomment-2066293003
