Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Jean-Marc Lasgouttes
Le 11/11/2014 20:58, Georg Baum a écrit : commit 8f93600d3fa8182ba43973075cf37e7ecb2be8d3 Author: Georg Baum b...@lyx.org Date: Tue Nov 11 07:22:14 2014 +0100 Prevent accidental usage of wrong copy constructor Is this what we use boost:noncopyable for in some places? It would be nice

Re: [LyX/master] Fix overflow of inset over text (see #9224)

2014-11-14 Thread Uwe Stöhr
Am 14.11.2014 um 11:21 schrieb Jean-Marc Lasgouttes: commit 81badf8550f2fd5e614a22261c4885d747a72106 Author: Jean-Marc Lasgouttes lasgout...@lyx.org Date: Fri Nov 14 11:13:38 2014 +0100 + static_castchar_type(row.sel_end), +

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Georg Baum
Jean-Marc Lasgouttes wrote: Le 11/11/2014 20:58, Georg Baum a écrit : commit 8f93600d3fa8182ba43973075cf37e7ecb2be8d3 Author: Georg Baum b...@lyx.org Date: Tue Nov 11 07:22:14 2014 +0100 Prevent accidental usage of wrong copy constructor Is this what we use boost:noncopyable for

[RFC] should we disallow to insert floats, tables, figures and notes when cursor is in math

2014-11-14 Thread Uwe Stöhr
In http://www.lyx.org/trac/ticket/704 John Levon once complained about that we allow to insert floats etc. while the cursor is in math. Currently we still allow this but place the float before the formula. This might not what the user expects (he might have it behind the formula or wonders

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Abdelrazak Younes
On 14/11/2014 18:48, Georg Baum wrote: Jean-Marc Lasgouttes wrote: Le 11/11/2014 20:58, Georg Baum a écrit : commit 8f93600d3fa8182ba43973075cf37e7ecb2be8d3 Author: Georg Baum b...@lyx.org Date: Tue Nov 11 07:22:14 2014 +0100 Prevent accidental usage of wrong copy constructor Is

Re: [LyX/master] Fix recursive math macro crash (bug #9140)

2014-11-14 Thread Georg Baum
Georg Baum wrote: commit 0f2069b8a5967aac1b4f841214294f1fe21d2cad Author: Georg Baum b...@lyx.org Date: Fri Nov 14 21:30:42 2014 +0100 Fix recursive math macro crash (bug #9140) This is different from bug #8999, since in this case a new macro instance is created. You

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Georg Baum
Abdelrazak Younes wrote: Nitpick: private stuff is at the end of the class not at the top. Sure, but the private noncopyable stuff is the only thing I prefer at the top, since it is then near the other contructors. Abdel. PS: I got some nice ideas today about teletext charset handling

Re: [LyX/master] Fix recursive math macro crash (bug #9140)

2014-11-14 Thread Richard Heck
On 11/14/2014 08:34 PM, Georg Baum wrote: Georg Baum wrote: commit 0f2069b8a5967aac1b4f841214294f1fe21d2cad Author: Georg Baum b...@lyx.org Date: Fri Nov 14 21:30:42 2014 +0100 Fix recursive math macro crash (bug #9140) This is different from bug #8999, since in this case a

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Richard Heck
On 11/14/2014 09:02 PM, Jean-Marc Lasgouttes wrote: I agree with you actually. My points were probably: * we should get rid of the few users of boost::noncopyable * ot would be better to use the terse style of comments we use elsewhere. It turns out there are only three places we use it. Now

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Jean-Marc Lasgouttes
Le 11/11/2014 20:58, Georg Baum a écrit : commit 8f93600d3fa8182ba43973075cf37e7ecb2be8d3 Author: Georg Baum Date: Tue Nov 11 07:22:14 2014 +0100 Prevent accidental usage of wrong copy constructor Is this what we use boost:noncopyable for in some places? It would be

Re: [LyX/master] Fix overflow of inset over text (see #9224)

2014-11-14 Thread Uwe Stöhr
Am 14.11.2014 um 11:21 schrieb Jean-Marc Lasgouttes: commit 81badf8550f2fd5e614a22261c4885d747a72106 Author: Jean-Marc Lasgouttes Date: Fri Nov 14 11:13:38 2014 +0100 + static_cast(row.sel_end), +

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Georg Baum
Jean-Marc Lasgouttes wrote: > Le 11/11/2014 20:58, Georg Baum a écrit : >> commit 8f93600d3fa8182ba43973075cf37e7ecb2be8d3 >> Author: Georg Baum >> Date: Tue Nov 11 07:22:14 2014 +0100 >> >> Prevent accidental usage of wrong copy constructor > > Is this what we use

[RFC] should we disallow to insert floats, tables, figures and notes when cursor is in math

2014-11-14 Thread Uwe Stöhr
In http://www.lyx.org/trac/ticket/704 John Levon once complained about that we allow to insert floats etc. while the cursor is in math. Currently we still allow this but place the float before the formula. This might not what the user expects (he might have it behind the formula or wonders

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Abdelrazak Younes
On 14/11/2014 18:48, Georg Baum wrote: Jean-Marc Lasgouttes wrote: Le 11/11/2014 20:58, Georg Baum a écrit : commit 8f93600d3fa8182ba43973075cf37e7ecb2be8d3 Author: Georg Baum Date: Tue Nov 11 07:22:14 2014 +0100 Prevent accidental usage of wrong copy constructor Is

Re: [LyX/master] Fix recursive math macro crash (bug #9140)

2014-11-14 Thread Georg Baum
Georg Baum wrote: > commit 0f2069b8a5967aac1b4f841214294f1fe21d2cad > Author: Georg Baum > Date: Fri Nov 14 21:30:42 2014 +0100 > > Fix recursive math macro crash (bug #9140) > > This is different from bug #8999, since in this case a new macro > instance is

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Georg Baum
Abdelrazak Younes wrote: > Nitpick: private stuff is at the end of the class not at the top. Sure, but the private noncopyable stuff is the only thing I prefer at the top, since it is then near the other contructors. > Abdel. > > PS: I got some nice ideas today about teletext charset handling

Re: [LyX/master] Fix recursive math macro crash (bug #9140)

2014-11-14 Thread Richard Heck
On 11/14/2014 08:34 PM, Georg Baum wrote: Georg Baum wrote: commit 0f2069b8a5967aac1b4f841214294f1fe21d2cad Author: Georg Baum Date: Fri Nov 14 21:30:42 2014 +0100 Fix recursive math macro crash (bug #9140) This is different from bug #8999, since in this case

Re: [LyX/master] Prevent accidental usage of wrong copy constructor

2014-11-14 Thread Richard Heck
On 11/14/2014 09:02 PM, Jean-Marc Lasgouttes wrote: I agree with you actually. My points were probably: * we should get rid of the few users of boost::noncopyable * ot would be better to use the terse style of comments we use elsewhere. It turns out there are only three places we use it. Now