diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2019-06-07 22:06:46 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2019-06-11 08:34:34 +0200 |
commit | b70ab9e9e5630a965c5e6cc40c2480dddf1e2384 (patch) | |
tree | d804d6dc19912aef68a9fb858ec2d36eee104d2a /jvmfwk/plugins | |
parent | Veto process exit while an OLE client is connected (diff) | |
download | core-b70ab9e9e5630a965c5e6cc40c2480dddf1e2384.tar.gz core-b70ab9e9e5630a965c5e6cc40c2480dddf1e2384.zip |
Fix swapping of vector elements
...which had been broken with edc85fb55f8adc30a1416c6c854c89097060fb21
"clang-tidy performance-unnecessary-copy-init in idl..reportdesign" (and could
cause "Tools - Options... - LibreOffice - Advanced - Java Options" to
erroneously list a single JRE multiple times, instead of listing different
JREs).
Change-Id: I6799903be39a5c6aa601131fad6eab3edea8d9e4
Reviewed-on: https://gerrit.libreoffice.org/73679
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'jvmfwk/plugins')
-rw-r--r-- | jvmfwk/plugins/sunmajor/pluginlib/util.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/jvmfwk/plugins/sunmajor/pluginlib/util.cxx b/jvmfwk/plugins/sunmajor/pluginlib/util.cxx index 5824448b51d7..f45a796dcad7 100644 --- a/jvmfwk/plugins/sunmajor/pluginlib/util.cxx +++ b/jvmfwk/plugins/sunmajor/pluginlib/util.cxx @@ -731,8 +731,7 @@ void bubbleSortVersion(vector<rtl::Reference<VendorBase> >& vec) } if(nCmp == 1) // cur > next { - vec.at(j-1)= cur; - vec.at(j)= next; + std::swap(vec.at(j-1), vec.at(j)); } } ++cIter; |