diff options
Diffstat (limited to 'include')
21 files changed, 125 insertions, 218 deletions
diff --git a/include/cru/base/Event.h b/include/cru/base/Event.h index 18d2c570..1f57e100 100644 --- a/include/cru/base/Event.h +++ b/include/cru/base/Event.h @@ -11,58 +11,36 @@ #include <vector> namespace cru { -class EventRevoker; +class EventHandlerRevoker; -namespace details { -template <class> -inline constexpr bool always_false_v = false; - -// Base class of all Event<T...>. -// It erases event args types and provides a -// unified form to create event revoker and -// revoke(remove) handler. -class EventBase : public SelfResolvable<EventBase> { - friend EventRevoker; +class EventBase : public Object, public SelfResolvable<EventBase> { + friend EventHandlerRevoker; protected: - using EventHandlerToken = long; - - EventBase() {} - CRU_DELETE_COPY(EventBase) - CRU_DEFAULT_MOVE(EventBase) - virtual ~EventBase() = default; + using EventHandlerToken = int; - // Remove the handler with the given token. If the token - // corresponds to no handler (which might have be revoked - // before), then nothing will be done. + /** + * Remove the handler with the given token. If the token corresponds to no + * handler (which might have be revoked before), then nothing will be done. + */ virtual void RemoveHandler(EventHandlerToken token) = 0; - // Create a revoker with the given token. - inline EventRevoker CreateRevoker(EventHandlerToken token); + inline EventHandlerRevoker CreateRevoker(EventHandlerToken token); }; -} // namespace details -// A non-copyable and movable event revoker. -// Call function call operator to revoke the handler. -class EventRevoker { - friend details::EventBase; +class EventHandlerRevoker { + friend EventBase; private: - EventRevoker(ObjectResolver<details::EventBase>&& resolver, - details::EventBase::EventHandlerToken token) + EventHandlerRevoker(ObjectResolver<EventBase>&& resolver, + EventBase::EventHandlerToken token) : resolver_(std::move(resolver)), token_(token) {} public: - EventRevoker(const EventRevoker& other) = default; - EventRevoker(EventRevoker&& other) = default; - EventRevoker& operator=(const EventRevoker& other) = default; - EventRevoker& operator=(EventRevoker&& other) = default; - ~EventRevoker() = default; - - // Revoke the registered handler. If the event has already - // been destroyed, then nothing will be done. If one of the - // copies calls this, then other copies's calls will have no - // effect. (They have the same token.) + /** + * Revoke the registered handler. If the event has already been destroyed or + * the handler is already revoked, nothing will be done. + */ void operator()() const { if (const auto event = resolver_.Resolve()) { event->RemoveHandler(token_); @@ -70,34 +48,17 @@ class EventRevoker { } private: - ObjectResolver<details::EventBase> resolver_; - details::EventBase::EventHandlerToken token_; + ObjectResolver<EventBase> resolver_; + EventBase::EventHandlerToken token_; }; -inline EventRevoker details::EventBase::CreateRevoker(EventHandlerToken token) { - return EventRevoker(CreateResolver(), token); +inline EventHandlerRevoker EventBase::CreateRevoker(EventHandlerToken token) { + return EventHandlerRevoker(CreateResolver(), token); } -// int -> int -// Point -> const Point& -// int& -> int& -template <typename TRaw> -using DeducedEventArgs = std::conditional_t< - std::is_lvalue_reference_v<TRaw>, TRaw, - std::conditional_t<std::is_scalar_v<TRaw>, TRaw, const TRaw&>>; - -struct IBaseEvent { - protected: - IBaseEvent() = default; - CRU_DELETE_COPY(IBaseEvent) - CRU_DEFAULT_MOVE(IBaseEvent) - ~IBaseEvent() = default; // Note that user can't destroy a Event via IEvent. - // So destructor should be protected. - +struct IBaseEvent : public virtual Interface { using SpyOnlyHandler = std::function<void()>; - - public: - virtual EventRevoker AddSpyOnlyHandler(SpyOnlyHandler handler) = 0; + virtual EventHandlerRevoker AddSpyOnlyHandler(SpyOnlyHandler handler) = 0; }; // Provides an interface of event. @@ -106,94 +67,52 @@ struct IBaseEvent { // be able to emit the event. template <typename TEventArgs> struct IEvent : virtual IBaseEvent { - public: - using EventArgs = DeducedEventArgs<TEventArgs>; - using EventHandler = std::function<void(EventArgs)>; - using ShortCircuitHandler = std::function<bool(EventArgs)>; - - protected: - IEvent() = default; - CRU_DELETE_COPY(IEvent) - CRU_DEFAULT_MOVE(IEvent) - ~IEvent() = default; // Note that user can't destroy a Event via IEvent. So - // destructor should be protected. - - public: - virtual EventRevoker AddHandler(EventHandler handler) = 0; - virtual EventRevoker AddShortCircuitHandler(ShortCircuitHandler handler) = 0; - virtual EventRevoker PrependShortCircuitHandler( - ShortCircuitHandler handler) = 0; + using Args = TEventArgs; + using Handler = std::function<void(Args)>; + virtual EventHandlerRevoker AddHandler(Handler handler) = 0; }; // A non-copyable non-movable Event class. // It stores a list of event handlers. template <typename TEventArgs> -class Event : public details::EventBase, public IEvent<TEventArgs> { - using typename IEvent<TEventArgs>::EventArgs; +class Event : public EventBase, public IEvent<TEventArgs> { + using typename IEvent<TEventArgs>::Args; using typename IBaseEvent::SpyOnlyHandler; - using typename IEvent<TEventArgs>::EventHandler; - using typename IEvent<TEventArgs>::ShortCircuitHandler; + using typename IEvent<TEventArgs>::Handler; private: struct HandlerData { - HandlerData(EventHandlerToken token, ShortCircuitHandler handler) + HandlerData(EventHandlerToken token, Handler handler) : token(token), handler(std::move(handler)) {} EventHandlerToken token; - ShortCircuitHandler handler; + Handler handler; }; public: - Event() = default; - CRU_DELETE_COPY(Event) - CRU_DEFAULT_MOVE(Event) - ~Event() = default; - - EventRevoker AddSpyOnlyHandler(SpyOnlyHandler handler) override { - return AddShortCircuitHandler([handler = std::move(handler)](EventArgs) { - handler(); - return false; - }); - } - - EventRevoker AddHandler(EventHandler handler) override { - return AddShortCircuitHandler( - [handler = std::move(handler)](EventArgs args) { - handler(args); - return false; - }); + EventHandlerRevoker AddSpyOnlyHandler(SpyOnlyHandler handler) override { + return AddHandler([handler = std::move(handler)](Args) { handler(); }); } - // Handler return true to short circuit following handlers. - EventRevoker AddShortCircuitHandler(ShortCircuitHandler handler) override { + EventHandlerRevoker AddHandler(Handler handler) override { const auto token = current_token_++; this->handler_data_list_.emplace_back(token, std::move(handler)); return CreateRevoker(token); } - // Handler return true to short circuit following handlers. - EventRevoker PrependShortCircuitHandler( - ShortCircuitHandler handler) override { - const auto token = current_token_++; - this->handler_data_list_.emplace(this->handler_data_list_.cbegin(), token, - std::move(handler)); - return CreateRevoker(token); - } - // This method will make a copy of all handlers. Because user might delete a // handler in a handler, which may lead to seg fault as the handler is // deleted while being executed. Thanks to this behavior, all handlers will // be taken a snapshot when Raise is called, so even if you delete a handler // during this period, all handlers in the snapshot will be executed. - void Raise(EventArgs args) { - std::vector<ShortCircuitHandler> handlers; + void Raise(Args args) { + std::vector<Handler> handlers; handlers.reserve(this->handler_data_list_.size()); for (const auto& data : this->handler_data_list_) { handlers.push_back(data.handler); } for (const auto& handler : handlers) { - auto short_circuit = handler(args); - if (short_circuit) return; + handler(args); } } @@ -213,8 +132,8 @@ class Event : public details::EventBase, public IEvent<TEventArgs> { }; namespace details { -struct EventRevokerDestroyer { - void operator()(EventRevoker* p) { +struct EventHandlerRevokerDestroyer { + void operator()(EventHandlerRevoker* p) { (*p)(); delete p; } @@ -222,51 +141,38 @@ struct EventRevokerDestroyer { } // namespace details // A guard class for event revoker. Automatically revoke it when destroyed. -class EventRevokerGuard { +class EventHandlerRevokerGuard { public: - EventRevokerGuard() = default; - explicit EventRevokerGuard(EventRevoker&& revoker) - : revoker_(new EventRevoker(std::move(revoker))) {} - EventRevokerGuard(const EventRevokerGuard& other) = delete; - EventRevokerGuard(EventRevokerGuard&& other) = default; - EventRevokerGuard& operator=(const EventRevokerGuard& other) = delete; - EventRevokerGuard& operator=(EventRevokerGuard&& other) = default; - ~EventRevokerGuard() = default; - - EventRevoker Get() { + EventHandlerRevokerGuard() = default; + explicit EventHandlerRevokerGuard(EventHandlerRevoker&& revoker) + : revoker_(new EventHandlerRevoker(std::move(revoker))) {} + + EventHandlerRevoker Get() { // revoker is only null when this is moved // you shouldn't use a moved instance assert(revoker_); return *revoker_; } - EventRevoker Release() { return std::move(*revoker_.release()); } + EventHandlerRevoker Release() { return std::move(*revoker_.release()); } void Reset() { revoker_.reset(); } - void Reset(EventRevoker&& revoker) { - revoker_.reset(new EventRevoker(std::move(revoker))); + void Reset(EventHandlerRevoker&& revoker) { + revoker_.reset(new EventHandlerRevoker(std::move(revoker))); } private: - std::unique_ptr<EventRevoker, details::EventRevokerDestroyer> revoker_; + std::unique_ptr<EventHandlerRevoker, details::EventHandlerRevokerDestroyer> revoker_; }; -class EventRevokerListGuard { - public: - EventRevokerListGuard() = default; - EventRevokerListGuard(const EventRevokerListGuard& other) = delete; - EventRevokerListGuard(EventRevokerListGuard&& other) = default; - EventRevokerListGuard& operator=(const EventRevokerListGuard& other) = delete; - EventRevokerListGuard& operator=(EventRevokerListGuard&& other) = default; - ~EventRevokerListGuard() = default; - +class EventHandlerRevokerListGuard { public: - void Add(EventRevoker&& revoker) { - event_revoker_guard_list_.push_back(EventRevokerGuard(std::move(revoker))); + void Add(EventHandlerRevoker&& revoker) { + event_revoker_guard_list_.push_back(EventHandlerRevokerGuard(std::move(revoker))); } - EventRevokerListGuard& operator+=(EventRevoker&& revoker) { + EventHandlerRevokerListGuard& operator+=(EventHandlerRevoker&& revoker) { this->Add(std::move(revoker)); return *this; } @@ -276,6 +182,6 @@ class EventRevokerListGuard { bool IsEmpty() const { return event_revoker_guard_list_.empty(); } private: - std::vector<EventRevokerGuard> event_revoker_guard_list_; + std::vector<EventHandlerRevokerGuard> event_revoker_guard_list_; }; } // namespace cru diff --git a/include/cru/platform/gui/InputMethod.h b/include/cru/platform/gui/InputMethod.h index d178cb8a..2d5d0b76 100644 --- a/include/cru/platform/gui/InputMethod.h +++ b/include/cru/platform/gui/InputMethod.h @@ -61,6 +61,6 @@ struct IInputMethodContext : virtual IPlatformResource { virtual IEvent<std::nullptr_t>* CompositionStartEvent() = 0; virtual IEvent<std::nullptr_t>* CompositionEndEvent() = 0; virtual IEvent<std::nullptr_t>* CompositionEvent() = 0; - virtual IEvent<std::string>* TextEvent() = 0; + virtual IEvent<const std::string&>* TextEvent() = 0; }; } // namespace cru::platform::gui diff --git a/include/cru/platform/gui/Window.h b/include/cru/platform/gui/Window.h index 3be5b21b..b8973c5a 100644 --- a/include/cru/platform/gui/Window.h +++ b/include/cru/platform/gui/Window.h @@ -102,16 +102,16 @@ struct CRU_PLATFORM_GUI_API INativeWindow : virtual IPlatformResource { virtual IEvent<std::nullptr_t>* PaintEvent() = 0; virtual IEvent<WindowVisibilityType>* VisibilityChangeEvent() = 0; - virtual IEvent<Size>* ResizeEvent() = 0; + virtual IEvent<const Size&>* ResizeEvent() = 0; virtual IEvent<FocusChangeType>* FocusEvent() = 0; virtual IEvent<MouseEnterLeaveType>* MouseEnterLeaveEvent() = 0; - virtual IEvent<Point>* MouseMoveEvent() = 0; - virtual IEvent<NativeMouseButtonEventArgs>* MouseDownEvent() = 0; - virtual IEvent<NativeMouseButtonEventArgs>* MouseUpEvent() = 0; - virtual IEvent<NativeMouseWheelEventArgs>* MouseWheelEvent() = 0; - virtual IEvent<NativeKeyEventArgs>* KeyDownEvent() = 0; - virtual IEvent<NativeKeyEventArgs>* KeyUpEvent() = 0; + virtual IEvent<const Point&>* MouseMoveEvent() = 0; + virtual IEvent<const NativeMouseButtonEventArgs&>* MouseDownEvent() = 0; + virtual IEvent<const NativeMouseButtonEventArgs&>* MouseUpEvent() = 0; + virtual IEvent<const NativeMouseWheelEventArgs&>* MouseWheelEvent() = 0; + virtual IEvent<const NativeKeyEventArgs&>* KeyDownEvent() = 0; + virtual IEvent<const NativeKeyEventArgs&>* KeyUpEvent() = 0; virtual IInputMethodContext* GetInputMethodContext() = 0; }; diff --git a/include/cru/platform/gui/osx/InputMethod.h b/include/cru/platform/gui/osx/InputMethod.h index 9044a76c..ebb51b3e 100644 --- a/include/cru/platform/gui/osx/InputMethod.h +++ b/include/cru/platform/gui/osx/InputMethod.h @@ -42,7 +42,7 @@ class OsxInputMethodContext : public OsxGuiResource, IEvent<std::nullptr_t>* CompositionEvent() override; - IEvent<std::string>* TextEvent() override; + IEvent<const std::string&>* TextEvent() override; bool IsEnabled(); diff --git a/include/cru/platform/gui/osx/Window.h b/include/cru/platform/gui/osx/Window.h index f4bda28e..e36f4238 100644 --- a/include/cru/platform/gui/osx/Window.h +++ b/include/cru/platform/gui/osx/Window.h @@ -66,16 +66,16 @@ class OsxWindow : public OsxGuiResource, public INativeWindow { IEvent<std::nullptr_t>* PaintEvent() override; IEvent<WindowVisibilityType>* VisibilityChangeEvent() override; - IEvent<Size>* ResizeEvent() override; + IEvent<const Size&>* ResizeEvent() override; IEvent<FocusChangeType>* FocusEvent() override; IEvent<MouseEnterLeaveType>* MouseEnterLeaveEvent() override; - IEvent<Point>* MouseMoveEvent() override; - IEvent<NativeMouseButtonEventArgs>* MouseDownEvent() override; - IEvent<NativeMouseButtonEventArgs>* MouseUpEvent() override; - IEvent<NativeMouseWheelEventArgs>* MouseWheelEvent() override; - IEvent<NativeKeyEventArgs>* KeyDownEvent() override; - IEvent<NativeKeyEventArgs>* KeyUpEvent() override; + IEvent<const Point&>* MouseMoveEvent() override; + IEvent<const NativeMouseButtonEventArgs&>* MouseDownEvent() override; + IEvent<const NativeMouseButtonEventArgs&>* MouseUpEvent() override; + IEvent<const NativeMouseWheelEventArgs&>* MouseWheelEvent() override; + IEvent<const NativeKeyEventArgs&>* KeyDownEvent() override; + IEvent<const NativeKeyEventArgs&>* KeyUpEvent() override; IInputMethodContext* GetInputMethodContext() override; diff --git a/include/cru/platform/gui/sdl/Window.h b/include/cru/platform/gui/sdl/Window.h index c818ecce..8ffadd02 100644 --- a/include/cru/platform/gui/sdl/Window.h +++ b/include/cru/platform/gui/sdl/Window.h @@ -64,16 +64,16 @@ class SdlWindow : public SdlResource, public virtual INativeWindow { IEvent<std::nullptr_t>* PaintEvent() override; IEvent<WindowVisibilityType>* VisibilityChangeEvent() override; - IEvent<Size>* ResizeEvent() override; + IEvent<const Size&>* ResizeEvent() override; IEvent<FocusChangeType>* FocusEvent() override; IEvent<MouseEnterLeaveType>* MouseEnterLeaveEvent() override; - IEvent<Point>* MouseMoveEvent() override; - IEvent<NativeMouseButtonEventArgs>* MouseDownEvent() override; - IEvent<NativeMouseButtonEventArgs>* MouseUpEvent() override; - IEvent<NativeMouseWheelEventArgs>* MouseWheelEvent() override; - IEvent<NativeKeyEventArgs>* KeyDownEvent() override; - IEvent<NativeKeyEventArgs>* KeyUpEvent() override; + IEvent<const Point&>* MouseMoveEvent() override; + IEvent<const NativeMouseButtonEventArgs&>* MouseDownEvent() override; + IEvent<const NativeMouseButtonEventArgs&>* MouseUpEvent() override; + IEvent<const NativeMouseWheelEventArgs&>* MouseWheelEvent() override; + IEvent<const NativeKeyEventArgs&>* KeyDownEvent() override; + IEvent<const NativeKeyEventArgs&>* KeyUpEvent() override; IInputMethodContext* GetInputMethodContext() override; @@ -91,14 +91,14 @@ class SdlWindow : public SdlResource, public virtual INativeWindow { Event<std::nullptr_t> destroy_event_; Event<std::nullptr_t> paint_event_; Event<WindowVisibilityType> visibility_change_event_; - Event<Size> resize_event_; + Event<const Size&> resize_event_; Event<FocusChangeType> focus_event_; Event<MouseEnterLeaveType> mouse_enter_leave_event_; - Event<Point> mouse_move_event_; - Event<NativeMouseButtonEventArgs> mouse_down_event_; - Event<NativeMouseButtonEventArgs> mouse_up_event_; - Event<NativeMouseWheelEventArgs> mouse_wheel_event_; - Event<NativeKeyEventArgs> key_down_event_; - Event<NativeKeyEventArgs> key_up_event_; + Event<const Point&> mouse_move_event_; + Event<const NativeMouseButtonEventArgs&> mouse_down_event_; + Event<const NativeMouseButtonEventArgs&> mouse_up_event_; + Event<const NativeMouseWheelEventArgs&> mouse_wheel_event_; + Event<const NativeKeyEventArgs&> key_down_event_; + Event<const NativeKeyEventArgs&> key_up_event_; }; } // namespace cru::platform::gui::xcb diff --git a/include/cru/platform/gui/win/InputMethod.h b/include/cru/platform/gui/win/InputMethod.h index c246cbd7..fa4d5882 100644 --- a/include/cru/platform/gui/win/InputMethod.h +++ b/include/cru/platform/gui/win/InputMethod.h @@ -66,7 +66,7 @@ class CRU_WIN_GUI_API WinInputMethodContext IEvent<std::nullptr_t>* CompositionEvent() override; - IEvent<std::string>* TextEvent() override; + IEvent<const std::string&>* TextEvent() override; private: void OnWindowNativeMessage(WindowNativeMessageEventArgs& args); @@ -78,11 +78,11 @@ class CRU_WIN_GUI_API WinInputMethodContext private: WinNativeWindow* native_window_; - EventRevokerListGuard event_guard_; + EventHandlerRevokerListGuard event_guard_; Event<std::nullptr_t> composition_start_event_; Event<std::nullptr_t> composition_end_event_; Event<std::nullptr_t> composition_event_; - Event<std::string> text_event_; + Event<const std::string&> text_event_; }; } // namespace cru::platform::gui::win diff --git a/include/cru/platform/gui/win/Window.h b/include/cru/platform/gui/win/Window.h index cd720fbd..f315f3d3 100644 --- a/include/cru/platform/gui/win/Window.h +++ b/include/cru/platform/gui/win/Window.h @@ -69,19 +69,19 @@ class CRU_WIN_GUI_API WinNativeWindow : public WinNativeResource, IEvent<WindowVisibilityType>* VisibilityChangeEvent() override { return &visibility_change_event_; } - IEvent<Size>* ResizeEvent() override { return &resize_event_; } + IEvent<const Size&>* ResizeEvent() override { return &resize_event_; } IEvent<FocusChangeType>* FocusEvent() override { return &focus_event_; } IEvent<MouseEnterLeaveType>* MouseEnterLeaveEvent() override { return &mouse_enter_leave_event_; } - IEvent<Point>* MouseMoveEvent() override { return &mouse_move_event_; } + IEvent<const Point&>* MouseMoveEvent() override { return &mouse_move_event_; } IEvent<platform::gui::NativeMouseButtonEventArgs>* MouseDownEvent() override { return &mouse_down_event_; } IEvent<platform::gui::NativeMouseButtonEventArgs>* MouseUpEvent() override { return &mouse_up_event_; } - IEvent<NativeMouseWheelEventArgs>* MouseWheelEvent() override { + IEvent<const NativeMouseWheelEventArgs&>* MouseWheelEvent() override { return &mouse_wheel_event_; } @@ -200,11 +200,11 @@ class CRU_WIN_GUI_API WinNativeWindow : public WinNativeResource, Event<std::nullptr_t> create_event_; Event<std::nullptr_t> destroy_event_; Event<std::nullptr_t> paint_event_; - Event<Size> resize_event_; + Event<const Size&> resize_event_; Event<WindowVisibilityType> visibility_change_event_; Event<FocusChangeType> focus_event_; Event<MouseEnterLeaveType> mouse_enter_leave_event_; - Event<Point> mouse_move_event_; + Event<const Point&> mouse_move_event_; Event<platform::gui::NativeMouseButtonEventArgs> mouse_down_event_; Event<platform::gui::NativeMouseButtonEventArgs> mouse_up_event_; Event<platform::gui::NativeMouseWheelEventArgs> mouse_wheel_event_; diff --git a/include/cru/platform/gui/xcb/InputMethod.h b/include/cru/platform/gui/xcb/InputMethod.h index e32ed63e..710ea2e5 100644 --- a/include/cru/platform/gui/xcb/InputMethod.h +++ b/include/cru/platform/gui/xcb/InputMethod.h @@ -67,7 +67,7 @@ class XcbXimInputMethodContext : public XcbResource, IEvent<std::nullptr_t>* CompositionStartEvent() override; IEvent<std::nullptr_t>* CompositionEndEvent() override; IEvent<std::nullptr_t>* CompositionEvent() override; - IEvent<std::string>* TextEvent() override; + IEvent<const std::string&>* TextEvent() override; private: void CreateIc(xcb_window_t window); @@ -84,6 +84,6 @@ class XcbXimInputMethodContext : public XcbResource, Event<std::nullptr_t> composition_start_event_; Event<std::nullptr_t> composition_end_event_; Event<std::nullptr_t> composition_event_; - Event<std::string> text_event_; + Event<const std::string&> text_event_; }; } // namespace cru::platform::gui::xcb diff --git a/include/cru/platform/gui/xcb/Window.h b/include/cru/platform/gui/xcb/Window.h index bfd86297..4cb5f6da 100644 --- a/include/cru/platform/gui/xcb/Window.h +++ b/include/cru/platform/gui/xcb/Window.h @@ -68,16 +68,16 @@ class XcbWindow : public XcbResource, public virtual INativeWindow { IEvent<std::nullptr_t>* PaintEvent() override; IEvent<WindowVisibilityType>* VisibilityChangeEvent() override; - IEvent<Size>* ResizeEvent() override; + IEvent<const Size&>* ResizeEvent() override; IEvent<FocusChangeType>* FocusEvent() override; IEvent<MouseEnterLeaveType>* MouseEnterLeaveEvent() override; - IEvent<Point>* MouseMoveEvent() override; - IEvent<NativeMouseButtonEventArgs>* MouseDownEvent() override; - IEvent<NativeMouseButtonEventArgs>* MouseUpEvent() override; - IEvent<NativeMouseWheelEventArgs>* MouseWheelEvent() override; - IEvent<NativeKeyEventArgs>* KeyDownEvent() override; - IEvent<NativeKeyEventArgs>* KeyUpEvent() override; + IEvent<const Point&>* MouseMoveEvent() override; + IEvent<const NativeMouseButtonEventArgs&>* MouseDownEvent() override; + IEvent<const NativeMouseButtonEventArgs&>* MouseUpEvent() override; + IEvent<const NativeMouseWheelEventArgs&>* MouseWheelEvent() override; + IEvent<const NativeKeyEventArgs&>* KeyDownEvent() override; + IEvent<const NativeKeyEventArgs&>* KeyUpEvent() override; IInputMethodContext* GetInputMethodContext() override; @@ -127,14 +127,14 @@ class XcbWindow : public XcbResource, public virtual INativeWindow { Event<std::nullptr_t> paint_event_; Event<WindowVisibilityType> visibility_change_event_; - Event<Size> resize_event_; + Event<const Size&> resize_event_; Event<FocusChangeType> focus_event_; Event<MouseEnterLeaveType> mouse_enter_leave_event_; - Event<Point> mouse_move_event_; - Event<NativeMouseButtonEventArgs> mouse_down_event_; - Event<NativeMouseButtonEventArgs> mouse_up_event_; - Event<NativeMouseWheelEventArgs> mouse_wheel_event_; - Event<NativeKeyEventArgs> key_down_event_; - Event<NativeKeyEventArgs> key_up_event_; + Event<const Point&> mouse_move_event_; + Event<const NativeMouseButtonEventArgs&> mouse_down_event_; + Event<const NativeMouseButtonEventArgs&> mouse_up_event_; + Event<const NativeMouseWheelEventArgs&> mouse_wheel_event_; + Event<const NativeKeyEventArgs&> key_down_event_; + Event<const NativeKeyEventArgs&> key_up_event_; }; } // namespace cru::platform::gui::xcb diff --git a/include/cru/ui/controls/Button.h b/include/cru/ui/controls/Button.h index 3c01be32..e8fa50f1 100644 --- a/include/cru/ui/controls/Button.h +++ b/include/cru/ui/controls/Button.h @@ -33,7 +33,7 @@ class CRU_UI_API Button : public SingleChildControl<render::BorderRenderObject>, return click_detector_.StateChangeEvent(); } - IEvent<helper::ClickEventArgs>* ClickEvent() { + IEvent<const helper::ClickEventArgs&>* ClickEvent() { return click_detector_.ClickEvent(); } diff --git a/include/cru/ui/controls/IconButton.h b/include/cru/ui/controls/IconButton.h index 9e7572e0..e52d2a26 100644 --- a/include/cru/ui/controls/IconButton.h +++ b/include/cru/ui/controls/IconButton.h @@ -39,7 +39,7 @@ class CRU_UI_API IconButton : public NoChildControl, return click_detector_.StateChangeEvent(); } - IEvent<helper::ClickEventArgs>* ClickEvent() { + IEvent<const helper::ClickEventArgs&>* ClickEvent() { return click_detector_.ClickEvent(); } @@ -87,7 +87,8 @@ class CRU_UI_API IconButton : public NoChildControl, void SetIconWithSvgPathDataString(std::string_view icon_svg_path_data_string, const Rect& view_port); void SetIconWithSvgPathDataStringResourceKey( - std::string_view icon_svg_path_data_string_resource_key, const Rect& view_port); + std::string_view icon_svg_path_data_string_resource_key, + const Rect& view_port); std::shared_ptr<platform::graphics::IBrush> GetContentBrush() const override { return GetIconFillBrush(); diff --git a/include/cru/ui/controls/RootControl.h b/include/cru/ui/controls/RootControl.h index 5fa8090e..e662b655 100644 --- a/include/cru/ui/controls/RootControl.h +++ b/include/cru/ui/controls/RootControl.h @@ -28,7 +28,7 @@ class CRU_UI_API RootControl Control* attached_control_; - EventRevokerListGuard + EventHandlerRevokerListGuard gain_focus_on_create_and_destroy_when_lose_focus_event_guard_; }; } // namespace cru::ui::controls diff --git a/include/cru/ui/controls/TextHostControlService.h b/include/cru/ui/controls/TextHostControlService.h index 529225a7..a9f4f22b 100644 --- a/include/cru/ui/controls/TextHostControlService.h +++ b/include/cru/ui/controls/TextHostControlService.h @@ -187,8 +187,8 @@ class CRU_UI_API TextHostControlService : public Object { Event<std::nullptr_t> text_change_event_; - EventRevokerListGuard event_guard_; - EventRevokerListGuard input_method_context_event_guard_; + EventHandlerRevokerListGuard event_guard_; + EventHandlerRevokerListGuard input_method_context_event_guard_; std::string text_; TextRange selection_; diff --git a/include/cru/ui/events/RoutedEvent.h b/include/cru/ui/events/RoutedEvent.h index a01839a6..9a07e1e9 100644 --- a/include/cru/ui/events/RoutedEvent.h +++ b/include/cru/ui/events/RoutedEvent.h @@ -16,7 +16,7 @@ class CRU_UI_API RoutedEvent { using RawEventArgs = TEventArgs; using IEventType = IEvent<TEventArgs&>; - using EventArgs = typename IEventType::EventArgs; + using EventArgs = typename IEventType::Args; RoutedEvent() = default; RoutedEvent(const RoutedEvent& other) = delete; diff --git a/include/cru/ui/helper/ClickDetector.h b/include/cru/ui/helper/ClickDetector.h index eaf1f4af..2eb453d9 100644 --- a/include/cru/ui/helper/ClickDetector.h +++ b/include/cru/ui/helper/ClickDetector.h @@ -57,7 +57,7 @@ class ClickDetector : public Object { // deactivated. void SetTriggerButton(MouseButton trigger_button); - IEvent<ClickEventArgs>* ClickEvent() { return &event_; } + IEvent<const ClickEventArgs&>* ClickEvent() { return &event_; } IEvent<ClickState>* StateChangeEvent() { return &state_change_event_; } @@ -72,10 +72,10 @@ class ClickDetector : public Object { bool enable_ = true; MouseButton trigger_button_ = MouseButtons::Left | MouseButtons::Right; - Event<ClickEventArgs> event_; + Event<const ClickEventArgs&> event_; Event<ClickState> state_change_event_; - std::vector<EventRevokerGuard> event_rovoker_guards_; + std::vector<EventHandlerRevokerGuard> event_rovoker_guards_; Point down_point_; MouseButton button_; diff --git a/include/cru/ui/helper/ShortcutHub.h b/include/cru/ui/helper/ShortcutHub.h index d2c9952b..1b600c48 100644 --- a/include/cru/ui/helper/ShortcutHub.h +++ b/include/cru/ui/helper/ShortcutHub.h @@ -131,6 +131,6 @@ class CRU_UI_API ShortcutHub : public Object { Event<events::KeyEventArgs&> fallback_event_; - EventRevokerListGuard event_guard_; + EventHandlerRevokerListGuard event_guard_; }; } // namespace cru::ui::helper diff --git a/include/cru/ui/host/WindowHost.h b/include/cru/ui/host/WindowHost.h index 6e7a0f2e..889665d5 100644 --- a/include/cru/ui/host/WindowHost.h +++ b/include/cru/ui/host/WindowHost.h @@ -156,7 +156,7 @@ class CRU_UI_API WindowHost : public Object, public SelfResolvable<WindowHost> { Event<AfterLayoutEventArgs> after_layout_event_; std::vector<std::function<void()> > after_layout_stable_action_; - std::vector<EventRevokerGuard> event_revoker_guards_; + std::vector<EventHandlerRevokerGuard> event_revoker_guards_; controls::Control* mouse_hover_control_ = nullptr; diff --git a/include/cru/ui/render/ScrollBar.h b/include/cru/ui/render/ScrollBar.h index 8c531489..50b21386 100644 --- a/include/cru/ui/render/ScrollBar.h +++ b/include/cru/ui/render/ScrollBar.h @@ -136,7 +136,7 @@ class CRU_UI_API ScrollBar : public Object { std::optional<ScrollBarAreaKind> mouse_hover_; std::optional<ScrollBarAreaKind> mouse_press_; - EventRevokerListGuard event_guard_; + EventHandlerRevokerListGuard event_guard_; Event<Scroll> scroll_attempt_event_; diff --git a/include/cru/ui/render/ScrollRenderObject.h b/include/cru/ui/render/ScrollRenderObject.h index 63a49aa3..348d7af1 100644 --- a/include/cru/ui/render/ScrollRenderObject.h +++ b/include/cru/ui/render/ScrollRenderObject.h @@ -95,6 +95,6 @@ class CRU_UI_API ScrollRenderObject : public SingleChildRenderObject { bool is_mouse_wheel_enabled_ = true; - EventRevokerListGuard guard_; + EventHandlerRevokerListGuard guard_; }; } // namespace cru::ui::render diff --git a/include/cru/ui/style/StyleRuleSet.h b/include/cru/ui/style/StyleRuleSet.h index 0b7a237d..d89c3ae1 100644 --- a/include/cru/ui/style/StyleRuleSet.h +++ b/include/cru/ui/style/StyleRuleSet.h @@ -63,7 +63,7 @@ class CRU_UI_API StyleRuleSet : public Object, public model::IListChangeNotify { Event<model::ListChange> list_change_event_; std::shared_ptr<StyleRuleSet> parent_ = nullptr; - EventRevokerGuard parent_change_event_guard_; + EventHandlerRevokerGuard parent_change_event_guard_; std::vector<StyleRule> rules_; }; @@ -90,6 +90,6 @@ class CRU_UI_API StyleRuleSetBind { // child first, parent last. std::vector<StyleRuleSet*> ruleset_chain_cache_; - EventRevokerListGuard guard_; + EventHandlerRevokerListGuard guard_; }; } // namespace cru::ui::style |