On Wed, 11 Oct 2023 11:32:37 GMT, Alexey Ushakov <[email protected]> wrote:
>> Hello! Actually, I don't like the fix: >> 1. It was the first time I touched Objective-C code, so the code itself is >> very poor. >> 2. I didn't add any tests because I am not familiar with the test framework. >> 3. At some point we received a number of exception reports related to my >> changes (most-likely), and I have no clue what was the reason. We did some >> investigation with @avu but unsuccessfully. >> >> So, my hope was that someone more experienced will handle the problem >> properly, but passed three years... >> >> @avu Can you help to handle submitting the PR from our side? > > @anstarovoyt thanks for the update. Yes, we'll handle it. > 3. At some point we received a number of exception reports related to my > changes (most-likely), and I have no clue what was the reason. We did some > investigation with @avu but unsuccessfully. We also have an open issue in this state that seems related: https://github.com/mucommander/mucommander/issues/859 > @avu Can you help to handle submitting the PR from our side? This is a long standing issue in OpenJDK and when I reported it, the OpenJDK guys closed the issue without making a real effort to investigate it, so your help would certainly be appreciated! (even a partial fix like this one, currently this functionality doesn't work at all with OpenJDK). Thanks! ------------- PR Comment: https://git.openjdk.org/jdk/pull/15980#issuecomment-1757564813
