diff options
| author | Chris Sherlock <chris.sherlock79@gmail.com> | 2016-02-10 16:54:47 +1100 |
|---|---|---|
| committer | Chris Sherlock <chris.sherlock79@gmail.com> | 2016-02-10 21:03:34 +1100 |
| commit | 3ff9dd6ff36f21d9bea1851cea05a4ed4228722d (patch) | |
| tree | bc68900860674e3e648b1e34060068106c5bd181 /avmedia | |
| parent | 574482123b0fcd7deed9345fd524e9fc1d8bfb5e (diff) | |
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
Diffstat (limited to 'avmedia')
| -rw-r--r-- | avmedia/source/viewer/mediawindow_impl.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/avmedia/source/viewer/mediawindow_impl.cxx b/avmedia/source/viewer/mediawindow_impl.cxx index dbf34150187b..6206abb73300 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 ); |