diff options
| author | Noel Grandin <noel@peralex.com> | 2016-05-19 12:47:50 +0200 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2016-05-20 07:55:30 +0000 |
| commit | 8a2149c1059c70d67b4897e8a7751cf028a5136b (patch) | |
| tree | 10d01599b5b5b2ea1d361c32d438cf47371f21ae /framework/source/jobs | |
| parent | 1b89a57aa5ac4faf59bc1e13eb1e0cad9933bacc (diff) | |
clang-tidy misc-assign-operator-signature
Change-Id: I2953a88d9e2f5923732865ef17615d5928ac5f5f
Reviewed-on: https://gerrit.libreoffice.org/25154
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'framework/source/jobs')
| -rw-r--r-- | framework/source/jobs/jobdata.cxx | 3 | ||||
| -rw-r--r-- | framework/source/jobs/jobresult.cxx | 3 |
2 files changed, 4 insertions, 2 deletions
diff --git a/framework/source/jobs/jobdata.cxx b/framework/source/jobs/jobdata.cxx index c3b453d83691..c1807a4ea47c 100644 --- a/framework/source/jobs/jobdata.cxx +++ b/framework/source/jobs/jobdata.cxx @@ -71,7 +71,7 @@ JobData::JobData( const JobData& rCopy ) @param rCopy the original instance, from which we must copy all data */ -void JobData::operator=( const JobData& rCopy ) +JobData& JobData::operator=( const JobData& rCopy ) { SolarMutexGuard g; // Please don't copy the uno service manager reference. @@ -84,6 +84,7 @@ void JobData::operator=( const JobData& rCopy ) m_sEvent = rCopy.m_sEvent; m_lArguments = rCopy.m_lArguments; m_aLastExecutionResult = rCopy.m_aLastExecutionResult; + return *this; } /** diff --git a/framework/source/jobs/jobresult.cxx b/framework/source/jobs/jobresult.cxx index 01adf16c9981..c9f5a0218af2 100644 --- a/framework/source/jobs/jobresult.cxx +++ b/framework/source/jobs/jobresult.cxx @@ -157,7 +157,7 @@ JobResult::~JobResult() @param rCopy reference to the other instance, which should be used for copying. */ -void JobResult::operator=( const JobResult& rCopy ) +JobResult& JobResult::operator=( const JobResult& rCopy ) { SolarMutexGuard g; m_aPureResult = rCopy.m_aPureResult; @@ -165,6 +165,7 @@ void JobResult::operator=( const JobResult& rCopy ) m_lArguments = rCopy.m_lArguments; m_bDeactivate = rCopy.m_bDeactivate; m_aDispatchResult = rCopy.m_aDispatchResult; + return *this; } /** |