[Libreoffice-commits] core.git: sc/source

Kohei Yoshida kohei.yoshida at gmail.com
Sat Feb 3 04:37:31 UTC 2018


 sc/source/filter/orcus/interface.cxx |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit a1c36eff089c3cd695bd78090575ca1c7677121e
Author: Kohei Yoshida <kohei.yoshida at gmail.com>
Date:   Fri Feb 2 22:07:28 2018 -0500

    Ignore the alpha value in font colors.
    
    Using it would remove the color altogether.
    
    Change-Id: I2c14316c9c9c885f4e7615e4057e752a58223227
    Reviewed-on: https://gerrit.libreoffice.org/49170
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Kohei Yoshida <libreoffice at kohei.us>

diff --git a/sc/source/filter/orcus/interface.cxx b/sc/source/filter/orcus/interface.cxx
index 4cfd8447e2d5..40140ae93b5e 100644
--- a/sc/source/filter/orcus/interface.cxx
+++ b/sc/source/filter/orcus/interface.cxx
@@ -1485,12 +1485,13 @@ void ScOrcusStyles::set_font_underline_color(orcus::spreadsheet::color_elem_t al
     maCurrentFont.maUnderlineColor = Color(alpha, red, green, blue);
 }
 
-void ScOrcusStyles::set_font_color(orcus::spreadsheet::color_elem_t alpha,
+void ScOrcusStyles::set_font_color(orcus::spreadsheet::color_elem_t /*alpha*/,
             orcus::spreadsheet::color_elem_t red,
             orcus::spreadsheet::color_elem_t green,
             orcus::spreadsheet::color_elem_t blue)
 {
-    maCurrentFont.maColor = Color(alpha, red, green, blue);
+    // Ignore the alpha value for now.
+    maCurrentFont.maColor = Color(red, green, blue);
     maCurrentFont.mbHasFontAttr = true;
 }
 


More information about the Libreoffice-commits mailing list