From ee2a8ecd577e5fa31d73b92252a7fb05449b6dd9 Mon Sep 17 00:00:00 2001 From: Tor Lillqvist Date: Mon, 23 Mar 2015 09:28:19 +0200 Subject: 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 --- vcl/source/gdi/pdfwriter_impl.cxx | 2 ++ 1 file changed, 2 insertions(+) diff --git a/vcl/source/gdi/pdfwriter_impl.cxx b/vcl/source/gdi/pdfwriter_impl.cxx index fc44073b4cb1..b1fd8f60364c 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(); -- cgit