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 | |
| parent | 89f883bd90a50587868a57397b6350ed9559a20f (diff) | |
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')
| -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; |