On Wed, 23 Mar 2022 18:15:54 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Tejesh R has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Updated based on review comments
>
> src/java.desktop/share/classes/javax/swing/ImageIcon.java line 338:
> 
>> 336:             mTracker.removeImage(image, id);
>> 337: 
>> 338:             if (interrupted && ((loadStatus & MediaTracker.LOADING )!= 
>> 0)) {
> 
> Suggestion:
> 
>             if (interrupted && ((loadStatus & MediaTracker.LOADING ) != 0)) {
> 
> There should be one space on either side of a binary operator.

It seems to me that as I mentioned there's at least a possibility that the 
image actually is loaded. This thread being interrupted doesn't stop the Media 
loading threads.

Since there's only one image for this id, LOADING and COMPLETE ought to be
mutually exclusive. In fact I'd expect only one of the flags to be set.

So I can I suppose see the logic that this API returning ABORTED instead of 
LOADING
makes sense, but that is because this method removes it from the tracker ..

So I guess I can live with it .. especially since I think this an unlikely 
scenario.

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

PR: https://git.openjdk.java.net/jdk/pull/7754

Reply via email to