Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7b743be3 authored by Kevin DuBois's avatar Kevin DuBois
Browse files

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
parent 70435e9b

services/surfaceflinger/Scheduler/DispSync.cpp

+1 −0
Original line number Diff line number Diff line
@@ -43,6 +43,7 @@ using std::min; 
namespace android {



DispSync::~DispSync() = default;

DispSync::Callback::~Callback() = default;



namespace impl {

services/surfaceflinger/Scheduler/DispSync.h

+12 −5
Original line number Diff line number Diff line
@@ -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 @@ class DispSync { 
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 @@ public: 
    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 @@ private: 
} // namespace impl



} // namespace android
 


#endif // ANDROID_DISPSYNC_H