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
QmlStreamer: Use asynchronous rendering and image streaming; reuse QuickRenderer from Tide #133
Conversation
@@ -9,11 +9,13 @@ set(DEFLECTQT_HEADERS | |||
QmlStreamerImpl.h | |||
) | |||
set(DEFLECTQT_PUBLIC_HEADERS | |||
QuickRenderer.h |
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.
alphabetical order: u after m (also below)
* After construction, move the object to a dedicated render thread before | ||
* calling any other function. | ||
* | ||
* @param quickWindow the window to render into if not @sa offscreen |
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.
could be clarified - how is it used when offscreen
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.
Humm forget this remark, I'm reading it again and the class description + offscreen parameter description contain enough information.
|
||
signals: | ||
/** Emitted at the end of render(). Originates from render thread. */ | ||
void afterRender(); |
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.
From the changes in Tide, I see that it is necessary to connect this signal to do the swapBuffer when not in offscreen mode, right? Maybe add that to the doc?
|
||
_context->makeCurrent( _getSurface( )); | ||
|
||
_ensureFBO(); |
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.
I would find it clearer to do:
if( offscreen )
_ensureFBO();
instead of:
void QuickRenderer::_ensureFBO()
if( !_offscreen )
return;
…ickRenderer from Tide
No description provided.