avmedia/source/viewer/mediawindow_impl.cxx | 2 include/svx/svdview.hxx | 10 - include/vcl/commandevent.hxx | 259 ++++------------------------- svx/source/sdr/contact/sdrmediawindow.cxx | 2 vcl/source/window/commandevent.cxx | 162 ++++++++++++++++++ 5 files changed, 208 insertions(+), 227 deletions(-)
New commits: commit 63fd137cd54fb6df0c812a97d962239b8d0996a0 Author: Chris Sherlock <chris.sherloc...@gmail.com> Date: Wed Feb 10 17:04:56 2016 +1100 svx: updated comment function signature return types Change-Id: I4ebe95660b18fae2339876866cd95c7ca752e63d diff --git a/include/svx/svdview.hxx b/include/svx/svdview.hxx index eb2b653..614fe1f 100644 --- a/include/svx/svdview.hxx +++ b/include/svx/svdview.hxx @@ -265,11 +265,11 @@ public: // ~~~~~~~~~~~~~~~~~~~~ // simple events: // ~~~~~~~~~~~~~~~~ -// sal_Bool KeyInput(const KeyEvent& rKEvt, vcl::Window* pWin); -// sal_Bool MouseButtonDown(const MouseEvent& rMEvt, vcl::Window* pWin); -// sal_Bool MouseButtonUp(const MouseEvent& rMEvt, vcl::Window* pWin); -// sal_Bool MouseMove(const MouseEvent& rMEvt, vcl::Window* pWin); -// sal_Bool Command(const CommandEvent& rCEvt, vcl::Window* pWin); +// bool KeyInput(const KeyEvent& rKEvt, vcl::Window* pWin); +// bool MouseButtonDown(const MouseEvent& rMEvt, vcl::Window* pWin); +// bool MouseButtonUp(const MouseEvent& rMEvt, vcl::Window* pWin); +// bool MouseMove(const MouseEvent& rMEvt, vcl::Window* pWin); +// bool Command(const CommandEvent& rCEvt, vcl::Window* pWin); // // Exchange (Clipboard derzeit noch ohne SdrPrivateData): // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ commit 3ff9dd6ff36f21d9bea1851cea05a4ed4228722d Author: Chris Sherlock <chris.sherloc...@gmail.com> Date: Wed Feb 10 16:54:47 2016 +1100 vcl: move CommandEvent functions from header to cxx file Moving functions from commandevent.hxx to commandevent.cxx, this means they are no longer inline, but there was no clear reason as to why they were inline in the first place! I've also reordered the header to ensure that CommandEvent is the first fully declared class, the data classes get declared after this (hence the forward declarations). This makes reading this header much easier. One more small thing: a few if statements had braces in the primary if block, but not in the else block, which is IMO bad style and can possibly lead to errors. I've made the else block use braces. Change-Id: I79b010bd66d3c9f4a74d4fa470f62d0263c811f4 diff --git a/avmedia/source/viewer/mediawindow_impl.cxx b/avmedia/source/viewer/mediawindow_impl.cxx index dbf3415..6206abb 100644 --- a/avmedia/source/viewer/mediawindow_impl.cxx +++ b/avmedia/source/viewer/mediawindow_impl.cxx @@ -117,7 +117,7 @@ void MediaChildWindow::KeyUp( const KeyEvent& rKEvt ) void MediaChildWindow::Command( const CommandEvent& rCEvt ) { const CommandEvent aTransformedEvent( GetParent()->ScreenToOutputPixel( OutputToScreenPixel( rCEvt.GetMousePosPixel() ) ), - rCEvt.GetCommand(), rCEvt.IsMouseEvent(), rCEvt.GetEventData() ); + rCEvt.GetCommand(), rCEvt.IsMouseEvent(), rCEvt.GetEventData() ); SystemChildWindow::Command( rCEvt ); GetParent()->Command( aTransformedEvent ); diff --git a/include/vcl/commandevent.hxx b/include/vcl/commandevent.hxx index a9fef58..9670aea 100644 --- a/include/vcl/commandevent.hxx +++ b/include/vcl/commandevent.hxx @@ -38,6 +38,45 @@ #define EXTTEXTINPUT_CURSOR_INVISIBLE ((sal_uInt16)0x0001) #define EXTTEXTINPUT_CURSOR_OVERWRITE ((sal_uInt16)0x0002) +class CommandExtTextInputData; +class CommandWheelData; +class CommandScrollData; +class CommandModKeyData; +class CommandDialogData; +class CommandMediaData; +class CommandSelectionChangeData; +class CommandSwipeData; +class CommandLongPressData; +enum class CommandEventId; + +class VCL_DLLPUBLIC CommandEvent +{ +private: + Point maPos; + void* mpData; + CommandEventId mnCommand; + bool mbMouseEvent; + +public: + CommandEvent(); + CommandEvent( const Point& rMousePos, CommandEventId nCmd, + bool bMEvt = false, const void* pCmdData = nullptr ); + + CommandEventId GetCommand() const { return mnCommand; } + const Point& GetMousePosPixel() const { return maPos; } + bool IsMouseEvent() const { return mbMouseEvent; } + void* GetEventData() const { return mpData; } + + const CommandExtTextInputData* GetExtTextInputData() const; + const CommandWheelData* GetWheelData() const; + const CommandScrollData* GetAutoScrollData() const; + const CommandModKeyData* GetModKeyData() const; + const CommandDialogData* GetDialogData() const; + CommandMediaData* GetMediaData() const; + const CommandSelectionChangeData* GetSelectionChangeData() const; + const CommandSwipeData* GetSwipeData() const; + const CommandLongPressData* GetLongPressData() const; +}; class VCL_DLLPUBLIC CommandExtTextInputData { @@ -66,7 +105,6 @@ public: bool IsOnlyCursorChanged() const { return mbOnlyCursor; } }; - class VCL_DLLPUBLIC CommandInputContextData { private: @@ -77,19 +115,6 @@ public: CommandInputContextData( LanguageType eLang ); }; - -inline CommandInputContextData::CommandInputContextData() -{ - meLanguage = LANGUAGE_DONTKNOW; -} - - -inline CommandInputContextData::CommandInputContextData( LanguageType eLang ) -{ - meLanguage = eLang; -} - - enum class CommandWheelMode { NONE = 0, @@ -99,7 +124,6 @@ enum class CommandWheelMode DATAZOOM = 4 }; - // Magic value used in mnLines field in CommandWheelData #define COMMAND_WHEEL_PAGESCROLL ((sal_uLong)0xFFFFFFFF) @@ -139,34 +163,6 @@ public: { return ((mnCode & KEY_MOD2) != 0); } }; - -inline CommandWheelData::CommandWheelData() -{ - mnDelta = 0; - mnNotchDelta = 0; - mnLines = 0; - mnWheelMode = CommandWheelMode::NONE; - mnCode = 0; - mbHorz = false; - mbDeltaIsPixel = false; -} - - -inline CommandWheelData::CommandWheelData( long nWheelDelta, long nWheelNotchDelta, - sal_uLong nScrollLines, - CommandWheelMode nWheelMode, sal_uInt16 nKeyModifier, - bool bHorz, bool bDeltaIsPixel ) -{ - mnDelta = nWheelDelta; - mnNotchDelta = nWheelNotchDelta; - mnLines = nScrollLines; - mnWheelMode = nWheelMode; - mnCode = nKeyModifier; - mbHorz = bHorz; - mbDeltaIsPixel = bDeltaIsPixel; -} - - class VCL_DLLPUBLIC CommandScrollData { private: @@ -181,21 +177,6 @@ public: long GetDeltaY() const { return mnDeltaY; } }; - -inline CommandScrollData::CommandScrollData() -{ - mnDeltaX = 0; - mnDeltaY = 0; -} - - -inline CommandScrollData::CommandScrollData( long nDeltaX, long nDeltaY ) -{ - mnDeltaX = nDeltaX; - mnDeltaY = nDeltaY; -} - - class VCL_DLLPUBLIC CommandModKeyData { private: @@ -211,26 +192,12 @@ public: bool IsRightShift() const { return (mnCode & MODKEY_RSHIFT) != 0; } }; - -inline CommandModKeyData::CommandModKeyData() -{ - mnCode = 0L; -} - - -inline CommandModKeyData::CommandModKeyData( sal_uInt16 nCode ) -{ - mnCode = nCode; -} - - enum class ShowDialogId { Preferences = 1, About = 2, }; - class VCL_DLLPUBLIC CommandDialogData { ShowDialogId m_nDialogId; @@ -242,7 +209,6 @@ public: ShowDialogId GetDialogId() const { return m_nDialogId; } }; - // Media Commands enum class MediaCommand { @@ -270,7 +236,6 @@ enum class MediaCommand PreviousTrackHold = 22,// Button Left holding pressed. }; - class VCL_DLLPUBLIC CommandMediaData { MediaCommand m_nMediaId; @@ -286,7 +251,6 @@ public: bool GetPassThroughToOS() const { return m_bPassThroughToOS; } }; - class VCL_DLLPUBLIC CommandSelectionChangeData { private: @@ -301,21 +265,6 @@ public: sal_uLong GetEnd() const { return mnEnd; } }; - -inline CommandSelectionChangeData::CommandSelectionChangeData() -{ - mnStart = mnEnd = 0; -} - - -inline CommandSelectionChangeData::CommandSelectionChangeData( sal_uLong nStart, - sal_uLong nEnd ) -{ - mnStart = nStart; - mnEnd = nEnd; -} - - class VCL_DLLPUBLIC CommandSwipeData { double mnVelocityX; @@ -351,7 +300,6 @@ public: double getY() const { return mnY; } }; - enum class CommandEventId { NONE = 0, @@ -378,135 +326,6 @@ enum class CommandEventId LongPress = 22, }; - -class VCL_DLLPUBLIC CommandEvent -{ -private: - Point maPos; - void* mpData; - CommandEventId mnCommand; - bool mbMouseEvent; - -public: - CommandEvent(); - CommandEvent( const Point& rMousePos, CommandEventId nCmd, - bool bMEvt = false, const void* pCmdData = nullptr ); - - CommandEventId GetCommand() const { return mnCommand; } - const Point& GetMousePosPixel() const { return maPos; } - bool IsMouseEvent() const { return mbMouseEvent; } - void* GetEventData() const { return mpData; } - - const CommandExtTextInputData* GetExtTextInputData() const; - const CommandWheelData* GetWheelData() const; - const CommandScrollData* GetAutoScrollData() const; - const CommandModKeyData* GetModKeyData() const; - const CommandDialogData* GetDialogData() const; - CommandMediaData* GetMediaData() const; - const CommandSelectionChangeData* GetSelectionChangeData() const; - const CommandSwipeData* GetSwipeData() const; - const CommandLongPressData* GetLongPressData() const; -}; - - -inline CommandEvent::CommandEvent() -{ - mpData = nullptr; - mnCommand = CommandEventId::NONE; - mbMouseEvent = false; -} - - -inline CommandEvent::CommandEvent( const Point& rMousePos, - CommandEventId nCmd, bool bMEvt, const void* pCmdData ) : - maPos( rMousePos ) -{ - mpData = const_cast<void*>(pCmdData); - mnCommand = nCmd; - mbMouseEvent = bMEvt; -} - - -inline const CommandExtTextInputData* CommandEvent::GetExtTextInputData() const -{ - if ( mnCommand == CommandEventId::ExtTextInput ) - return static_cast<const CommandExtTextInputData*>(mpData); - else - return nullptr; -} - - -inline const CommandWheelData* CommandEvent::GetWheelData() const -{ - if ( mnCommand == CommandEventId::Wheel ) - return static_cast<const CommandWheelData*>(mpData); - else - return nullptr; -} - - -inline const CommandScrollData* CommandEvent::GetAutoScrollData() const -{ - if ( mnCommand == CommandEventId::AutoScroll ) - return static_cast<const CommandScrollData*>(mpData); - else - return nullptr; -} - - -inline const CommandModKeyData* CommandEvent::GetModKeyData() const -{ - if( mnCommand == CommandEventId::ModKeyChange ) - return static_cast<const CommandModKeyData*>(mpData); - else - return nullptr; -} - - -inline const CommandDialogData* CommandEvent::GetDialogData() const -{ - if( mnCommand == CommandEventId::ShowDialog ) - return static_cast<const CommandDialogData*>(mpData); - else - return nullptr; -} - - -inline CommandMediaData* CommandEvent::GetMediaData() const -{ - if( mnCommand == CommandEventId::Media ) - return static_cast<CommandMediaData*>(mpData); - else - return nullptr; -} - - -inline const CommandSelectionChangeData* CommandEvent::GetSelectionChangeData() const -{ - if( mnCommand == CommandEventId::SelectionChange ) - return static_cast<const CommandSelectionChangeData*>(mpData); - else - return nullptr; -} - - -inline const CommandSwipeData* CommandEvent::GetSwipeData() const -{ - if( mnCommand == CommandEventId::Swipe ) - return static_cast<const CommandSwipeData*>(mpData); - else - return nullptr; -} - - -inline const CommandLongPressData* CommandEvent::GetLongPressData() const -{ - if( mnCommand == CommandEventId::LongPress ) - return static_cast<const CommandLongPressData*>(mpData); - else - return nullptr; -} - #endif // INCLUDED_VCL_COMMANDEVENT_HXX /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/svx/source/sdr/contact/sdrmediawindow.cxx b/svx/source/sdr/contact/sdrmediawindow.cxx index d5f0734..1292d32 100644 --- a/svx/source/sdr/contact/sdrmediawindow.cxx +++ b/svx/source/sdr/contact/sdrmediawindow.cxx @@ -107,7 +107,7 @@ void SdrMediaWindow::Command( const CommandEvent& rCEvt ) if( pWindow && getWindow() ) { const CommandEvent aTransformedEvent( pWindow->ScreenToOutputPixel( getWindow()->OutputToScreenPixel( rCEvt.GetMousePosPixel() ) ), - rCEvt.GetCommand(), rCEvt.IsMouseEvent(), rCEvt.GetEventData() ); + rCEvt.GetCommand(), rCEvt.IsMouseEvent(), rCEvt.GetEventData() ); pWindow->Command( aTransformedEvent ); } diff --git a/vcl/source/window/commandevent.cxx b/vcl/source/window/commandevent.cxx index 143628b..5fa8e4c 100644 --- a/vcl/source/window/commandevent.cxx +++ b/vcl/source/window/commandevent.cxx @@ -32,7 +32,10 @@ CommandExtTextInputData::CommandExtTextInputData( const OUString& rText, memcpy( mpTextAttr, pTextAttr, maText.getLength()*sizeof(sal_uInt16) ); } else + { mpTextAttr = nullptr; + } + mnCursorPos = nCursorPos; mnCursorFlags = nCursorFlags; mbOnlyCursor = bOnlyCursor; @@ -47,7 +50,10 @@ CommandExtTextInputData::CommandExtTextInputData( const CommandExtTextInputData& memcpy( mpTextAttr, rData.mpTextAttr, maText.getLength()*sizeof(sal_uInt16) ); } else + { mpTextAttr = nullptr; + } + mnCursorPos = rData.mnCursorPos; mnCursorFlags = rData.mnCursorFlags; mbOnlyCursor = rData.mbOnlyCursor; @@ -58,4 +64,160 @@ CommandExtTextInputData::~CommandExtTextInputData() delete [] mpTextAttr; } +CommandInputContextData::CommandInputContextData() +{ + meLanguage = LANGUAGE_DONTKNOW; +} + +CommandInputContextData::CommandInputContextData( LanguageType eLang ) +{ + meLanguage = eLang; +} + +CommandWheelData::CommandWheelData() +{ + mnDelta = 0; + mnNotchDelta = 0; + mnLines = 0; + mnWheelMode = CommandWheelMode::NONE; + mnCode = 0; + mbHorz = false; + mbDeltaIsPixel = false; +} + +CommandWheelData::CommandWheelData( long nWheelDelta, long nWheelNotchDelta, + sal_uLong nScrollLines, + CommandWheelMode nWheelMode, sal_uInt16 nKeyModifier, + bool bHorz, bool bDeltaIsPixel ) +{ + mnDelta = nWheelDelta; + mnNotchDelta = nWheelNotchDelta; + mnLines = nScrollLines; + mnWheelMode = nWheelMode; + mnCode = nKeyModifier; + mbHorz = bHorz; + mbDeltaIsPixel = bDeltaIsPixel; +} + +CommandScrollData::CommandScrollData() +{ + mnDeltaX = 0; + mnDeltaY = 0; +} + +CommandScrollData::CommandScrollData( long nDeltaX, long nDeltaY ) +{ + mnDeltaX = nDeltaX; + mnDeltaY = nDeltaY; +} + +CommandModKeyData::CommandModKeyData() +{ + mnCode = 0L; +} + +CommandModKeyData::CommandModKeyData( sal_uInt16 nCode ) +{ + mnCode = nCode; +} + +CommandSelectionChangeData::CommandSelectionChangeData() +{ + mnStart = mnEnd = 0; +} + +CommandSelectionChangeData::CommandSelectionChangeData( sal_uLong nStart, sal_uLong nEnd ) +{ + mnStart = nStart; + mnEnd = nEnd; +} + +CommandEvent::CommandEvent() +{ + mpData = nullptr; + mnCommand = CommandEventId::NONE; + mbMouseEvent = false; +} + +CommandEvent::CommandEvent( const Point& rMousePos, + CommandEventId nCmd, bool bMEvt, const void* pCmdData ) : + maPos( rMousePos ) +{ + mpData = const_cast<void*>(pCmdData); + mnCommand = nCmd; + mbMouseEvent = bMEvt; +} + +const CommandExtTextInputData* CommandEvent::GetExtTextInputData() const +{ + if ( mnCommand == CommandEventId::ExtTextInput ) + return static_cast<const CommandExtTextInputData*>(mpData); + else + return nullptr; +} + +const CommandWheelData* CommandEvent::GetWheelData() const +{ + if ( mnCommand == CommandEventId::Wheel ) + return static_cast<const CommandWheelData*>(mpData); + else + return nullptr; +} + +const CommandScrollData* CommandEvent::GetAutoScrollData() const +{ + if ( mnCommand == CommandEventId::AutoScroll ) + return static_cast<const CommandScrollData*>(mpData); + else + return nullptr; +} + +const CommandModKeyData* CommandEvent::GetModKeyData() const +{ + if( mnCommand == CommandEventId::ModKeyChange ) + return static_cast<const CommandModKeyData*>(mpData); + else + return nullptr; +} + +const CommandDialogData* CommandEvent::GetDialogData() const +{ + if( mnCommand == CommandEventId::ShowDialog ) + return static_cast<const CommandDialogData*>(mpData); + else + return nullptr; +} + +CommandMediaData* CommandEvent::GetMediaData() const +{ + if( mnCommand == CommandEventId::Media ) + return static_cast<CommandMediaData*>(mpData); + else + return nullptr; +} + +const CommandSelectionChangeData* CommandEvent::GetSelectionChangeData() const +{ + if( mnCommand == CommandEventId::SelectionChange ) + return static_cast<const CommandSelectionChangeData*>(mpData); + else + return nullptr; +} + +const CommandSwipeData* CommandEvent::GetSwipeData() const +{ + if( mnCommand == CommandEventId::Swipe ) + return static_cast<const CommandSwipeData*>(mpData); + else + return nullptr; +} + +const CommandLongPressData* CommandEvent::GetLongPressData() const +{ + if( mnCommand == CommandEventId::LongPress ) + return static_cast<const CommandLongPressData*>(mpData); + else + return nullptr; +} + /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits