diff options
author | Christophe Fergeau <cfergeau@redhat.com> | 2011-06-24 14:20:32 +0200 |
---|---|---|
committer | Christophe Fergeau <cfergeau@redhat.com> | 2011-07-22 16:53:56 +0200 |
commit | cacca4dce6be474a2e564eba544af8222228f492 (patch) | |
tree | 709b5f0ac1c267a44a0eb81343fc09e3125be763 | |
parent | 1d808d31f42925ff3dcbc3504bc2c9c49ebd2ef7 (diff) | |
download | spice-cacca4dce6be474a2e564eba544af8222228f492.tar.gz spice-cacca4dce6be474a2e564eba544af8222228f492.tar.xz spice-cacca4dce6be474a2e564eba544af8222228f492.zip |
red_worker: simplify red_rgb_to_24bpp prototype
It takes a lot of arguments, "id" is unused, "frame" and
"frame_size" can be obtained from the "stream" argument, so
can get rid of 3 arguments to make things more readable.
-rw-r--r-- | server/red_worker.c | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/server/red_worker.c b/server/red_worker.c index 817fc411..c17194ee 100644 --- a/server/red_worker.c +++ b/server/red_worker.c @@ -7336,20 +7336,22 @@ static void pixel_rgb16bpp_to_24(uint8_t *src, uint8_t *dest) } static int red_rgb_to_24bpp (RedWorker *worker, const SpiceRect *src, - const SpiceBitmap *image, - uint8_t *frame, size_t frame_stride, - int id, Stream *stream) + const SpiceBitmap *image, Stream *stream) { SpiceChunks *chunks; uint32_t image_stride; uint8_t *frame_row; size_t offset; int i, x, chunk; + uint8_t *frame; + size_t frame_stride; chunks = image->data; offset = 0; chunk = 0; image_stride = image->stride; + frame = mjpeg_encoder_get_frame(stream->mjpeg_encoder); + frame_stride = mjpeg_encoder_get_frame_stride(stream->mjpeg_encoder); const int skip_lines = stream->top_down ? src->top : image->y - (src->bottom - 0); for (i = 0; i < skip_lines; i++) { @@ -7409,8 +7411,6 @@ static inline int red_send_stream_data(DisplayChannel *display_channel, SpiceImage *image; RedChannel *channel; RedWorker* worker; - uint8_t *frame; - size_t frame_stride; int n; ASSERT(stream); @@ -7432,12 +7432,8 @@ static inline int red_send_stream_data(DisplayChannel *display_channel, return TRUE; } - frame = mjpeg_encoder_get_frame(stream->mjpeg_encoder); - frame_stride = mjpeg_encoder_get_frame_stride(stream->mjpeg_encoder); - if (!red_rgb_to_24bpp(worker, &drawable->red_drawable->u.copy.src_area, - &image->u.bitmap, frame, frame_stride, - stream - worker->streams_buf, stream)) { + &image->u.bitmap, stream)) { return FALSE; } |