Re: [PATCH 2.6.8-rc2] intel8x0.c to include CK804 audio support

Lee Revell
2004-07-27T22:05:12Z

On Tue, 2004-07-27 at 02:43, Denis Vlasenko wrote:
> On Tuesday 27 July 2004 07:57, Paul Jackson wrote:
> > > For now the only fix for people using an X environment ... insert file
> > > ...
> >
> > Another fix is to copy from a non-brain damaged window, such as a gui
> > text editor window (nedit, for example).
> >
> > I'm guessing that the tabs are lost in the cut or copy operation, not in
> > the paste operation.
>
> because 'cut' in a xterm cannot know that those eight spaces
> once were a tab. xterm is probably storing screen as a char+attr
> two-dimensional array. There are no tabs, only spaces.
>
> > But file insertion is, in general, a sufficiently winning choice that I
> > think it's better just to get in the habit of always inserting patches
> > that way, at least on email clients that support it.
>
> I do it all the time.

The easiest suggestion I have seen so far is:

diff foo bar | xclip

Much easier than 'insert file', works perfectly. This is not installed
on Debian by default for some reason.

Might be a useful FAQ addition.

Lee