Re: [PATCH 5/6] worker: remove redundant check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2015-09-29 at 11:56 +0100, Frediano Ziglio wrote:
> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>
> 
> The check is done in draw()


I'm not sure whether this patch can be pulled forward easily. At a
minimum, the commit log references a function that doesn't exist in the
current branch ("draw()"). I spent a little bit of time trying to figure
out whether this patch was still valid, but decided that there was very
little advantage to bringing this patch forward to master, so I decided
not to spend more time on it. I'd recommend just leaving this for now.


> ---
>  server/red_worker.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/server/red_worker.c b/server/red_worker.c
> index 38a41a2..3ad6fc0 100644
> --- a/server/red_worker.c
> +++ b/server/red_worker.c
> @@ -5064,10 +5064,6 @@ static int red_process_commands(RedWorker *worker, uint32_t max_pipe_size, int *
>                                     &update, ext_cmd.cmd.data)) {
>                  break;
>              }
> -            if (!validate_surface(worker, update.surface_id)) {
> -                rendering_incorrect("QXL_CMD_UPDATE");
> -                break;
> -            }
>              red_update_area(worker, &update.area, update.surface_id);
>              worker->qxl->st->qif->notify_update(worker->qxl, update.update_id);
>              release_info_ext.group_id = ext_cmd.group_id;


_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]