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

[Tor Lillqvist]

 vcl/source/gdi/pdfwriter_impl.cxx |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit ee2a8ecd577e5fa31d73b92252a7fb05449b6dd9
Author: Tor Lillqvist <tml at collabora.com>
Date:   Mon Mar 23 09:28:19 2015 +0200

    Add comment that a "missing" 'break' is intentional fall-through
    
    Also, add one missing 'break', which however did not have any effect on code
    semantics, as the next case label only contained a 'break' anyway. (So I guess
    it could be claimed that it is also a case of intentional fall-through...)
    
    Thanks to fedor_qd for pointing these out.
    
    Change-Id: I430a71b59c78391e92da7ca14290a4e3cdade496

diff --git a/vcl/source/gdi/pdfwriter_impl.cxx b/vcl/source/gdi/pdfwriter_impl.cxx
index fc44073..b1fd8f6 100644
--- a/vcl/source/gdi/pdfwriter_impl.cxx
+++ b/vcl/source/gdi/pdfwriter_impl.cxx
@@ -9674,6 +9674,7 @@ void PDFWriterImpl::drawStraightTextLine( OStringBuffer& aLine, long nWidth, Fon
                 nLinePos    = HCONV( pFontEntry->maMetric.mnDUnderlineOffset1 );
                 nLinePos2   = HCONV( pFontEntry->maMetric.mnDUnderlineOffset2 );
             }
+            break;
         default:
             break;
     }
@@ -11010,6 +11011,7 @@ bool PDFWriterImpl::writeGradientFunction( GradientEmit& rObject )
             aCol[1] = rObject.m_aGradient.GetEndColor().GetGreen();
             aCol[2] = rObject.m_aGradient.GetEndColor().GetBlue();
             CHECK_RETURN( writeBuffer( aCol, 3 ) );
+            // fall-through
         case GradientStyle_LINEAR:
         {
             aCol[0] = rObject.m_aGradient.GetStartColor().GetRed();