-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix unhandled matrix enums #2972
Fix unhandled matrix enums #2972
Conversation
This reverts commit e8da01f.
* extend api, add change log entry
@@ -141,3 +141,9 @@ void ofBaseVideoPlayer::previousFrame(){ | |||
// ofLogWarning("ofBaseVideoPlayer") << "getPixelFormat() not implemented"; | |||
// return OF_PIXELS_RGB; | |||
//} | |||
|
|||
//--------------------------------------------------------------------------- | |||
ofMatrix4x4 ofBaseRenderer::getCurrentOrientationMatrix() const { |
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.
any special reason for not making this pure virtual, instead of a warning-only skeleton implementation?
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.
you mean pure virtual as in
ofMatrix4x4 ofBaseRenderer::getCurrentOrientationMatrix() const = 0;
?
Then we'd have to provide implementations with all renderers, which includes Cairo etc. The way i suggest it here this would function as a reminder for renderers that might want to implement this in the future.
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.
Yes.
I see, makes sense, thank you.
thanks! |
fix for #2969.