diff options
| author | Tor Lillqvist <tml@collabora.com> | 2015-03-23 09:28:19 +0200 |
|---|---|---|
| committer | Tor Lillqvist <tml@collabora.com> | 2015-03-23 09:44:40 +0200 |
| commit | ee2a8ecd577e5fa31d73b92252a7fb05449b6dd9 (patch) | |
| tree | c9931117b329a93413ad8a35094bf3557a19d034 | |
| parent | 5cbb51c009fa266e8418ef93799ac64e431c22f5 (diff) | |
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
| -rw-r--r-- | vcl/source/gdi/pdfwriter_impl.cxx | 2 |
1 files changed, 2 insertions, 0 deletions
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(); |