distro-configs/LibreOfficeOssFuzz.conf |    5 +++--
 vcl/opengl/win/gdiimpl.cxx             |    5 ++---
 2 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 2109e7d9a0d3dab172961ec19e75da1fa8a6cc61
Author: Caolán McNamara <caol...@redhat.com>
Date:   Thu Dec 8 13:40:34 2016 +0000

    the return from ImplInit *is* checked
    
    bool WinOpenGLContext::init(HDC hDC, HWND hWnd)
    {
    ... return ImplInit();
    }
    
    rtl::Reference<OpenGLContext> WinOpenGLSalGraphicsImpl::CreateWinContext()
    {
    ... if (!xContext->init(mrWinParent.mhLocalDC, mrWinParent.mhWnd))
    }
    
    etc.
    
    Another problem here is ...
    
    "If we notice that OpenGL is broken the first time being called, it is not 
too
    late to call disableOpenGLAndTerminateForRestart(). The first time this 
will be
    called is from displaying the splash screen"
    
    isn't true when the first start causes opengl to be disabled, then on 
subsequent
    starts you can enter this function from launching an opengl slide 
transition.
    
    (note opengl is disabled if opengl is < 30, so all the versioning checks the
    slide transitions do is probably pointless now. The only actually useful use
    of opengl we have seems to be somewhat unloved)
    
    Change-Id: I1e159f826447520f39ea063fd4c0c83220b1385f

diff --git a/vcl/opengl/win/gdiimpl.cxx b/vcl/opengl/win/gdiimpl.cxx
index 6215c4c..3f5da83 100644
--- a/vcl/opengl/win/gdiimpl.cxx
+++ b/vcl/opengl/win/gdiimpl.cxx
@@ -473,9 +473,8 @@ bool compiledShaderBinariesWork()
 
 bool WinOpenGLContext::ImplInit()
 {
-    // Failures here typically means that OpenGL can't be used. Returning 
false is fairly pointless
-    // as the calling code doesn't even check, but oh well. If we notice that 
OpenGL is broken the
-    // first time being called, it is not too late to call
+    // Failures here typically means that OpenGL can't be used.
+    // If we notice that OpenGL is broken the first time being called, it is 
not too late to call
     // disableOpenGLAndTerminateForRestart(). The first time this will be 
called is from displaying
     // the splash screen, so if OpenGL is broken, it is "early enough" for us 
to be able to disable
     // OpenGL and terminate bluntly with EXITHELPER_NORMAL_RESTART, thus 
causing the wrapper process
commit b42225d5d11a78e186797d9753ff40867f6b678a
Author: Caolán McNamara <caol...@redhat.com>
Date:   Thu Dec 8 11:44:13 2016 +0000

    oss-fuzz: disable-gui now works
    
    Change-Id: I8c9349299e67532c60fcd1e887fc0e33e45a29b8

diff --git a/distro-configs/LibreOfficeOssFuzz.conf 
b/distro-configs/LibreOfficeOssFuzz.conf
index 652eb83..cf911a6 100644
--- a/distro-configs/LibreOfficeOssFuzz.conf
+++ b/distro-configs/LibreOfficeOssFuzz.conf
@@ -1,11 +1,12 @@
 --enable-fuzzers
+--disable-gui
+--disable-gtk
+--disable-gtk3
 --disable-postgresql-sdbc
 --disable-firebird-sdbc
 --disable-coinmp
 --disable-lpsolve
 --disable-gltf
---disable-gtk
---disable-gtk3
 --disable-gio
 --disable-dconf
 --disable-collada
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to