Make mojo_system static and mojo_system_impl a component, never use both

mojo/system/entrypoints.cc

Index: mojo/system/entrypoints.cc
diff --git a/mojo/public/system/core_private.cc b/mojo/system/entrypoints.cc
similarity index 69%
rename from mojo/public/system/core_private.cc
rename to mojo/system/entrypoints.cc
index bd9d6b85fdf2579037c392f0b4b43428d4da0265..096fd02573fddad00e1498f2c220e9e98ea278ae 100644
--- a/mojo/public/system/core_private.cc
+++ b/mojo/system/entrypoints.cc
@@ -1,30 +1,41 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
+// Copyright 2014 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 "mojo/public/system/core_private.h"
+#include "mojo/system/core.h"
 
-#include <assert.h>
-#include <stddef.h>
+static mojo::system::Core* g_core = NULL;

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

nit: I'd still just move this down a few lines, to be able to omit the
"mojo::system::".

 
-static mojo::Core* g_core = NULL;
+namespace mojo {
+namespace system {
+namespace entrypoints {
+
+void SetCore(Core* core) {
+  g_core = core;
+}
+
+Core* GetCore() {
+  return g_core;
+}
+
+}  // namespace entrypoints
+}  // namepace system
+}  // namespace mojo
 
+// Definitions of the system functions.
 extern "C" {
 
 MojoTimeTicks MojoGetTimeTicksNow() {
-  assert(g_core);
   return g_core->GetTimeTicksNow();
 }
 
 MojoResult MojoClose(MojoHandle handle) {
-  assert(g_core);
   return g_core->Close(handle);
 }
 
 MojoResult MojoWait(MojoHandle handle,
                     MojoWaitFlags flags,
                     MojoDeadline deadline) {
-  assert(g_core);
   return g_core->Wait(handle, flags, deadline);
 }
 
@@ -32,13 +43,11 @@ MojoResult MojoWaitMany(const MojoHandle* handles,
                         const MojoWaitFlags* flags,
                         uint32_t num_handles,
                         MojoDeadline deadline) {
-  assert(g_core);
   return g_core->WaitMany(handles, flags, num_handles, deadline);
 }
 
 MojoResult MojoCreateMessagePipe(MojoHandle* message_pipe_handle0,
                                  MojoHandle* message_pipe_handle1) {
-  assert(g_core);
   return g_core->CreateMessagePipe(message_pipe_handle0, message_pipe_handle1);
 }
 
@@ -48,51 +57,45 @@ MojoResult MojoWriteMessage(MojoHandle message_pipe_handle,
                             const MojoHandle* handles,
                             uint32_t num_handles,
                             MojoWriteMessageFlags flags) {
-  assert(g_core);
-  return g_core->WriteMessage(message_pipe_handle, bytes, num_bytes, handles,
-                              num_handles, flags);
+  return g_core->WriteMessage(
+      message_pipe_handle, bytes, num_bytes, handles, num_handles, flags);
 }
 
 MojoResult MojoReadMessage(MojoHandle message_pipe_handle,
                            void* bytes,
                            uint32_t* num_bytes,
                            MojoHandle* handles,
-                           uint32_t* num_handles,
+                         uint32_t* num_handles,
                            MojoReadMessageFlags flags) {
-  assert(g_core);
-  return g_core->ReadMessage(message_pipe_handle, bytes, num_bytes, handles,
-                             num_handles, flags);
+  return g_core->ReadMessage(
+      message_pipe_handle, bytes, num_bytes, handles, num_handles, flags);
 }
 
 MojoResult MojoCreateDataPipe(const MojoCreateDataPipeOptions* options,
                               MojoHandle* data_pipe_producer_handle,
                               MojoHandle* data_pipe_consumer_handle) {
-  assert(g_core);
-  return g_core->CreateDataPipe(options, data_pipe_producer_handle,
-                                data_pipe_consumer_handle);
+  return g_core->CreateDataPipe(
+      options, data_pipe_producer_handle, data_pipe_consumer_handle);
 }
 
 MojoResult MojoWriteData(MojoHandle data_pipe_producer_handle,
                          const void* elements,
                          uint32_t* num_elements,
                          MojoWriteDataFlags flags) {
-  assert(g_core);
-  return g_core->WriteData(data_pipe_producer_handle, elements, num_elements,
-                           flags);
+  return g_core->WriteData(
+      data_pipe_producer_handle, elements, num_elements, flags);
 }
 
 MojoResult MojoBeginWriteData(MojoHandle data_pipe_producer_handle,
                               void** buffer,
                               uint32_t* buffer_num_elements,
                               MojoWriteDataFlags flags) {
-  assert(g_core);
-  return g_core->BeginWriteData(data_pipe_producer_handle, buffer,
-                                buffer_num_elements, flags);
+  return g_core->BeginWriteData(
+      data_pipe_producer_handle, buffer, buffer_num_elements, flags);
 }
 
 MojoResult MojoEndWriteData(MojoHandle data_pipe_producer_handle,
                             uint32_t num_elements_written) {
-  assert(g_core);
   return g_core->EndWriteData(data_pipe_producer_handle, num_elements_written);
 }
 
@@ -100,23 +103,20 @@ MojoResult MojoReadData(MojoHandle data_pipe_consumer_handle,
                         void* elements,
                         uint32_t* num_elements,
                         MojoReadDataFlags flags) {
-  assert(g_core);
-  return g_core->ReadData(data_pipe_consumer_handle, elements, num_elements,
-                          flags);
+  return g_core->ReadData(
+      data_pipe_consumer_handle, elements, num_elements, flags);
 }
 
 MojoResult MojoBeginReadData(MojoHandle data_pipe_consumer_handle,
                              const void** buffer,
                              uint32_t* buffer_num_elements,
                              MojoReadDataFlags flags) {
-  assert(g_core);
-  return g_core->BeginReadData(data_pipe_consumer_handle, buffer,
-                               buffer_num_elements, flags);
+  return g_core->BeginReadData(
+      data_pipe_consumer_handle, buffer, buffer_num_elements, flags);
 }
 
 MojoResult MojoEndReadData(MojoHandle data_pipe_consumer_handle,
                            uint32_t num_elements_read) {
-  assert(g_core);
   return g_core->EndReadData(data_pipe_consumer_handle, num_elements_read);
 }
 
@@ -124,7 +124,6 @@ MojoResult MojoCreateSharedBuffer(
     const struct MojoCreateSharedBufferOptions* options,
     uint64_t num_bytes,
     MojoHandle* shared_buffer_handle) {
-  assert(g_core);
   return g_core->CreateSharedBuffer(options, num_bytes, shared_buffer_handle);
 }
 
@@ -132,9 +131,8 @@ MojoResult MojoDuplicateBufferHandle(
     MojoHandle buffer_handle,
     const struct MojoDuplicateBufferHandleOptions* options,
     MojoHandle* new_buffer_handle) {
-  assert(g_core);
-  return g_core->DuplicateBufferHandle(buffer_handle, options,
-                                       new_buffer_handle);
+  return g_core->DuplicateBufferHandle(
+      buffer_handle, options, new_buffer_handle);
 }
 
 MojoResult MojoMapBuffer(MojoHandle buffer_handle,
@@ -142,36 +140,11 @@ MojoResult MojoMapBuffer(MojoHandle buffer_handle,
                          uint64_t num_bytes,
                          void** buffer,
                          MojoMapBufferFlags flags) {
-  assert(g_core);
   return g_core->MapBuffer(buffer_handle, offset, num_bytes, buffer, flags);
 }
 
 MojoResult MojoUnmapBuffer(void* buffer) {
-  assert(g_core);
   return g_core->UnmapBuffer(buffer);
 }
 
 }  // extern "C"
-
-namespace mojo {
-
-Core::~Core() {
-}
-
-// static
-void Core::Init(Core* core) {
-  assert(!g_core);
-  g_core = core;
-}
-
-// static
-Core* Core::Get() {
-  return g_core;
-}
-
-// static
-void Core::Reset() {
-  g_core = NULL;
-}
-
-}  // namespace mojo