matthiasblaesing commented on PR #3995: URL: https://github.com/apache/netbeans/pull/3995#issuecomment-1118976937
The fact that the change only affects currently unsupported cases is a good argument, that this is a safe change. I'm still uncomfortable with the fact, that the `SingleMethod` class changed, while the PHP module demonstrated a way to reference qualified names without changing the definition. In Javadoc there an established syntax to refer to members - a reference could be coded as `package.class.innerclass#method(ParamType1,ParamType2)`. An open question: is support for overloads required (implied with the parameter based syntax). -- 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. To unsubscribe, e-mail: [email protected] 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
