sdedic commented on pull request #2306: URL: https://github.com/apache/netbeans/pull/2306#issuecomment-672878728
Very nice catch! Indeed, the Compat class was (conceptually) broken from the initial refactor. The issue was hidden until 9f47905f7 which removed the (now obsolete) `getActions()` implementation from the `BinaryFS` ... and even after until some old-API client was REALLY loaded at runtime. So it seems that as a rule of thumb, the compat class should have ***no*** abstract methods in it. The `@PatchFor` processor could enforce that. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
