diff options
| author | Michael Weghorn <m.weghorn@posteo.de> | 2015-08-22 16:50:56 +0200 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2015-08-23 06:02:42 +0000 |
| commit | b712b1f63492a311e4a51cffd516b3e202a140e6 (patch) | |
| tree | 6703ded631b477680b87da3f1fe6b398c595e5f7 /shell/source/win32 | |
| parent | 93cd7b78c29c11ccc87c19c845c6617acb834630 (diff) | |
tdf#39440 cppcheck: redundantAssignment
This addresses some of cppcheck's "redundantAssignment" warnings
("Variable '<varName>' is reassigned a value before the old one
has been used").
Change-Id: I8fd2950d5aa1a3f4bd9e1c4c336abe465d1f657e
Reviewed-on: https://gerrit.libreoffice.org/17926
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'shell/source/win32')
| -rw-r--r-- | shell/source/win32/simplemail/smplmailclient.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/shell/source/win32/simplemail/smplmailclient.cxx b/shell/source/win32/simplemail/smplmailclient.cxx index d7504c5b6bee..359cc25ea9a1 100644 --- a/shell/source/win32/simplemail/smplmailclient.cxx +++ b/shell/source/win32/simplemail/smplmailclient.cxx @@ -125,12 +125,11 @@ namespace /* private */ return false; oslProcess proc; - oslProcessError err = osl_Process_E_Unknown; /* for efficiency reasons we are using a 'bad' cast here as a vector or OUStrings is nothing else than an array of pointers to rtl_uString's */ - err = osl_executeProcess( + oslProcessError err = osl_executeProcess( senddocUrl.pData, (rtl_uString**)&rCommandArgs[0], rCommandArgs.size(), |