-
Notifications
You must be signed in to change notification settings - Fork 311
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Macro-generated connectConnector methods. #1447
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Nice and straightforward. Thanks!
@@ -857,7 +859,14 @@ class Input : public AbstractInput { | |||
"Path to a Component to satisfy the Connector '" \ | |||
#cname "' of type " #T " (description: " comment ").") \ | |||
}; \ | |||
/** @endcond */ | |||
/** @endcond */ \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update auto generated comment for the property to : "Path to a Component that satisfies the Connector ` ... "
@aseth1 I have addressed your comment. |
Ready for review. |
LGTM 👍 @aymanhab? |
LGTM as well. Thanks @chrisdembia |
Thank you. |
This PR is the counterpart to #1441 for connectors. The original plan was to rename connectors to sockets first before introducing the
connectSocket_*()
macro function. But we decided to switch the order of these changes, since having the macro-generated method in place for the alpha release is perhaps more important than having the rename to sockets for the alpha.This is my second attempt at creating this PR, as #1446 shows an outdated diff (from before #1441 was merged).