From c35d1af6fa616273237354a372c01f5605f10701 Mon Sep 17 00:00:00 2001 From: Emily Janzer Date: Tue, 14 May 2019 12:08:05 -0700 Subject: [PATCH] Add `override` to BridgeJSCallInvoker Summary: In D15252375 I made JSCallInvoker an abstract class and extended it in BridgeJSCallInvoker, but I forgot to add the `override` keyword for overridden methods. Reviewed By: fkgozali Differential Revision: D15328292 fbshipit-source-id: 3e75faf1b4a968b80643b8a97071ab2e122fd643 --- ReactCommon/jscallinvoker/jsireact/BridgeJSCallInvoker.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ReactCommon/jscallinvoker/jsireact/BridgeJSCallInvoker.h b/ReactCommon/jscallinvoker/jsireact/BridgeJSCallInvoker.h index bb0712747e..fa21c2a74d 100644 --- a/ReactCommon/jscallinvoker/jsireact/BridgeJSCallInvoker.h +++ b/ReactCommon/jscallinvoker/jsireact/BridgeJSCallInvoker.h @@ -31,7 +31,7 @@ class BridgeJSCallInvoker : public JSCallInvoker { public: BridgeJSCallInvoker(std::weak_ptr reactInstance); - void invokeAsync(std::function &&func); + void invokeAsync(std::function &&func) override; // TODO: add sync support private: