Reformat all code parts

src/anbox/application_manager.h

@@ -19,8 +19,8 @@
 #define ANBOX_APPLICATION_MANAGER_H_
 
 #include "anbox/android/intent.h"
-#include "anbox/graphics/rect.h"
 #include "anbox/do_not_copy_or_move.h"
+#include "anbox/graphics/rect.h"
 
 #include <string>
 

src/anbox/cli.cpp

@@ -105,14 +105,16 @@ void cli::CommandWithSubcommands::help(std::ostream& out) {
       << std::endl;
 
   if (flags_.size() > 0) {
-    out << std::endl << pattern::options << std::endl;
+    out << std::endl
+        << pattern::options << std::endl;
     for (const auto& flag : flags_)
       out << boost::format(pattern::option) % flag->name() % flag->description()
           << std::endl;
   }
 
   if (commands_.size() > 0) {
-    out << std::endl << pattern::commands << std::endl;
+    out << std::endl
+        << pattern::commands << std::endl;
     for (const auto& cmd : commands_) {
       if (cmd.second)
         out << boost::format(pattern::command) % cmd.second->name() %
@@ -222,7 +224,8 @@ void cli::CommandWithFlagsAndAction::help(std::ostream& out) {
       << std::endl;
 
   if (flags_.size() > 0) {
-    out << std::endl << boost::format(pattern::options) << std::endl;
+    out << std::endl
+        << boost::format(pattern::options) << std::endl;
     for (const auto& flag : flags_)
       out << boost::format(pattern::option) % flag->name() % flag->description()
           << std::endl;

src/anbox/common/message_channel.cpp

@@ -16,8 +16,7 @@
 
 namespace anbox {
 namespace common {
-MessageChannelBase::MessageChannelBase(size_t capacity) :
+MessageChannelBase::MessageChannelBase(size_t capacity) : pos_(0U),
-        pos_(0U),
                                                           count_(0U),
                                                           capacity_(capacity),
                                                           lock_(),

src/anbox/common/message_channel.h

@@ -17,8 +17,8 @@
 
 #include <stddef.h>
 
-#include <mutex>
 #include <condition_variable>
+#include <mutex>
 
 namespace anbox {
 namespace common {
@@ -26,14 +26,14 @@ namespace common {
 // Base non-templated class used to reduce the amount of template
 // specialization.
 class MessageChannelBase {
-public:
+ public:
   // Constructor. |capacity| is the buffer capacity in messages.
   MessageChannelBase(size_t capacity);
 
   // Destructor.
   ~MessageChannelBase();
 
-protected:
+ protected:
   // Call this method in the sender thread before writing a new message.
   // This returns the position of the available slot in the message array
   // where to copy the new fixed-size message. After the copy, call
@@ -54,7 +54,7 @@ protected:
   // the corresponding message.
   void after_read();
 
-private:
+ private:
   size_t pos_;
   size_t count_;
   size_t capacity_;
@@ -75,7 +75,7 @@ private:
 //
 template <typename T, size_t CAPACITY>
 class MessageChannel : public MessageChannelBase {
-public:
+ public:
   MessageChannel() : MessageChannelBase(CAPACITY) {}
 
   void send(const T& msg) {
@@ -90,7 +90,7 @@ public:
     after_read();
   }
 
-private:
+ private:
   T mItems[CAPACITY];
 };
 }  // namespace common

src/anbox/common/scope_ptr.h

@@ -39,7 +39,8 @@ struct FuncDelete {
   FuncDelete& operator=(FuncDelete&& other) = default;
 
   // To be able to copy/move from all compatible template instantiations.
-    template <class U> friend struct FuncDelete;
+  template <class U>
+  friend struct FuncDelete;
 
   // Template constructors and move assignment from compatible instantiations.
   template <class U>
@@ -63,7 +64,7 @@ struct FuncDelete {
     mF(t);
   }
 
-private:
+ private:
   Func mF;
 };
 

src/anbox/container/client.cpp

@@ -18,11 +18,11 @@
 #include "anbox/container/client.h"
 #include "anbox/config.h"
 #include "anbox/container/management_api_stub.h"
+#include "anbox/logger.h"
 #include "anbox/network/local_socket_messenger.h"
 #include "anbox/rpc/channel.h"
 #include "anbox/rpc/message_processor.h"
 #include "anbox/rpc/pending_call_cache.h"
-#include "anbox/logger.h"
 
 namespace ba = boost::asio;
 namespace bs = boost::system;

src/anbox/dbus/skeleton/application_manager.cpp

@@ -45,7 +45,7 @@ ApplicationManager::ApplicationManager(
         reader >> top;
         reader >> right;
         reader >> bottom;
-        graphics::Rect launch_bounds{left,top,right,bottom};
+        graphics::Rect launch_bounds{left, top, right, bottom};
 
         core::dbus::Message::Ptr reply;
 

src/anbox/graphics/emugl/RenderThread.h

@@ -36,7 +36,7 @@ class RenderThread : public emugl::Thread {
   // decoding operations between all threads.
   // TODO(digit): Why is this needed here? Shouldn't this be handled
   //              by the decoders themselves or at a lower-level?
-  static RenderThread* create(const std::shared_ptr<Renderer> &renderer, IOStream* stream, emugl::Mutex* mutex);
+  static RenderThread* create(const std::shared_ptr<Renderer>& renderer, IOStream* stream, emugl::Mutex* mutex);
 
   // Destructor.
   virtual ~RenderThread();
@@ -51,7 +51,7 @@ class RenderThread : public emugl::Thread {
  private:
   RenderThread();  // No default constructor
 
-  RenderThread(const std::shared_ptr<Renderer> &renderer, IOStream* stream, emugl::Mutex* mutex);
+  RenderThread(const std::shared_ptr<Renderer>& renderer, IOStream* stream, emugl::Mutex* mutex);
 
   virtual intptr_t main();
 

src/anbox/graphics/emugl/WindowSurface.h

@@ -56,7 +56,9 @@ class WindowSurface {
   bool flushColorBuffer();
 
   // Used by bind() below.
-  enum BindType { BIND_READ, BIND_DRAW, BIND_READDRAW };
+  enum BindType { BIND_READ,
+                  BIND_DRAW,
+                  BIND_READDRAW };
 
   // TODO(digit): What is this used for exactly? For example, the
   // mReadContext is never used by this class. The mDrawContext is only

src/anbox/logger.h

@@ -33,7 +33,12 @@ class Logger : public DoNotCopyOrMove {
  public:
   // Severity enumerates all known severity levels
   // applicable to log messages.
-  enum class Severity { kTrace, kDebug, kInfo, kWarning, kError, kFatal };
+  enum class Severity { kTrace,
+                        kDebug,
+                        kInfo,
+                        kWarning,
+                        kError,
+                        kFatal };
 
   // A Location describes the origin of a log message.
   struct Location {