Skip to content
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

Question about qt_sink class constructor #2055

Closed
ashley-b opened this issue Aug 17, 2021 · 1 comment
Closed

Question about qt_sink class constructor #2055

ashley-b opened this issue Aug 17, 2021 · 1 comment

Comments

@ashley-b
Copy link

When I was directly constructing a qt_sink in a multi sink arrangement with a file logger. I found It was displaying nothing till I released I was missing meta_method since it defaults to an empty string. I think the arguments qt_object and meta_method should not have default as it does make sense, since there are no setters to change theses values after construction.

qt_sink(QObject *qt_object = nullptr, const std::string &meta_method = "") {

@gabime
Copy link
Owner

gabime commented Aug 17, 2021

agreed. .pr would be most welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants