commit ea47ff8d26c831f1380f5f9f5c7fe958e40d63a5
author Andreas Gampe <agampe@google.com> Thu Nov 03 08:20:17 2016 -0700
committer Andreas Gampe <agampe@google.com> Thu Nov 03 11:16:36 2016 -0700
tree 4a93f9b6181cabbec0f19e222449f3ec3b6faca0
parent 1eeb00c7499c520159a38af4be3835aee7ee7b38

ART: Add "explicit" to constructors

Add explicit to two constructors with single argument.

Bug: 32619234
Test: m
Change-Id: I6ee6c8264d76017247e2236170c776c801719a0a

runtime/handle_scope.h

diff --git a/runtime/handle_scope.h b/runtime/handle_scope.h
index 8a0aba6..adb7d8a 100644
--- a/runtime/handle_scope.h
+++ b/runtime/handle_scope.h
@@ -69,7 +69,7 @@
         number_of_references_(num_references) {}
 
   // Variable sized constructor.
-  BaseHandleScope(BaseHandleScope* link)
+  explicit BaseHandleScope(BaseHandleScope* link)
       : link_(link),
         number_of_references_(kNumReferencesVariableSized) {}
 

runtime/thread.h

diff --git a/runtime/thread.h b/runtime/thread.h
index 75b5b12..4f26803 100644
--- a/runtime/thread.h
+++ b/runtime/thread.h
@@ -1567,7 +1567,8 @@
 
 class SCOPED_CAPABILITY ScopedAssertNoThreadSuspension {
  public:
-  ALWAYS_INLINE ScopedAssertNoThreadSuspension(const char* cause) ACQUIRE(Roles::uninterruptible_) {
+  ALWAYS_INLINE explicit ScopedAssertNoThreadSuspension(const char* cause)
+      ACQUIRE(Roles::uninterruptible_) {
     if (kIsDebugBuild) {
       self_ = Thread::Current();
       old_cause_ = self_->StartAssertNoThreadSuspension(cause);