diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2009-12-01 14:32:29 +0100 |
|---|---|---|
| committer | Eric Anholt <eric@anholt.net> | 2010-01-07 10:25:07 -0800 |
| commit | cf74caaa91667457bba3583b216dfc157d09f39e (patch) | |
| tree | 4415ac2df7bfa8377e5e9132dbe3ab06124eca86 | |
| parent | d8353c737b5b8077f499461b1e6c78f09544d226 (diff) | |
Xv: kill an unnecessary if
We always pass a non-null pointer for crtc_ret, no point to check
for this.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Eric Anholt <eric@anholt.net>
| -rw-r--r-- | src/i830_video.c | 27 |
1 files changed, 13 insertions, 14 deletions
diff --git a/src/i830_video.c b/src/i830_video.c index 59236aeb..aa3a604b 100644 --- a/src/i830_video.c +++ b/src/i830_video.c @@ -1218,26 +1218,25 @@ i830_clip_video_helper(ScrnInfoPtr scrn, Bool ret; RegionRec crtc_region_local; RegionPtr crtc_region = reg; + BoxRec crtc_box; /* * For overlay video, compute the relevant CRTC and * clip video to that */ - if (crtc_ret) { - BoxRec crtc_box; - xf86CrtcPtr crtc = i830_covering_crtc(scrn, dst, - adaptor_priv->desired_crtc, - &crtc_box); - - /* For textured video, we don't actually want to clip at all. */ - if (crtc && !adaptor_priv->textured) { - REGION_INIT(screen, &crtc_region_local, &crtc_box, 1); - crtc_region = &crtc_region_local; - REGION_INTERSECT(screen, crtc_region, crtc_region, - reg); - } - *crtc_ret = crtc; + xf86CrtcPtr crtc = i830_covering_crtc(scrn, dst, + adaptor_priv->desired_crtc, + &crtc_box); + + /* For textured video, we don't actually want to clip at all. */ + if (crtc && !adaptor_priv->textured) { + REGION_INIT(screen, &crtc_region_local, &crtc_box, 1); + crtc_region = &crtc_region_local; + REGION_INTERSECT(screen, crtc_region, crtc_region, + reg); } + *crtc_ret = crtc; + ret = xf86XVClipVideoHelper(dst, xa, xb, ya, yb, crtc_region, width, height); if (crtc_region != reg) |