You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@quickstep.apache.org by na...@apache.org on 2016/06/23 22:44:27 UTC

[9/9] incubator-quickstep git commit: Fix bug in initialization order.

Fix bug in initialization order.


Project: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/commit/1852df07
Tree: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/tree/1852df07
Diff: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/diff/1852df07

Branch: refs/heads/memcpy_opt
Commit: 1852df07a5f42e64133bac333c50b05f43d20c68
Parents: 5ff1211
Author: Navneet Potti <na...@apache.org>
Authored: Mon Jun 13 17:22:09 2016 -0500
Committer: Navneet Potti <na...@apache.org>
Committed: Mon Jun 13 18:18:11 2016 -0500

----------------------------------------------------------------------
 storage/ValueAccessor.hpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-quickstep/blob/1852df07/storage/ValueAccessor.hpp
----------------------------------------------------------------------
diff --git a/storage/ValueAccessor.hpp b/storage/ValueAccessor.hpp
index b2eecd8..dc999b1 100644
--- a/storage/ValueAccessor.hpp
+++ b/storage/ValueAccessor.hpp
@@ -320,9 +320,9 @@ class TupleIdSequenceAdapterValueAccessor : public ValueAccessor {
       : accessor_(accessor),
         owned_accessor_(take_ownership_of_accessor ? accessor : nullptr),
         id_sequence_(id_sequence),
+        current_position_(id_sequence.before_begin()),
         num_tuples_(id_sequence.numTuples()),
-        end_(id_sequence.end()),
-        current_position_(id_sequence.before_begin()) {
+        end_(id_sequence.end()) {
   }
 
   ~TupleIdSequenceAdapterValueAccessor() override {