Re: [patch] bug 4147: Multiple cells paste as one in tables

Andre Poenitz wrote: On Thu, Nov 01, 2007 at 07:37:26PM +0100, Abdelrazak Younes wrote: Abdelrazak Younes wrote: Andre Poenitz wrote: On Wed, Oct 31, 2007 at 06:25:31PM +0100, Jean-Marc Lasgouttes wrote: Abdelrazak Younes <[EMAIL PROTECTED]> writes: If somebody use the docstream with latin8

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 08, 2007 at 11:28:32PM +0100, Andre Poenitz wrote: > On Thu, Nov 08, 2007 at 11:07:29PM +0100, Enrico Forestieri wrote: > > it would be cumbersome changing each instance of something like > > > >ws.os() << ' '; > > > > or > > > >ws.os() << '\n'; > > > > in tha

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 08, 2007 at 11:07:29PM +0100, Enrico Forestieri wrote: > it would be cumbersome changing each instance of something like > >ws.os() << ' '; > > or > >ws.os() << '\n'; > > in that way. And someone will forgot that and will use one of the > above. But we'd get

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 08, 2007 at 10:59:40PM +0100, Andre Poenitz wrote: > On Thu, Nov 01, 2007 at 07:37:26PM +0100, Abdelrazak Younes wrote: > > Abdelrazak Younes wrote: > >> Andre Poenitz wrote: > >>> On Wed, Oct 31, 2007 at 06:25:31PM +0100, Jean-Marc Lasgouttes wrote: > Abdelrazak Younes <[EMAIL PRO

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 07:37:26PM +0100, Abdelrazak Younes wrote: > Abdelrazak Younes wrote: >> Andre Poenitz wrote: >>> On Wed, Oct 31, 2007 at 06:25:31PM +0100, Jean-Marc Lasgouttes wrote: Abdelrazak Younes <[EMAIL PROTECTED]> writes: > If somebody use the docstream with latin8 cha

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 08:57:40PM +0100, Abdelrazak Younes wrote: > Is that Myshkin? In latex, it would be My\v{s}kin, but my ignorance didn't let me use the correct transliteration ;-) Now let's stop here and go forward. -- Enrico

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 07:37:26PM +0100, Abdelrazak Younes wrote: > Abdelrazak Younes wrote: >> Andre Poenitz wrote: >>> On Wed, Oct 31, 2007 at 06:25:31PM +0100, Jean-Marc Lasgouttes wrote: Abdelrazak Younes <[EMAIL PROTECTED]> writes: > If somebody use the docstream with latin8 cha

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Thu, Nov 01, 2007 at 07:30:50PM +0100, Abdelrazak Younes wrote: Enrico Forestieri wrote: On Thu, Nov 01, 2007 at 07:11:05PM +0100, Juergen Spitzmueller wrote: Enrico Forestieri wrote: If you think so, then a combination of the two patches is ok. You mean what A

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Thu, Nov 01, 2007 at 07:24:04PM +0100, Abdelrazak Younes wrote: I must say I am disappointed by your attitude Enrico, I just don't understand why you are so sarcastic. Maybe I am just mean. Please accept my apologies. And accept mine. Abdel.

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 07:30:50PM +0100, Abdelrazak Younes wrote: > Enrico Forestieri wrote: > > On Thu, Nov 01, 2007 at 07:11:05PM +0100, Juergen Spitzmueller wrote: > > > >> Enrico Forestieri wrote: > >> > >>> If you think so, then a combination of the two patches is ok. > >> You mean what Abd

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 07:24:04PM +0100, Abdelrazak Younes wrote: > I must say I am disappointed by your attitude Enrico, I just > don't understand why you are so sarcastic. Maybe I am just mean. Please accept my apologies. -- Enrico

Re: [patch] bug 4147: Multiple cells paste as one in tables

Abdelrazak Younes wrote: > I am just a C++ ignorant but I am reasonably confident my change will > work on all platforms. But your change to branch might not be enough... After Enrico's explanations, I'm just about backporting your fix. And please stop that ignorance discourse ;-) Jürgen

Re: [patch] bug 4147: Multiple cells paste as one in tables

