[dev] [PATCH] Removing wrapping newlines from selection

2014-08-22 Thread Ben Hendrickson
When getting selected text, lines that were wrapped because of length ought not include the wrapping newline in the selection. This comes up, for example, when copying a bash command that is long enough to wrap from the console and pasting it back into the console. The extra newline breaks it. Si

Re: [dev] [PATCH] Removing wrapping newlines from selection

2014-08-22 Thread Silvan Jegen
Hi On Fri, Aug 22, 2014 at 09:25:04AM -0700, Ben Hendrickson wrote: > When getting selected text, lines that were wrapped because of length > ought not include the wrapping newline in the selection. > > This comes up, for example, when copying a bash command that is long > enough to wrap from the

Re: [dev] [PATCH] Removing wrapping newlines from selection

2014-08-22 Thread Ben Hendrickson
On Fri, Aug 22, 2014 at 12:00 PM, Silvan Jegen wrote: >> + if (term.line[y][i - 1].mode & ATTR_WRAP) > > The preferred style in st.c seems to be the one without a space > after the 'if'. There still are about 18 other places where this > convention is broken however. Good point. What is proto

Re: [dev] [PATCH] Removing wrapping newlines from selection

2014-08-22 Thread Silvan Jegen
On Fri, Aug 22, 2014 at 12:54:55PM -0700, Ben Hendrickson wrote: > On Fri, Aug 22, 2014 at 12:00 PM, Silvan Jegen wrote: > >> + if (term.line[y][i - 1].mode & ATTR_WRAP) > > > > The preferred style in st.c seems to be the one without a space > > after the 'if'. There still are about 18 other p