author | alwin <alwin> | 2005-04-02 21:55:07 (UTC) |
---|---|---|
committer | alwin <alwin> | 2005-04-02 21:55:07 (UTC) |
commit | a5020219d43b9dd788cc090ae565c28f27dd6a58 (patch) (unidiff) | |
tree | c60f0ae6554817db54e45ea2eb6b1c9896193b3e /noncore/multimedia/opieplayer2/xinevideowidget.cpp | |
parent | b92357d5566b9ee220ff5f4e8eb612b2db32e20b (diff) | |
download | opie-a5020219d43b9dd788cc090ae565c28f27dd6a58.zip opie-a5020219d43b9dd788cc090ae565c28f27dd6a58.tar.gz opie-a5020219d43b9dd788cc090ae565c28f27dd6a58.tar.bz2 |
my last try to get mediaplayer2 more stable.
-> protected the videobuffer with mutex, make a deep copy of it so xine may
destruct its internal buffers as wanted
Diffstat (limited to 'noncore/multimedia/opieplayer2/xinevideowidget.cpp') (more/less context) (show whitespace changes)
-rw-r--r-- | noncore/multimedia/opieplayer2/xinevideowidget.cpp | 97 |
1 files changed, 69 insertions, 28 deletions
diff --git a/noncore/multimedia/opieplayer2/xinevideowidget.cpp b/noncore/multimedia/opieplayer2/xinevideowidget.cpp index 9e86041..211791b 100644 --- a/noncore/multimedia/opieplayer2/xinevideowidget.cpp +++ b/noncore/multimedia/opieplayer2/xinevideowidget.cpp | |||
@@ -1,48 +1,53 @@ | |||
1 | 1 | ||
2 | /* | 2 | /* |
3 | This file is part of the Opie Project | 3 | Â Â Â Â Â Â Â Â This file is part of the Opie Project |
4 | 4 | ||
5 | Copyright (c) 2002 Robert Griebl <sandman@handhelds.org> | 5 | Â Â Â Â Â Â Â Copyright (c) 2002 Max Reiss <harlekin@handhelds.org> |
6 | Copyright (c) 2002 L. Potter <ljp@llornkcor.com> | ||
6 | Copyright (c) 2002 Holger Freyther <zecke@handhelds.org> | 7 | Copyright (c) 2002 Holger Freyther <zecke@handhelds.org> |
7 | =. | 8 | =. |
8 | .=l. | 9 | .=l. |
9 | .>+-= | 10 | Â Â Â Â Â Â .>+-= |
10 | _;:, .> :=|. This program is free software; you can | 11 | Â _;:, Â Â .> Â Â :=|. This program is free software; you can |
11 | .> <`_, > . <= redistribute it and/or modify it under | 12 | .> <`_, Â > Â . Â <= redistribute it and/or modify it under |
12 | :`=1 )Y*s>-.-- : the terms of the GNU General Public | 13 | :`=1 )Y*s>-.-- Â : the terms of the GNU General Public |
13 | .="- .-=="i, .._ License as published by the Free Software | 14 | .="- .-=="i, Â Â .._ License as published by the Free Software |
14 | - . .-<_> .<> Foundation; either version 2 of the License, | 15 | Â - . Â .-<_> Â Â .<> Foundation; either version 2 of the License, |
15 | ._= =} : or (at your option) any later version. | 16 | Â Â Â ._= =} Â Â Â : or (at your option) any later version. |
16 | .%`+i> _;_. | 17 | Â Â .%`+i> Â Â Â _;_. |
17 | .i_,=:_. -<s. This program is distributed in the hope that | 18 | Â Â .i_,=:_. Â Â Â -<s. This program is distributed in the hope that |
18 | + . -:. = it will be useful, but WITHOUT ANY WARRANTY; | 19 | Â Â Â + Â . Â -:. Â Â Â = it will be useful, but WITHOUT ANY WARRANTY; |
19 | : .. .:, . . . without even the implied warranty of | 20 | Â Â : .. Â Â .:, Â Â . . . without even the implied warranty of |
20 | =_ + =;=|` MERCHANTABILITY or FITNESS FOR A | 21 | Â Â =_ Â Â Â Â + Â Â =;=|` MERCHANTABILITY or FITNESS FOR A |
21 | _.=:. : :=>`: PARTICULAR PURPOSE. See the GNU | 22 | Â _.=:. Â Â Â : Â Â :=>`: PARTICULAR PURPOSE. See the GNU |
22 | ..}^=.= = ; Library General Public License for more | 23 | ..}^=.= Â Â Â = Â Â Â ; Library General Public License for more |
23 | ++= -. .` .: details. | 24 | ++= Â -. Â Â .` Â Â .: details. |
24 | : = ...= . :.=- | 25 | Â : Â Â = Â ...= . :.=- |
25 | -. .:....=;==+<; You should have received a copy of the GNU | 26 | Â -. Â .:....=;==+<; You should have received a copy of the GNU |
26 | -_. . . )=. = Library General Public License along with | 27 | Â -_. . . Â )=. Â = Library General Public License along with |
27 | -- :-=` this library; see the file COPYING.LIB. | 28 | Â Â -- Â Â Â Â :-=` this library; see the file COPYING.LIB. |
28 | If not, write to the Free Software Foundation, | 29 | If not, write to the Free Software Foundation, |
29 | Inc., 59 Temple Place - Suite 330, | 30 | Inc., 59 Temple Place - Suite 330, |
30 | Boston, MA 02111-1307, USA. | 31 | Boston, MA 02111-1307, USA. |
31 | 32 | ||
32 | */ | 33 | */ |
33 | 34 | ||
35 | #include "xinevideowidget.h" | ||
36 | #include <opie2/odebug.h> | ||
37 | |||
34 | #include <qimage.h> | 38 | #include <qimage.h> |
35 | #include <qdirectpainter_qws.h> | 39 | #include <qdirectpainter_qws.h> |
36 | #include <qgfx_qws.h> | 40 | #include <qgfx_qws.h> |
37 | #include <qsize.h> | 41 | #include <qsize.h> |
38 | #include <qapplication.h> | 42 | #include <qapplication.h> |
39 | 43 | ||
40 | #include <qpe/resource.h> | 44 | #include <qpe/resource.h> |
41 | 45 | ||
42 | #include "xinevideowidget.h" | 46 | #include <pthread.h> |
47 | |||
43 | 48 | ||
44 | 49 | ||
45 | // 0 deg rot: copy a line from src to dst (use libc memcpy) | 50 | // 0 deg rot: copy a line from src to dst (use libc memcpy) |
46 | 51 | ||
47 | // 180 deg rot: copy a line from src to dst reversed | 52 | // 180 deg rot: copy a line from src to dst reversed |
48 | 53 | ||
@@ -118,23 +123,37 @@ XineVideoWidget::XineVideoWidget ( QWidget* parent, const char* name ) | |||
118 | 123 | ||
119 | m_logo = 0; | 124 | m_logo = 0; |
120 | m_buff = 0; | 125 | m_buff = 0; |
121 | m_bytes_per_line_fb = qt_screen-> linestep ( ); | 126 | m_bytes_per_line_fb = qt_screen-> linestep ( ); |
122 | m_bytes_per_pixel = ( qt_screen->depth() + 7 ) / 8; | 127 | m_bytes_per_pixel = ( qt_screen->depth() + 7 ) / 8; |
123 | m_rotation = 0; | 128 | m_rotation = 0; |
129 | m_lastsize = 0; | ||
124 | } | 130 | } |
125 | 131 | ||
126 | 132 | ||
127 | XineVideoWidget::~XineVideoWidget ( ) | 133 | XineVideoWidget::~XineVideoWidget ( ) |
128 | { | 134 | { |
135 | ThreadUtil::AutoLock a(m_bufmutex); | ||
136 | if (m_buff) { | ||
137 | delete[]m_buff; | ||
138 | m_lastsize=0; | ||
139 | m_buff = 0; | ||
140 | } | ||
141 | if (m_logo) { | ||
129 | delete m_logo; | 142 | delete m_logo; |
130 | } | 143 | } |
144 | } | ||
131 | 145 | ||
132 | void XineVideoWidget::clear ( ) | 146 | void XineVideoWidget::clear ( ) |
133 | { | 147 | { |
148 | ThreadUtil::AutoLock a(m_bufmutex); | ||
149 | if (m_buff) { | ||
150 | delete[]m_buff; | ||
151 | m_lastsize=0; | ||
134 | m_buff = 0; | 152 | m_buff = 0; |
153 | } | ||
135 | repaint ( false ); | 154 | repaint ( false ); |
136 | } | 155 | } |
137 | 156 | ||
138 | QSize XineVideoWidget::videoSize() const | 157 | QSize XineVideoWidget::videoSize() const |
139 | { | 158 | { |
140 | QSize s = size(); | 159 | QSize s = size(); |
@@ -149,12 +168,22 @@ QSize XineVideoWidget::videoSize() const | |||
149 | 168 | ||
150 | return s; | 169 | return s; |
151 | } | 170 | } |
152 | 171 | ||
153 | void XineVideoWidget::paintEvent ( QPaintEvent * ) | 172 | void XineVideoWidget::paintEvent ( QPaintEvent * ) |
154 | { | 173 | { |
174 | ThreadUtil::AutoLock a(m_bufmutex); | ||
175 | QPainter p ( this ); | ||
176 | p. fillRect ( rect (), black ); | ||
177 | if (m_logo) | ||
178 | p. drawImage ( 0, 0, *m_logo ); | ||
179 | } | ||
180 | |||
181 | void XineVideoWidget::paintEvent2 ( QPaintEvent * ) | ||
182 | { | ||
183 | ThreadUtil::AutoLock a(m_bufmutex); | ||
155 | if ( m_buff == 0 ) { | 184 | if ( m_buff == 0 ) { |
156 | QPainter p ( this ); | 185 | QPainter p ( this ); |
157 | p. fillRect ( rect ( ), black ); | 186 | p. fillRect ( rect ( ), black ); |
158 | if ( m_logo ) | 187 | if ( m_logo ) |
159 | p. drawImage ( 0, 0, *m_logo ); | 188 | p. drawImage ( 0, 0, *m_logo ); |
160 | } | 189 | } |
@@ -269,31 +298,43 @@ void XineVideoWidget::setLogo ( QImage* logo ) | |||
269 | delete m_logo; | 298 | delete m_logo; |
270 | m_logo = logo; | 299 | m_logo = logo; |
271 | } | 300 | } |
272 | 301 | ||
273 | void XineVideoWidget::setVideoFrame ( uchar* img, int w, int h, int bpl ) | 302 | void XineVideoWidget::setVideoFrame ( uchar* img, int w, int h, int bpl ) |
274 | { | 303 | { |
304 | // mutex area for AutoLock | ||
305 | { | ||
306 | if (m_bufmutex.isLocked()) { | ||
307 | // no time to wait - drop frame | ||
308 | return; | ||
309 | } | ||
310 | ThreadUtil::AutoLock a(m_bufmutex); | ||
275 | bool rot90 = (( -m_rotation ) & 1 ); | 311 | bool rot90 = (( -m_rotation ) & 1 ); |
312 | int l = h*m_bytes_per_pixel*w; | ||
313 | if (l>m_lastsize) { | ||
314 | if (m_buff) { | ||
315 | delete[]m_buff; | ||
316 | } | ||
317 | m_buff = new uchar[l]; | ||
318 | m_lastsize=l; | ||
319 | } | ||
276 | 320 | ||
277 | if ( rot90 ) { // if the rotation is 90 or 270 we have to swap width / height | 321 | if ( rot90 ) { // if the rotation is 90 or 270 we have to swap width / height |
278 | int d = w; | 322 | int d = w; |
279 | w = h; | 323 | w = h; |
280 | h = d; | 324 | h = d; |
281 | } | 325 | } |
282 | 326 | ||
283 | m_lastframe = m_thisframe; | 327 | m_lastframe = m_thisframe; |
284 | m_thisframe. setRect (( width ( ) - w ) / 2, ( height ( ) - h ) / 2, w , h ); | 328 | m_thisframe. setRect (( width ( ) - w ) / 2, ( height ( ) - h ) / 2, w , h ); |
285 | 329 | ||
286 | m_buff = img; | 330 | memcpy(m_buff,img,m_lastsize); |
287 | m_bytes_per_line_frame = bpl; | 331 | m_bytes_per_line_frame = bpl; |
332 | } // Release Mutex | ||
288 | 333 | ||
289 | // only repaint the area that *really* needs to be repainted | 334 | paintEvent2(0); |
290 | |||
291 | repaint ((( m_thisframe & m_lastframe ) != m_lastframe ) ? m_lastframe : m_thisframe, false ); | ||
292 | // ensure that we always have a valid frame! | ||
293 | m_buff = 0; | ||
294 | } | 335 | } |
295 | 336 | ||
296 | void XineVideoWidget::resizeEvent ( QResizeEvent * ) | 337 | void XineVideoWidget::resizeEvent ( QResizeEvent * ) |
297 | { | 338 | { |
298 | emit videoResized( videoSize() ); | 339 | emit videoResized( videoSize() ); |
299 | } | 340 | } |