Abdelrazak Younes wrote: Andre Poenitz wrote: On Wed, Oct 31, 2007 at 06:25:31PM +0100, Jean-Marc Lasgouttes wrote: Abdelrazak Younes <[EMAIL PROTECTED]> writes: If somebody use the docstream with latin8 character we will have bugs for sure. How could we forbid 8bit chars to operator

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Thu, Nov 01, 2007 at 07:11:05PM +0100, Juergen Spitzmueller wrote: Enrico Forestieri wrote: If you think so, then a combination of the two patches is ok. You mean what Abdel committed to trunk? Yes, that solves this bug. Now waiting for the next bug associated t

Re: [patch] bug 4147: Multiple cells paste as one in tables

Juergen Spitzmueller wrote: Abdelrazak Younes wrote: I apply a combination of your two patch Juergen. Enrico might think this is due to ignorance about C++ but this is just ignorance about a bug in the odocstream. For the time being, I just committed my second patch to branch. If Enrico confi

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Thu, Nov 01, 2007 at 06:05:04PM +0100, Abdelrazak Younes wrote: Juergen Spitzmueller wrote: Enrico Forestieri wrote: It also doesn't work on Windows and FreeBSD, for example. I think it would be embarrassing explaining it, as this would imply recognizing a basic i

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 07:11:05PM +0100, Juergen Spitzmueller wrote: > Enrico Forestieri wrote: > > > If you think so, then a combination of the two patches is ok. > > You mean what Abdel committed to trunk? Yes, that solves this bug. Now waiting for the next bug associated to unsigned chars a

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Thu, Nov 01, 2007 at 05:20:17PM +0100, Abdelrazak Younes wrote: Enrico Forestieri wrote: On Thu, Nov 01, 2007 at 03:54:06PM +0100, Abdelrazak Younes wrote: Juergen Spitzmueller wrote: Abdelrazak Younes wrote: As for adding the possibility to output an unsigned

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 06:05:04PM +0100, Abdelrazak Younes wrote: > Juergen Spitzmueller wrote: > > Enrico Forestieri wrote: > > > >> It also doesn't work on Windows and FreeBSD, for example. I think it > >> would be embarrassing explaining it, as this would imply recognizing a > >> basic ignora

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 05:20:17PM +0100, Abdelrazak Younes wrote: > Enrico Forestieri wrote: > > On Thu, Nov 01, 2007 at 03:54:06PM +0100, Abdelrazak Younes wrote: > > > >> Juergen Spitzmueller wrote: > >>> Abdelrazak Younes wrote: > >>> > As for adding the possibility to output an unsigned

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: > If you think so, then a combination of the two patches is ok. You mean what Abdel committed to trunk? Jürgen

Re: [patch] bug 4147: Multiple cells paste as one in tables

Abdelrazak Younes wrote: > I apply a combination of your two patch Juergen. Enrico might think this > is due to ignorance about C++ but this is just ignorance about a bug in > the odocstream. For the time being, I just committed my second patch to branch. If Enrico confirms your char->char_type c

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 05:55:49PM +0100, Juergen Spitzmueller wrote: > Enrico Forestieri wrote: > > > It also doesn't work on Windows and FreeBSD, for example. I think it > > would be embarrassing explaining it, as this would imply recognizing a > > basic ignorance about C++, and this should not

Re: [patch] bug 4147: Multiple cells paste as one in tables

Andre Poenitz wrote: On Wed, Oct 31, 2007 at 06:25:31PM +0100, Jean-Marc Lasgouttes wrote: Abdelrazak Younes <[EMAIL PROTECTED]> writes: If somebody use the docstream with latin8 character we will have bugs for sure. How could we forbid 8bit chars to operator

Re: [patch] bug 4147: Multiple cells paste as one in tables

Juergen Spitzmueller wrote: Enrico Forestieri wrote: It also doesn't work on Windows and FreeBSD, for example. I think it would be embarrassing explaining it, as this would imply recognizing a basic ignorance about C++, and this should not be the case. You refer to my first or my second patch

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: > It also doesn't work on Windows and FreeBSD, for example. I think it > would be embarrassing explaining it, as this would imply recognizing a > basic ignorance about C++, and this should not be the case. You refer to my first or my second patch? If to the latter, please

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Thu, Nov 01, 2007 at 03:54:06PM +0100, Abdelrazak Younes wrote: Juergen Spitzmueller wrote: Abdelrazak Younes wrote: As for adding the possibility to output an unsigned char to the docstream I think this buy us absolutely nothing and that it is not even worth supp

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Thu, Nov 01, 2007 at 03:54:06PM +0100, Abdelrazak Younes wrote: > Juergen Spitzmueller wrote: > > Abdelrazak Younes wrote: > > > >> As for adding the possibility to output an unsigned char to the > >> docstream I think this buy us absolutely nothing and that it is not even > >> worth supportin

