<div dir="ltr"><div><div>Hi Bryce, and thanks a lot for the review !<br><br></div>Of course I have tested it :) ; the steps are the following :<br>Copy some text in weston-terminal, scroll it out (by pressing [Enter] repeatedly e.g.), try to paste.<br></div><div>It does work with the patch only.<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-09-23 2:10 GMT+02:00 Bryce Harrington <span dir="ltr"><<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, Sep 19, 2015 at 01:51:05PM +0200, Manuel Bachmann wrote:<br>
> We could not paste text when its source went outside the<br>
> visible part of the buffer ; this is because we were<br>
> incorrectly assuming that our iterator should start at<br>
> row 0, while it could very well be negative.<br>
><br>
> Signed-off-by: Manuel Bachmann <manuel.bachmann@iot.bzh><br>
<br>
</span>Interesting find.  I assume this patch has been tested to verify it<br>
fixes the problem?<br>
<br>
Reviewed-by: Bryce Harrington <<a href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</a>><br>
<span class=""><br>
> ---<br>
>  clients/terminal.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/clients/terminal.c b/clients/terminal.c<br>
> index c5d5d60..7406f50 100644<br>
> --- a/clients/terminal.c<br>
> +++ b/clients/terminal.c<br>
> @@ -942,7 +942,7 @@ terminal_send_selection(struct terminal *terminal, int fd)<br>
>               close(fd);<br>
>               return;<br>
>       }<br>
> -     for (row = 0; row < terminal->height; row++) {<br>
> +     for (row = terminal->selection_start_row; row < terminal->height; row++) {<br>
>               p_row = terminal_get_row(terminal, row);<br>
>               for (col = 0; col < terminal->width; col++) {<br>
>                       if (p_row[col].ch == 0x200B) /* space glyph */<br>
> --<br>
> 1.8.3.1<br>
><br>
</span>> _______________________________________________<br>
> wayland-devel mailing list<br>
> <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr">Regards,<br><span><div><div dir="ltr"><span><font color="#888888"><div><i><small>Manuel Bachmann, Graphics Engineer<br>
          <a href="http://iot.bzh" target="_blank"> www.iot.bzh</a>
        </small></i></div>
  </font></span><div>

</div></div></div></span><br></div></div>
</div>