Helium: Initial proto and Syncable interface definition

blimp/net/helium/syncable.cc

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "blimp/net/helium/syncable.h"
+
+namespace blimp {
+
+SyncableObject::SyncableObject(const VectorClock& clock) : clock_(clock) {}
+
+template <class ChangesetType>
+SyncableMember<ChangesetType>::SyncableMember(SyncableObject* parent)
+    : parent_(parent) {}
+
+template <class ChangesetType>
+VectorClock SyncableMember<ChangesetType>::IncrementParentClock() {
+  parent_->get_clock().IncrementLocal();
+  return parent_->get_clock();
+}
+
+// Instantiate the template for int (Unit test uses it)
+template class Syncable<int>;
+template class Syncable<helium::Changeset>;
+template class SyncableMember<helium::Changeset>;
+template class SyncableMember<int>;

[scf, 2016/09/29 23:20:08]

is there a way to write this template instantiation in a cleaner way?

[steimel, 2016/09/30 16:43:58]

AFAIK, you either have to do this, define all the methods in your .h file, or
include this file at the end of your .h file

[scf, 2016/09/30 17:26:23]

The problem is that chromium style complained about having this constructor in
the .h file as it became "Complex"

+
+}  // namespace blimp