Make mojo_system static and mojo_system_impl a component, never use both

mojo/system/entrypoints.h

Index: mojo/system/entrypoints.h
diff --git a/mojo/system/entrypoints.h b/mojo/system/entrypoints.h
new file mode 100644
index 0000000000000000000000000000000000000000..c8048ea109f613bc98f284045af9fbdfb01423d3
--- /dev/null
+++ b/mojo/system/entrypoints.h
@@ -0,0 +1,19 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.

[viettrungluu, 2014/04/10 19:38:06]

Missing include guards (even if this file could be multiply-included safely
currently).

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+namespace mojo {
+namespace system {
+class Core;

[viettrungluu, 2014/04/10 19:38:06]

nit: add a blank line above?

+
+namespace entrypoints {
+
+// Sets the instance of Core to be used by system functions.
+void SetCore(Core* core);
+// Gets the instance of Core to be used by system functions.
+Core* GetCore();
+
+}  // namespace entrypoints
+}  // namepace system
+}  // namespace mojo
+

[viettrungluu, 2014/04/10 19:38:06]

I have a totally irrational pet peeve against extra end-of-file newlines.