On Sat, 22 Dec 2012 10:09:34 -0800 "Enlightenment SVN"
<[email protected]> said:

ummm... this actually first isn't a bug - use xterm. it pastes the newline. so
you've actually broken functionality compared with "the originl terminal
emulator"" (well from my pov xterm is the bar to meet function-wise, and then
improve from there, and here i can't double-click a line of a command and just
middle click past to run it .. newline included).

so compare xterm vs terminology before and after this patch. you'll see what i
mean.

> Log:
> terminology: remove trailing newline when selecting text. Fix #1967
>   
>   Patch by Panagiotis Galatsanos <galatsanosp at gmail com>
> 
> Author:       billiob
> Date:         2012-12-22 10:09:17 -0800 (Sat, 22 Dec 2012)
> New Revision: 81644
> Trac:         http://trac.enlightenment.org/e/changeset/81644
> 
> Modified:
>   trunk/terminology/src/bin/termio.c 
> 
> Modified: trunk/terminology/src/bin/termio.c
> ===================================================================
> --- trunk/terminology/src/bin/termio.c        2012-12-22 16:29:50 UTC (rev
> 81643) +++ trunk/terminology/src/bin/termio.c 2012-12-22 18:09:17 UTC
> (rev 81644) @@ -773,7 +773,16 @@
>                    for (i = 0; i < ev->len; i++)
>                      {
>                         tmp[i] = s[i];
> -                       if (tmp[i] == '\n') tmp[i] = '\r';
> +                       if (tmp[i] == '\n')
> +                         {
> +                            // remove trailing new line
> +                            if (i != (ev->len - 2)) tmp[i] = '\r';
> +                            else
> +                              {
> +                                 tmp[i] = '\0';
> +                                 break;
> +                              }
> +                         }
>                      }
>                    termpty_write(sd->pty, tmp, ev->len - 1);
>                    free(tmp);
> 
> 
> ------------------------------------------------------------------------------
> LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
> Remotely access PCs and mobile devices and provide instant support
> Improve your efficiency, and focus on delivering more value-add services
> Discover what IT Professionals Know. Rescue delivers
> http://p.sf.net/sfu/logmein_12329d2d
> _______________________________________________
> enlightenment-svn mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    [email protected]


------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to