[Libreoffice-commits] core.git: include/canvas

Stephan Bergmann sbergman at redhat.com
Tue Dec 1 03:31:08 PST 2015


 include/canvas/rendering/icolorbuffer.hxx |    5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

New commits:
commit 18cb1f747b2ae7573fb80df28755315b43dd8d0a
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Dec 1 12:27:43 2015 +0100

    Apparently no reason to force underlying type here
    
    ...presumably just cargo-cult from how enums get ABI-fixed in the URE interface.
    FMT_UNKNOWN turns out to be unused ever since the code was introduced with
    5c5e584c8f22c09cb999d7543a9415349910fa06 "INTEGRATION: CWS canvas02."
    
    Caused -Werror,-Wmicrosoft-enum-value with clang-cl.
    
    Change-Id: I353e4a2e71ff720a5368548d61ea1bded9c77a6c

diff --git a/include/canvas/rendering/icolorbuffer.hxx b/include/canvas/rendering/icolorbuffer.hxx
index 6b8c8c0..5439f65 100644
--- a/include/canvas/rendering/icolorbuffer.hxx
+++ b/include/canvas/rendering/icolorbuffer.hxx
@@ -42,10 +42,7 @@ namespace canvas
             FMT_A8R8G8B8,
 
             // 32-bit RGB pixel format, 8 bits per channel.
-            FMT_X8R8G8B8,
-
-            // for enum to 32bit
-            FMT_UNKNOWN = static_cast<sal_uInt32>(-1)
+            FMT_X8R8G8B8
         };
 
         virtual ~IColorBuffer() {}


More information about the Libreoffice-commits mailing list