diff --git a/sc/source/filter/excel/xeescher.cxx b/sc/source/filter/excel/xeescher.cxx
index 0fab6991bc94..b8b81aeb24f4 100644
--- a/sc/source/filter/excel/xeescher.cxx
+++ b/sc/source/filter/excel/xeescher.cxx
@@ -864,8 +864,8 @@ XclExpTbxControlObj::XclExpTbxControlObj( XclExpObjectManager& rRoot, Reference<
                 mnXclFont = GetFontBuffer().Insert(maFontData, EXC_COLOR_CTRLTEXT);
         }
 
-        // Align holds the model default - centered on a button, left elsewhere - where the
-        // control has it at all; VerticalAlign is void unless set, and the member default centers
+        // The default for Align is centered on a button, left elsewhere; the default for
+        // VerticalAlign is center
         if (sal_Int16 nApiHorAlign = 0; aCtrlProp.GetProperty(nApiHorAlign, u"Align"_ustr))
         {
             switch (nApiHorAlign)
diff --git a/sc/source/filter/oox/drawingfragment.cxx b/sc/source/filter/oox/drawingfragment.cxx
index 4492c35b1580..4c0e27ac5ad6 100644
--- a/sc/source/filter/oox/drawingfragment.cxx
+++ b/sc/source/filter/oox/drawingfragment.cxx
@@ -907,7 +907,7 @@ void VmlDrawing::convertControlFontData( AxFontData& rAxFontData, sal_uInt32& rn
     {
         if (OUString aFontId; rFontModel.moUnderlineClass.value().startsWith(u"font", &aFontId))
         {
-            if (FontRef rxFont = getStyles().getFont(aFontId.toInt32()); rxFont)
+            if (FontRef rxFont = getStyles().getFont(aFontId.toInt32()))
                 rAxFontData.mbDblUnderline
                     = (rxFont->getModel().mnUnderline == XML_double)
                       || (rxFont->getModel().mnUnderline == XML_doubleAccounting);