Merge lp://staging/~mrqtros/ubuntu-docviewer-app/ubuntu-docviewer-app-render-engine-tuning into lp://staging/ubuntu-docviewer-app

Proposed by Roman Shchekin
Status: Merged
Approved by: Stefano Verzegnassi
Approved revision: 192
Merged at revision: 194
Proposed branch: lp://staging/~mrqtros/ubuntu-docviewer-app/ubuntu-docviewer-app-render-engine-tuning
Merge into: lp://staging/ubuntu-docviewer-app
Diff against target: 193 lines (+61/-28)
4 files modified
po/com.ubuntu.docviewer.pot (+3/-3)
src/plugin/libreofficetoolkit-qml-plugin/renderengine.cpp (+34/-23)
src/plugin/libreofficetoolkit-qml-plugin/renderengine.h (+6/-2)
src/plugin/libreofficetoolkit-qml-plugin/rendertask.h (+18/-0)
To merge this branch: bzr merge lp://staging/~mrqtros/ubuntu-docviewer-app/ubuntu-docviewer-app-render-engine-tuning
Reviewer Review Type Date Requested Status
Stefano Verzegnassi Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+276337@code.staging.launchpad.net

Commit message

Code of RenderEngine become more SOLID (still a lot of work to do).

Description of the change

Code of RenderEngine become more SOLID (still a lot of work to do).

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Stefano Verzegnassi (verzegnassi-stefano) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches