[Groonga-commit] groonga/grnxx at 15b88d8 [master] Add enum SortOrder that is used to specify the sort order.

Back to archive index

susumu.yata null+****@clear*****
Fri May 30 16:39:36 JST 2014


susumu.yata	2014-05-30 16:39:36 +0900 (Fri, 30 May 2014)

  New Revision: 15b88d881e77da498c02472b1e5135d0e5df7af1
  https://github.com/groonga/grnxx/commit/15b88d881e77da498c02472b1e5135d0e5df7af1

  Message:
    Add enum SortOrder that is used to specify the sort order.
    
    ASCENDING_ORDER and DESCENDING_ORDER is available.

  Modified files:
    new-interface/sorter.hpp

  Modified: new-interface/sorter.hpp (+7 -5)
===================================================================
--- new-interface/sorter.hpp    2014-05-30 16:34:10 +0900 (28329e9)
+++ new-interface/sorter.hpp    2014-05-30 16:39:36 +0900 (03f60bc)
@@ -5,6 +5,11 @@
 
 namespace grnxx {
 
+enum SortOrder {
+  ASCENDING_ORDER,
+  DESCENDING_ORDER
+};
+
 class Sorter {
  public:
   Sorter();
@@ -16,9 +21,6 @@ class Sorter {
   // TODO: 条件を一気に設定するようにした方が使いやすい可能性がある.
   //       その場合, Table::create_sorter() で指定することも考慮すべきである.
 
-  // TODO: reverse_order を bool で受け取るよりは,
-  //       enum で受け取る方がわかりやすい.
-
   // 前提条件を追加する.
   // 成功すれば true を返す.
   // 失敗したときは *error にその内容を格納し, false を返す.
@@ -30,7 +32,7 @@ class Sorter {
   // - 式の評価結果が大小関係を持たない型になる.
   // - リソースを確保できない.
   virtual bool add_precondition(const Expression *expression,
-                                bool reverse_order,
+                                SortOrder order,
                                 Error *error) const = 0;
 
   // 整列条件を追加する.
@@ -44,7 +46,7 @@ class Sorter {
   // - 式の評価結果が大小関係を持たない型になる.
   // - リソースを確保できない.
   virtual bool add_condition(const Expression *expression,
-                             bool reverse_order,
+                             SortOrder order,
                              Error *error) const = 0;
 
   // 行の一覧を整列する.
-------------- next part --------------
HTML����������������������������...
Download 



More information about the Groonga-commit mailing list
Back to archive index