Re: [patch] bug 4147: Multiple cells paste as one in tables

Juergen Spitzmueller wrote: Abdelrazak Younes wrote: As for adding the possibility to output an unsigned char to the docstream I think this buy us absolutely nothing and that it is not even worth supporting. How about the attached patch? That should do. But I personally prefer you first pat

Re: [patch] bug 4147: Multiple cells paste as one in tables

Abdelrazak Younes wrote: > As for adding the possibility to output an unsigned char to the > docstream I think this buy us absolutely nothing and that it is not even > worth supporting. How about the attached patch? Jürgen Index: src/insets/InsetTabular.cpp ==

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Wed, Oct 31, 2007 at 06:14:35PM +0100, Abdelrazak Younes wrote: Jean-Marc Lasgouttes wrote: Enrico Forestieri <[EMAIL PROTECTED]> writes: Every character you output to a docstream must be ucs4 encoded. If its code point can be contained in an unsigned char, fine, o

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 06:25:31PM +0100, Jean-Marc Lasgouttes wrote: > Abdelrazak Younes <[EMAIL PROTECTED]> writes: > > > If somebody use the docstream with latin8 character we will have bugs > > for sure. > > How could we forbid 8bit chars to operator< private version? Sort of. Declare but n

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 05:32:21PM +0100, Jean-Marc Lasgouttes wrote: > Enrico Forestieri <[EMAIL PROTECTED]> writes: > > > Every character you output to a docstream must be ucs4 encoded. > > If its code point can be contained in an unsigned char, fine, > > otherwise you need a larger type. > > H

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 06:25:31PM +0100, Jean-Marc Lasgouttes wrote: > Abdelrazak Younes <[EMAIL PROTECTED]> writes: > > > If somebody use the docstream with latin8 character we will have bugs > > for sure. > > How could we forbid 8bit chars to operator< private version? This is going to becom

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 06:14:35PM +0100, Abdelrazak Younes wrote: > Jean-Marc Lasgouttes wrote: > > Enrico Forestieri <[EMAIL PROTECTED]> writes: > > > >> Every character you output to a docstream must be ucs4 encoded. > >> If its code point can be contained in an unsigned char, fine, > >> otherw

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 06:18:57PM +0100, Abdelrazak Younes wrote: > Enrico Forestieri wrote: > > On Wed, Oct 31, 2007 at 03:41:50PM +0100, Abdelrazak Younes wrote: > >> Jean-Marc Lasgouttes wrote: > >>> Enrico Forestieri <[EMAIL PROTECTED]> writes: > >>> > Not really. UCS-4 code points from 0

Re: [patch] bug 4147: Multiple cells paste as one in tables

Abdelrazak Younes <[EMAIL PROTECTED]> writes: > If somebody use the docstream with latin8 character we will have bugs > for sure. How could we forbid 8bit chars to operator<

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Wed, Oct 31, 2007 at 03:41:50PM +0100, Abdelrazak Younes wrote: Jean-Marc Lasgouttes wrote: Enrico Forestieri <[EMAIL PROTECTED]> writes: Not really. UCS-4 code points from 0x to 0x00ff exactly correspond to latin-1 code points. So unsigned char c = 0xb5;

Re: [patch] bug 4147: Multiple cells paste as one in tables

