Add TODO comments.

groonga · Nov 17, 2014 · 453b22e · 453b22e
1 parent 9f62e01
commit 453b22e
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 0 deletions.
diff --git a/include/grnxx/data_types/vector/bool.hpp b/include/grnxx/data_types/vector/bool.hpp
@@ -21,6 +21,8 @@ class Vector<Bool> {
   constexpr Vector(const Bool *data, size_t size) : data_(data), size_(size) {}
   explicit constexpr Vector(NA) : data_(nullptr), size_(NA()) {}
 
+  // TODO: The argument should be Int.
+  //       Also, N/A should be returned for an invalid "i".
   const Bool &operator[](size_t i) const {
     return data_[i];
   }

diff --git a/include/grnxx/data_types/vector/float.hpp b/include/grnxx/data_types/vector/float.hpp
@@ -23,6 +23,8 @@ class Vector<Float> {
         size_(size) {}
   explicit constexpr Vector(NA) : data_(nullptr), size_(NA()) {}
 
+  // TODO: The argument should be Int.
+  //       Also, N/A should be returned for an invalid "i".
   const Float &operator[](size_t i) const {
     return data_[i];
   }

diff --git a/include/grnxx/data_types/vector/geo_point.hpp b/include/grnxx/data_types/vector/geo_point.hpp
@@ -23,6 +23,8 @@ class Vector<GeoPoint> {
         size_(size) {}
   explicit constexpr Vector(NA) : data_(nullptr), size_(NA()) {}
 
+  // TODO: The argument should be Int.
+  //       Also, N/A should be returned for an invalid "i".
   const GeoPoint &operator[](size_t i) const {
     return data_[i];
   }

diff --git a/include/grnxx/data_types/vector/int.hpp b/include/grnxx/data_types/vector/int.hpp
@@ -21,6 +21,8 @@ class Vector<Int> {
   constexpr Vector(const Int *data, size_t size) : data_(data), size_(size) {}
   explicit constexpr Vector(NA) : data_(nullptr), size_(NA()) {}
 
+  // TODO: The argument should be Int.
+  //       Also, N/A should be returned for an invalid "i".
   const Int &operator[](size_t i) const {
     return data_[i];
   }