diff options
| author | Michael Stahl <mstahl@redhat.com> | 2012-01-23 13:42:37 +0100 |
|---|---|---|
| committer | Michael Stahl <mstahl@redhat.com> | 2012-01-23 13:44:52 +0100 |
| commit | 7177eebc6942cc66a20514a409788ef87b1383b5 (patch) | |
| tree | f01e4c52098579db651e7b591f626b94e46baed6 | |
| parent | 8f23e9813daae808aa2a81250b5e414dc57a3394 (diff) | |
connectivity: fdo#43479: sort order doesn't matter
| -rw-r--r-- | connectivity/source/drivers/file/FResultSet.cxx | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/connectivity/source/drivers/file/FResultSet.cxx b/connectivity/source/drivers/file/FResultSet.cxx index c47e92fa59a3..3316e3712ec7 100644 --- a/connectivity/source/drivers/file/FResultSet.cxx +++ b/connectivity/source/drivers/file/FResultSet.cxx @@ -1449,7 +1449,7 @@ sal_Bool OResultSet::OpenImpl() // order of columns that are not specified in ORDER BY // clause is undefined, so it doesn't hurt to sort on // these; pad the vectors to include them. - for (sal_Int32 i = 1; + for (sal_Int32 i = 1; // 0: bookmark (see setBoundedColumns) static_cast<size_t>(i) < m_aColMapping.size(); ++i) { if (::std::find(m_aOrderbyColumnNumber.begin(), @@ -1457,10 +1457,8 @@ sal_Bool OResultSet::OpenImpl() == m_aOrderbyColumnNumber.end()) { m_aOrderbyColumnNumber.push_back(i); - m_aOrderbyAscending.push_back( - (m_aOrderbyAscending.empty()) - ? SQL_ASC // default for no ORDER BY - : m_aOrderbyAscending.back()); + // ASC or DESC doesn't matter + m_aOrderbyAscending.push_back(SQL_ASC); } } bDistinct = sal_True; |