Re: [PATCH 2/2] Reduce indentation inverting condition

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

 



Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>

On Tue, Aug 02, 2016 at 10:12:21AM +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  server/stream.c | 27 +++++++++++++--------------
>  1 file changed, 13 insertions(+), 14 deletions(-)
> 
> diff --git a/server/stream.c b/server/stream.c
> index e56c419..e8c58fc 100644
> --- a/server/stream.c
> +++ b/server/stream.c
> @@ -250,20 +250,19 @@ static bool is_next_stream_frame(DisplayChannel *display,
>              return FALSE;
>          }
>      } else {
> -        if (rect_contains(&red_drawable->bbox, other_dest)) {
> -            int candidate_area = rect_get_area(&red_drawable->bbox);
> -            int other_area = rect_get_area(other_dest);
> -            /* do not stream drawables that are significantly
> -             * bigger than the original frame */
> -            if (candidate_area > 2 * other_area) {
> -                spice_debug("too big candidate:");
> -                spice_debug("prev box ==>");
> -                rect_debug(other_dest);
> -                spice_debug("new box ==>");
> -                rect_debug(&red_drawable->bbox);
> -                return FALSE;
> -            }
> -        } else {
> +        if (!rect_contains(&red_drawable->bbox, other_dest)) {
> +            return FALSE;
> +        }
> +        int candidate_area = rect_get_area(&red_drawable->bbox);
> +        int other_area = rect_get_area(other_dest);
> +        /* do not stream drawables that are significantly
> +         * bigger than the original frame */
> +        if (candidate_area > 2 * other_area) {
> +            spice_debug("too big candidate:");
> +            spice_debug("prev box ==>");
> +            rect_debug(other_dest);
> +            spice_debug("new box ==>");
> +            rect_debug(&red_drawable->bbox);
>              return FALSE;
>          }
>      }
> -- 
> 2.7.4
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://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]