diff options
author | Kohei Yoshida <kohei@libreoffice.org> | 2019-08-27 16:10:08 -0400 |
---|---|---|
committer | Kohei Yoshida <kohei@libreoffice.org> | 2019-08-28 00:04:32 +0200 |
commit | 0fb2927a8fe06e6c3255544b8e4c4c9c0f5a67d3 (patch) | |
tree | 7136bf1d426346b881cea44d12a0b8ac429f7a0c /sc/source | |
parent | bde3c5329f953f2a89e76e5677e8484564f4408b (diff) |
Remove unnecessary braces.
As to the reason why they were there can be explained by the commit
48b35ed43db87d21a943e66cfa4f59213dcfe3a1. They braces did make sense
before that commit.
Change-Id: Ie889d9e2404436c1cc0249dcf71bf18ebfd40636
Reviewed-on: https://gerrit.libreoffice.org/78212
Tested-by: Jenkins
Reviewed-by: Kohei Yoshida <kohei@libreoffice.org>
Diffstat (limited to 'sc/source')
-rw-r--r-- | sc/source/ui/xmlsource/xmlsourcedlg.cxx | 38 |
1 files changed, 17 insertions, 21 deletions
diff --git a/sc/source/ui/xmlsource/xmlsourcedlg.cxx b/sc/source/ui/xmlsource/xmlsourcedlg.cxx index 0cd202c8dec6..148aba1d0443 100644 --- a/sc/source/ui/xmlsource/xmlsourcedlg.cxx +++ b/sc/source/ui/xmlsource/xmlsourcedlg.cxx @@ -496,38 +496,34 @@ void ScXMLSourceDlg::OkPressed() ScOrcusImportXMLParam aParam; // Convert single cell links. + for (const auto& rEntry : maCellLinks) { - for (const auto& rEntry : maCellLinks) - { - OUString aPath = getXPath(*mxLbTree, *rEntry, aParam.maNamespaces); - const ScOrcusXMLTreeParam::EntryData* pUserData = ScOrcusXMLTreeParam::getUserData(*mxLbTree, *rEntry); + OUString aPath = getXPath(*mxLbTree, *rEntry, aParam.maNamespaces); + const ScOrcusXMLTreeParam::EntryData* pUserData = ScOrcusXMLTreeParam::getUserData(*mxLbTree, *rEntry); - aParam.maCellLinks.emplace_back( - pUserData->maLinkedPos, OUStringToOString(aPath, RTL_TEXTENCODING_UTF8)); - } + aParam.maCellLinks.emplace_back( + pUserData->maLinkedPos, OUStringToOString(aPath, RTL_TEXTENCODING_UTF8)); } // Convert range links. For now, an element with range link takes all its // child elements as its fields. + for (const auto& rEntry: maRangeLinks) { - for (const auto& rEntry: maRangeLinks) - { - const ScOrcusXMLTreeParam::EntryData* pUserData = ScOrcusXMLTreeParam::getUserData(*mxLbTree, *rEntry); + const ScOrcusXMLTreeParam::EntryData* pUserData = ScOrcusXMLTreeParam::getUserData(*mxLbTree, *rEntry); - ScOrcusImportXMLParam::RangeLink aRangeLink; - aRangeLink.maPos = pUserData->maLinkedPos; + ScOrcusImportXMLParam::RangeLink aRangeLink; + aRangeLink.maPos = pUserData->maLinkedPos; - // Go through all its child elements. - getFieldLinks(aRangeLink, aParam.maNamespaces, *mxLbTree, *rEntry); + // Go through all its child elements. + getFieldLinks(aRangeLink, aParam.maNamespaces, *mxLbTree, *rEntry); - // Add the reference entry as a row-group node, which will be used - // as a row position increment point. - OUString aThisEntry = getXPath(*mxLbTree, *rEntry, aParam.maNamespaces); - aRangeLink.maRowGroups.push_back( - OUStringToOString(aThisEntry, RTL_TEXTENCODING_UTF8)); + // Add the reference entry as a row-group node, which will be used + // as a row position increment point. + OUString aThisEntry = getXPath(*mxLbTree, *rEntry, aParam.maNamespaces); + aRangeLink.maRowGroups.push_back( + OUStringToOString(aThisEntry, RTL_TEXTENCODING_UTF8)); - aParam.maRangeLinks.push_back(aRangeLink); - } + aParam.maRangeLinks.push_back(aRangeLink); } // Remove duplicate namespace IDs. |