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

made logger class available to be downcasted into std::ostream #162

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

tomoyanonymous
Copy link

This is a kind of feature request though there's no API breaking, that logger class can be downcasted into std::ostream& so that user can post messages to Max window from classes in their own library.
Diff looks dirty but I used .clang-format on this repository.

@tomoyanonymous
Copy link
Author

sorry this class did not work correctly, so I will reopen when successfully implemented...

@tomoyanonymous
Copy link
Author

Fixed and confirmed posting to Max console properly. If I should squash commit, I will.

@tomoyanonymous tomoyanonymous force-pushed the feature_logger_ostream branch from 461e6cb to cf05754 Compare August 28, 2020 16:27
@tap
Copy link
Contributor

tap commented Sep 1, 2020

Hi @tomoyanonymous !

Before I can review your PRs I need you to sign our CLA at https://cycling74.com/cla

You only need to do this one time.

Thanks!
./Tim

@tomoyanonymous
Copy link
Author

Sent! Best regards.

@tomoyanonymous tomoyanonymous force-pushed the feature_logger_ostream branch from b21c932 to 614d72b Compare January 5, 2023 13:02
@tomoyanonymous tomoyanonymous force-pushed the feature_logger_ostream branch from 614d72b to b81d174 Compare January 5, 2023 13:03
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

Successfully merging this pull request may close these issues.

2 participants