diff options
author | Miklos Vajna <vmiklos@collabora.com> | 2019-03-29 13:40:21 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2019-03-29 15:20:49 +0100 |
commit | 75e152a7e8f384921d4417d0fb5d2db6a6f357cb (patch) | |
tree | 9086cffceca0bd57712e9c3f0c531144af4eb64e /vcl/CppunitTest_vcl_gen.mk | |
parent | 6c29dbb7a1c3a04f3cbaa3dd720db482755538d7 (diff) |
tdf#107966 vcl opengl: fix not drawn 1px wide polypolygons
Visible at e.g. in Calc: Format Cells/Borders/Line Styles.
The problem was that first commit
2e99e4e11d33679aed674eea0d6054d16d39d6df (opengl: use MVP matrix in
vertex shaders, pixel offsets, 2015-07-08) introduced the concept of
pixel offsets, setting the value (implicitly) to 0 in
OpenGLSalGraphicsImpl::DrawTrapezoid(), but using 0.5 in
OpenGLSalGraphicsImpl::FlushLinesOrTriangles().
This is fine, but then later commit
2003076c4318511a3d621558d3b44b4e8e6c6529 (opengl: batch draw
polypolygons, 2016-05-29) changed
OpenGLSalGraphicsImpl::drawPolyPolygon() to use deferred drawing instead
of DrawTrapezoid(), without doing any translation of the input
polypolygon. This resulted in loss of those polygons when used in the
above mentioned dialog, which has a listbox of bitmaps, where each line
style preview is drawn on a virtual device with a height of 1px. So at
the end the 1px offset meant the previews were simply missing.
('make CppunitTest_vcl_gen SAL_USE_VCLPLUGIN=gen SAL_FORCEGL=1' is
needed on Linux to see the test failing without the fix.)
Change-Id: Ia9f3d6e7cb38a43fe2f8a41746b538af68add43c
Reviewed-on: https://gerrit.libreoffice.org/69920
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'vcl/CppunitTest_vcl_gen.mk')
-rw-r--r-- | vcl/CppunitTest_vcl_gen.mk | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/vcl/CppunitTest_vcl_gen.mk b/vcl/CppunitTest_vcl_gen.mk index 84dd0ca25bb0..d4f701782ee8 100644 --- a/vcl/CppunitTest_vcl_gen.mk +++ b/vcl/CppunitTest_vcl_gen.mk @@ -16,6 +16,7 @@ $(eval $(call gb_CppunitTest_add_exception_objects,vcl_gen, \ )) $(eval $(call gb_CppunitTest_use_libraries,vcl_gen, \ + basegfx \ comphelper \ cppu \ cppuhelper \ |