diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-08-07 12:35:50 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-08-07 18:00:45 +0100 |
| commit | c3e8faa10e90b99601a19b2d46b83deac51d39d5 (patch) | |
| tree | eeb2be66c8d833d9d9ea2e4f8af066cab932cff3 /cppcanvas | |
| parent | 93a37a649755034d306767880572a5d81a43eceb (diff) | |
coverity#1228877 Structurally dead code
Change-Id: If3a3de8bae4c2d1b8e39d3255cd8ee7e985b4404
Diffstat (limited to 'cppcanvas')
| -rw-r--r-- | cppcanvas/source/mtfrenderer/textaction.cxx | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/cppcanvas/source/mtfrenderer/textaction.cxx b/cppcanvas/source/mtfrenderer/textaction.cxx index 717335790dad..ca2ff167c1a3 100644 --- a/cppcanvas/source/mtfrenderer/textaction.cxx +++ b/cppcanvas/source/mtfrenderer/textaction.cxx @@ -2144,6 +2144,8 @@ namespace cppcanvas const ::Color aEmptyColor( COL_AUTO ); + ActionSharedPtr ret; + // no DX array, and no need to subset - no need to store // DX array, then. if( !pDXArray && !bSubsettable ) @@ -2158,7 +2160,7 @@ namespace cppcanvas // nope if( rParms.maTextTransformation.is_initialized() ) { - return ActionSharedPtr( new TextAction( + ret = ActionSharedPtr( new TextAction( aStartPoint, rText, nStartPos, @@ -2169,7 +2171,7 @@ namespace cppcanvas } else { - return ActionSharedPtr( new TextAction( + ret = ActionSharedPtr( new TextAction( aStartPoint, rText, nStartPos, @@ -2182,7 +2184,7 @@ namespace cppcanvas { // at least one of the effects requested if( rParms.maTextTransformation.is_initialized() ) - return ActionSharedPtr( new EffectTextAction( + ret = ActionSharedPtr( new EffectTextAction( aStartPoint, aReliefOffset, rReliefColor, @@ -2196,7 +2198,7 @@ namespace cppcanvas rState, *rParms.maTextTransformation ) ); else - return ActionSharedPtr( new EffectTextAction( + ret = ActionSharedPtr( new EffectTextAction( aStartPoint, aReliefOffset, rReliefColor, @@ -2221,7 +2223,7 @@ namespace cppcanvas { // nope if( rParms.maTextTransformation.is_initialized() ) - return ActionSharedPtr( new TextArrayAction( + ret = ActionSharedPtr( new TextArrayAction( aStartPoint, rText, nStartPos, @@ -2231,7 +2233,7 @@ namespace cppcanvas rState, *rParms.maTextTransformation ) ); else - return ActionSharedPtr( new TextArrayAction( + ret = ActionSharedPtr( new TextArrayAction( aStartPoint, rText, nStartPos, @@ -2244,7 +2246,7 @@ namespace cppcanvas { // at least one of the effects requested if( rParms.maTextTransformation.is_initialized() ) - return ActionSharedPtr( new EffectTextArrayAction( + ret = ActionSharedPtr( new EffectTextArrayAction( aStartPoint, aReliefOffset, rReliefColor, @@ -2259,7 +2261,7 @@ namespace cppcanvas rState, *rParms.maTextTransformation ) ); else - return ActionSharedPtr( new EffectTextArrayAction( + ret = ActionSharedPtr( new EffectTextArrayAction( aStartPoint, aReliefOffset, rReliefColor, @@ -2274,9 +2276,7 @@ namespace cppcanvas rState ) ); } } -#if defined(__GNUC__) - return ActionSharedPtr(); -#endif + return ret; } } } |