[GitHub] [royale-asjs] greg-dove merged pull request #1126: Add support for function instances to getQualifiedClassName().

2021-10-24 Thread GitBox
greg-dove merged pull request #1126: URL: https://github.com/apache/royale-asjs/pull/1126 -- 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:

[GitHub] [royale-asjs] estanglerbm commented on pull request #1126: Add support for function instances to getQualifiedClassName().

2021-10-24 Thread GitBox
estanglerbm commented on pull request #1126: URL: https://github.com/apache/royale-asjs/pull/1126#issuecomment-950276758 Either way is fine with me; I just wanted to make sure the decision ("." or "::") was made deliberately. It might be good to document the difference ("." in Royale

[GitHub] [royale-asjs] estanglerbm edited a comment on pull request #1126: Add support for function instances to getQualifiedClassName().

2021-10-24 Thread GitBox
estanglerbm edited a comment on pull request #1126: URL: https://github.com/apache/royale-asjs/pull/1126#issuecomment-946343078 No rush. By @greg-dove: Thanks. In terms of Function 'class' vs. function instance (or other Class), I had wondered if maybe Function needs to be