Re: shell32: Remove dead assignments (Clang)

[Nikolay Sivov]

2011/7/6 Frédéric Delanoy :
> dlls/shell32/shelllink.c |   16 ++++++++--------
>  1 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/dlls/shell32/shelllink.c b/dlls/shell32/shelllink.c
> index 07fe7aa..fff0e79 100644
> --- a/dlls/shell32/shelllink.c
> +++ b/dlls/shell32/shelllink.c
> @@ -1189,29 +1189,29 @@ static HRESULT WINAPI IPersistStream_fnSave(
>         Stream_WriteLocationInfo( stm, This->sPath, &This->volume );
>
> if( This->sDescription )
> - r = Stream_WriteString( stm, This->sDescription );
> - Stream_WriteString( stm, This->sDescription );

This is questionable. If you write to a provided stream it's ok for it
to fail write operation, this needs a test. Method return value should
probably reflect write failure.