commit 7b743be32dbf576f43931be3ac968332dd0f070c
author Kevin DuBois <kevindubois@google.com> Wed Feb 27 10:05:48 2019 -0800
committer Kevin DuBois <kevindubois@google.com> Tue Mar 05 09:36:04 2019 -0800
tree 59309707c5db80791220f3629b211ff646c75546
parent 70435e9b3a53fee8610e6ba2d5c99c05faac3743

SF: fix up DispSync interface

Delete copy-assign/copy-constructor for pure virtual i/f DispSync
and its related Callback in the DispSync scope.

Test: compile and boot
Fixes: 127471260
Change-Id: I8df533a4f28f80e986c6285d9ae41f35622290a8

services/surfaceflinger/Scheduler/DispSync.h

diff --git a/services/surfaceflinger/Scheduler/DispSync.h b/services/surfaceflinger/Scheduler/DispSync.h
index 4a90f10..f629697 100644
--- a/services/surfaceflinger/Scheduler/DispSync.h
+++ b/services/surfaceflinger/Scheduler/DispSync.h
@@ -14,8 +14,7 @@
  * limitations under the License.
  */
 
-#ifndef ANDROID_DISPSYNC_H
-#define ANDROID_DISPSYNC_H
+#pragma once
 
 #include <stddef.h>
 
@@ -35,10 +34,16 @@
 public:
     class Callback {
     public:
-        virtual ~Callback() = default;
+        Callback() = default;
+        virtual ~Callback();
         virtual void onDispSyncEvent(nsecs_t when) = 0;
+
+    protected:
+        Callback(Callback const&) = delete;
+        Callback& operator=(Callback const&) = delete;
     };
 
+    DispSync() = default;
     virtual ~DispSync();
 
     virtual void reset() = 0;
@@ -57,6 +62,10 @@
     virtual nsecs_t expectedPresentTime() = 0;
 
     virtual void dump(std::string& result) const = 0;
+
+protected:
+    DispSync(DispSync const&) = delete;
+    DispSync& operator=(DispSync const&) = delete;
 };
 
 namespace impl {
@@ -239,5 +248,3 @@
 } // namespace impl
 
 } // namespace android
-
-#endif // ANDROID_DISPSYNC_H