On Tue, 26 Jul 2022 22:54:56 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> - replaced with exact functional equivalent (in the presence of exceptions, 
>> for example)
>
> Andy Goryachev has updated the pull request incrementally with two additional 
> commits since the last revision:
> 
>  - 8289397: review comments
>  - Revert "8289397: functional equivalent"
>    
>    This reverts commit 6ae0ef60b186c75993fe7faa099140b26593d308.

It looks like people split between two camps:
- those who prefer to leave the code as is and disable the warning (or 
suppressing the warning)
- those who prefer added value of the warning in preventing a possible bug

I just want to reiterate that paying attention to eclipse warnings already 
uncovered some bugs that went undetected for many years, so personally I do see 
value in addressing these.

However, at this point it is unclear to me how to proceed.  I can either ask 
for a consensus or vote, or withdraw this PR.

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

PR: https://git.openjdk.org/jfx/pull/851

Reply via email to