Jean-Marc Lasgouttes wrote: Enrico Forestieri <[EMAIL PROTECTED]> writes: Every character you output to a docstream must be ucs4 encoded. If its code point can be contained in an unsigned char, fine, otherwise you need a larger type. Hmm. I think this is a recipe for hard to find bugs. That

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 03:41:50PM +0100, Abdelrazak Younes wrote: > Jean-Marc Lasgouttes wrote: > > Enrico Forestieri <[EMAIL PROTECTED]> writes: > > > >> Not really. UCS-4 code points from 0x to 0x00ff exactly correspond > >> to latin-1 code points. So > >> > >>unsigned char c = 0xb5; >

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri <[EMAIL PROTECTED]> writes: > Every character you output to a docstream must be ucs4 encoded. > If its code point can be contained in an unsigned char, fine, > otherwise you need a larger type. Hmm. I think this is a recipe for hard to find bugs. JMarc

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 04:20:56PM +0100, Jean-Marc Lasgouttes wrote: > Enrico Forestieri <[EMAIL PROTECTED]> writes: > > > I don't understand either points. Why > > > > char_type c = 0xb5; > > os.put(c); > > > > would be ok, but > > > > unsigned char c = 0xb5; > > os.put(c); > > >

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri <[EMAIL PROTECTED]> writes: > I don't understand either points. Why > > char_type c = 0xb5; > os.put(c); > > would be ok, but > > unsigned char c = 0xb5; > os.put(c); > > would not? The first one is a wide char, so it is obviously unicode-encoded. The second one

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 03:13:56PM +0100, Jean-Marc Lasgouttes wrote: > Enrico Forestieri <[EMAIL PROTECTED]> writes: > > > Not really. UCS-4 code points from 0x to 0x00ff exactly correspond > > to latin-1 code points. So > > > >unsigned char c = 0xb5; > >os.put(c); > > > > gives me a

Re: [patch] bug 4147: Multiple cells paste as one in tables

Jean-Marc Lasgouttes wrote: Enrico Forestieri <[EMAIL PROTECTED]> writes: Not really. UCS-4 code points from 0x to 0x00ff exactly correspond to latin-1 code points. So unsigned char c = 0xb5; os.put(c); gives me a 'µ' character, and the assertion above is wrong. Maybe you have utf8

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri <[EMAIL PROTECTED]> writes: > Not really. UCS-4 code points from 0x to 0x00ff exactly correspond > to latin-1 code points. So > >unsigned char c = 0xb5; >os.put(c); > > gives me a 'µ' character, and the assertion above is wrong. > Maybe you have utf8 in mind. We shou

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 01:29:51PM +0100, Abdelrazak Younes wrote: > Enrico Forestieri wrote: > > On Wed, Oct 31, 2007 at 08:22:44AM +0100, Abdelrazak Younes wrote: > >> Enrico Forestieri wrote: > >>> On Tue, Oct 30, 2007 at 06:26:45PM +0100, Jürgen Spitzmüller wrote: > http://bugzilla.lyx.org

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Wed, Oct 31, 2007 at 08:22:44AM +0100, Abdelrazak Younes wrote: Enrico Forestieri wrote: On Tue, Oct 30, 2007 at 06:26:45PM +0100, Jürgen Spitzmüller wrote: http://bugzilla.lyx.org/show_bug.cgi?id=4147 The bug is the result of an encoding failure (use of char inste

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Wed, Oct 31, 2007 at 08:22:44AM +0100, Abdelrazak Younes wrote: > Enrico Forestieri wrote: > > On Tue, Oct 30, 2007 at 06:26:45PM +0100, Jürgen Spitzmüller wrote: > >> http://bugzilla.lyx.org/show_bug.cgi?id=4147 > >> > >> The bug is the result of an encoding failure (use of char instead of > >

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: > This is not going to work on systems where sizeof(wchar_t) == 2. I see. > I have attached a better fix to bug 4147 on bugzilla. AFAICS, it looks good. Anyway, try to get a feedback from Georg or anyone else who is familiar with this stuff, and commit afterwards. Jürg

Re: [patch] bug 4147: Multiple cells paste as one in tables

Enrico Forestieri wrote: On Tue, Oct 30, 2007 at 06:26:45PM +0100, Jürgen Spitzmüller wrote: http://bugzilla.lyx.org/show_bug.cgi?id=4147 The bug is the result of an encoding failure (use of char instead of char_type) that results in an unwanted conversion of a tabulator ('\t') to the digit '

Re: [patch] bug 4147: Multiple cells paste as one in tables

On Tue, Oct 30, 2007 at 06:26:45PM +0100, Jürgen Spitzmüller wrote: > http://bugzilla.lyx.org/show_bug.cgi?id=4147 > > The bug is the result of an encoding failure (use of char instead of > char_type) that results in an unwanted conversion of a tabulator ('\t') to > the digit '9'. > > I'll comm

[patch] bug 4147: Multiple cells paste as one in tables

http://bugzilla.lyx.org/show_bug.cgi?id=4147 The bug is the result of an encoding failure (use of char instead of char_type) that results in an unwanted conversion of a tabulator ('\t') to the digit '9'. I'll commit to branch and trunk on Friday if I get no objections. Jürgen Index: src/insets