Re: compiler warnings in trunk

2011-04-10 Thread Julien Rioux
On 10/04/2011 11:12 AM, Andre Poenitz wrote: On Sun, Apr 10, 2011 at 10:31:47AM -0400, Richard Heck wrote: On 04/09/2011 06:46 PM, Julien Rioux wrote: I always saw those two warnings (paraphrased): Lexer.cpp:197 may be used uninitialized void Lexer::Pimpl::verifyTable() { // [...]

Re: compiler warnings in trunk

2011-04-10 Thread Andre Poenitz
On Sun, Apr 10, 2011 at 10:31:47AM -0400, Richard Heck wrote: > On 04/09/2011 06:46 PM, Julien Rioux wrote: > >I always saw those two warnings (paraphrased): > >Lexer.cpp:197 may be used uninitialized void Lexer::Pimpl::verifyTable() { // [...] if (table <- this line?

Re: compiler warnings in trunk

2011-04-10 Thread Richard Heck
On 04/09/2011 06:46 PM, Julien Rioux wrote: I always saw those two warnings (paraphrased): Lexer.cpp:197 may be used uninitialized Server.cpp:1018 ignoring return value of write(...) The Lexer one has been there forever. I'm not sure why we get it, and I'm certainly not sure why removing the a

Re: Compiler Warnings

2010-06-09 Thread Jean-Marc LASGOUTTES
Stephan Witt writes: > Apropos... > > What about this? Looks good. JMarc

Re: Compiler warnings (session.h)

2006-10-28 Thread Bo Peng
I think this one is for you: In file included from /home/software/lyx-trunk/src/bufferlist.C:20: /home/software/lyx-trunk/src/session.h:40: warning: `class lyx::SessionSection' has virtual functions but non-virtual destructor OK. I will give them an empty one. Cheers, Bo

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Thu, Jul 17, 2003 at 08:56:23AM +0200, Lars Gullik Bjønnes wrote: > Andre Poenitz <[EMAIL PROTECTED]> writes: > > | On Thu, Jul 17, 2003 at 02:47:33AM +0200, Lars Gullik Bjønnes wrote: > | > | No 'å' in TeX after booting. Face it. You are forgotten in troll land. > | > > | > No go explain the

Re: Compiler warnings (harmless)

2003-07-16 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Thu, Jul 17, 2003 at 02:47:33AM +0200, Lars Gullik Bjønnes wrote: | > | No 'å' in TeX after booting. Face it. You are forgotten in troll land. | > | > No go explain the '^?' to me. | | ASCII 127 in the source there... | | Ok, in this case we might

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Thu, Jul 17, 2003 at 02:47:33AM +0200, Lars Gullik Bjønnes wrote: > | No 'å' in TeX after booting. Face it. You are forgotten in troll land. > > No go explain the '^?' to me. ASCII 127 in the source there... Ok, in this case we might use the 127 literally. Andre' -- Those who desire to giv

Re: Compiler warnings (harmless)

2003-07-16 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Wed, Jul 16, 2003 at 05:37:28PM +0200, Lars Gullik Bjønnes wrote: | > | Actually, looking at the 'fix', if this stops the wrning then it is | > | indeed brain dead: | > | | > | - theCatcode['\\'] = catEscape; | > | + theCatcode[int('\\')

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 05:21:56PM +0100, Angus Leeming wrote: > >> > >> I'm going to try and book a flight tonight. If it's not too > >> expensive this late then I'll see you in Chemnitz. > > > > RyanAir Stansted <-> Altenburg 30 GPB (24-29 July) > > > > > http://www.bookryanair.com/skyl

Re: Compiler warnings (harmless)

2003-07-16 Thread Angus Leeming
Andre Poenitz wrote: > On Wed, Jul 16, 2003 at 05:00:43PM +0100, Angus Leeming wrote: >> > Hello Angus. What about your plans for next week btw? >> >> I'm going to try and book a flight tonight. If it's not too >> expensive this late then I'll see you in Chemnitz. > > RyanAir Stansted <-> A

Re: Compiler warnings (harmless)

2003-07-16 Thread John Levon
On Wed, Jul 16, 2003 at 05:28:21PM +0200, Andre Poenitz wrote: > > http://gcc.gnu.org/ml/gcc/2003-03/msg00822.html > > First answer is wrong for C++, 'c' has type 'char'... I know, as I pointed out ... > Come on, leave the cruft in, and do something serious. Testing the > noupdate patch for ins

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 05:00:43PM +0100, Angus Leeming wrote: > > Hello Angus. What about your plans for next week btw? > > I'm going to try and book a flight tonight. If it's not too > expensive this late then I'll see you in Chemnitz. RyanAir Stansted <-> Altenburg 30 GPB (24-29 July)

Re: Compiler warnings (harmless)

2003-07-16 Thread Angus Leeming
Andre Poenitz wrote: > On Wed, Jul 16, 2003 at 04:42:50PM +0100, Angus Leeming wrote: >> > Then, for $DEITY's sake, change it the way you like. >> >> Why so scratchy? > > Because we've already spend much more time on that issue than it is > worth while more serious stuff lies dormant. > >> I wa

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 04:42:50PM +0100, Angus Leeming wrote: > > Then, for $DEITY's sake, change it the way you like. > > Why so scratchy? Because we've already spend much more time on that issue than it is worth while more serious stuff lies dormant. > I was only saying hello. Hello Angus.

Re: Compiler warnings (harmless)

2003-07-16 Thread Angus Leeming
Andre Poenitz wrote: > On Wed, Jul 16, 2003 at 04:21:54PM +0100, Angus Leeming wrote: >> Andre Poenitz wrote: >> >> Elsewhere, wouldn't >> >> void MathCursor::selPaste(lyx::size_type n) >> >> be better than using size_t here? >> > >> > Why? vector<>::size_type is required to be size_t, n

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 05:42:37PM +0200, Andre' Poenitz wrote: > This is the _initial_ set of TeX's cat codes. They have not changed > during the last twenty years and it is Don Knuth's emphatically > expressed will that they won't change ever. > > No 'å' in TeX after booting. Face it. You are f

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 05:37:28PM +0200, Lars Gullik Bjønnes wrote: > | Actually, looking at the 'fix', if this stops the wrning then it is > | indeed brain dead: > | > | - theCatcode['\\'] = catEscape; > | + theCatcode[int('\\')] = catEscape; > | > > except when somebody tries: >

Re: Compiler warnings (harmless)

2003-07-16 Thread Lars Gullik Bjønnes
Angus Leeming <[EMAIL PROTECTED]> writes: | Andre Poenitz wrote: | | > sigh | > | > This is completely legal code. | > | > I've committed a 'fix' nevertheless, just to shut up the compiler. | | Incidentally, what is it really trying to do? Warn us that the | subscript is signed and so might b

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 04:21:54PM +0100, Angus Leeming wrote: > Andre Poenitz wrote: > >> Elsewhere, wouldn't > >> void MathCursor::selPaste(lyx::size_type n) > >> be better than using size_t here? > > > > Why? vector<>::size_type is required to be size_t, not > > lyx::size_type... > >

Re: Compiler warnings (harmless)

2003-07-16 Thread Angus Leeming
Andre Poenitz wrote: >> Elsewhere, wouldn't >> void MathCursor::selPaste(lyx::size_type n) >> be better than using size_t here? > > Why? vector<>::size_type is required to be size_t, not > lyx::size_type... But you are hiding the fact that you store the stuff in a vector yet should stil

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 04:17:36PM +0100, John Levon wrote: > On Wed, Jul 16, 2003 at 05:07:45PM +0200, Andre Poenitz wrote: > > > In either case, we use only positive 7 bit values there. > > Well, assuming ascii ... > > Even so, I did ask about this, to no avail : > > http://gcc.gnu.org/ml/gcc

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 04:14:18PM +0100, Jose' Matos wrote: > On Wednesday 16 July 2003 16:07, Andre Poenitz wrote: > > > > In either case, we use only positive 7 bit values there. So the > > message _is_ braindead. > > Oh, do you mean that ISO C++ requires that '\\' is always less than > 12

Re: Compiler warnings (harmless)

2003-07-16 Thread Angus Leeming
John Levon wrote: > On Wed, Jul 16, 2003 at 03:36:42PM +0100, Angus Leeming wrote: > >> Incidentally, what is it really trying to do? Warn us that the >> subscript is signed and so might be negative? > > No, it's warning that "char" may be signed /or/ unsigned, and that's > platform dependent. >

Re: Compiler warnings (harmless)

2003-07-16 Thread John Levon
On Wed, Jul 16, 2003 at 05:07:45PM +0200, Andre Poenitz wrote: > In either case, we use only positive 7 bit values there. Well, assuming ascii ... Even so, I did ask about this, to no avail : http://gcc.gnu.org/ml/gcc/2003-03/msg00822.html I suspect because testing for positiveness would make

Re: Compiler warnings (harmless)

2003-07-16 Thread Jose' Matos
On Wednesday 16 July 2003 16:07, Andre Poenitz wrote: > > In either case, we use only positive 7 bit values there. > So the message _is_ braindead. Oh, do you mean that ISO C++ requires that '\\' is always less than 127? Ok, I think I stepped that part. ;-) I know that weird IBM charset, EBC

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 04:05:03PM +0100, John Levon wrote: > On Wed, Jul 16, 2003 at 03:36:42PM +0100, Angus Leeming wrote: > > > Incidentally, what is it really trying to do? Warn us that the > > subscript is signed and so might be negative? > > No, it's warning that "char" may be signed /or/

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 03:36:42PM +0100, Angus Leeming wrote: > Andre Poenitz wrote: > > > sigh > > > > This is completely legal code. > > > > I've committed a 'fix' nevertheless, just to shut up the compiler. > > Incidentally, what is it really trying to do? Warn us that the > subscript is s

Re: Compiler warnings (harmless)

2003-07-16 Thread John Levon
On Wed, Jul 16, 2003 at 03:36:42PM +0100, Angus Leeming wrote: > Incidentally, what is it really trying to do? Warn us that the > subscript is signed and so might be negative? No, it's warning that "char" may be signed /or/ unsigned, and that's platform dependent. We've been through this one be

Re: Compiler warnings (harmless)

2003-07-16 Thread Angus Leeming
Andre Poenitz wrote: > sigh > > This is completely legal code. > > I've committed a 'fix' nevertheless, just to shut up the compiler. Incidentally, what is it really trying to do? Warn us that the subscript is signed and so might be negative? Actually, looking at the 'fix', if this stops the

Re: Compiler warnings (harmless)

2003-07-16 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 04:18:43PM +0200, Michael Schmitt wrote: > math_parser.C:154: warning: array subscript has type `char' > math_parser.C:155: warning: array subscript has type `char' > math_parser.C:156: warning: array subscript has type `char' > math_parser.C:157: warning: array subscrip

Re: Compiler warnings

2003-01-10 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | > "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: | | Lars> but abs on a unsigned type or a type converted from unsiged is | Lars> an oxymoron, isn't it? | | Lars> but I guess you are planning a | | Lars> int(uint) - int t

Re: Compiler warnings

2003-01-10 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | @@ -33,6 +34,7 @@ | #include "language.h" | #include "debug.h" | | +using std::abs; | using std::endl; | using std::max; | using std::min; | @@ -213,8 +215,7 @@ void XScreen::draw(LyXText * text, Buffe | text->first_y = y; | |

Re: Compiler warnings

2003-01-10 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> but abs on a unsigned type or a type converted from unsiged is Lars> an oxymoron, isn't it? Lars> but I guess you are planning a Lars> int(uint) - int thing Yes. Lars> | And is abs always supposed to be in std:: for

Re: Compiler warnings

2003-01-10 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | > "Andre" == Andre Poenitz <[EMAIL PROTECTED]> writes: | | >> Is that true that unsigned int - int == int ? | | Andre> No. | | Andre> unsigned int - int == unsigned int | | Andre> As stupid as this maybe math-wise | | Yes, that's wh

Re: Compiler warnings

2003-01-10 Thread Andre Poenitz
On Fri, Jan 10, 2003 at 11:43:03AM +0100, Jean-Marc Lasgouttes wrote: > > "Andre" == Andre Poenitz <[EMAIL PROTECTED]> writes: > > >> Is that true that unsigned int - int == int ? > > Andre> No. > > Andre> unsigned int - int == unsigned int > > Andre> As stupid as this maybe math-wise

Re: Compiler warnings

2003-01-10 Thread Jean-Marc Lasgouttes
> "Andre" == Andre Poenitz <[EMAIL PROTECTED]> writes: >> Is that true that unsigned int - int == int ? Andre> No. Andre> unsigned int - int == unsigned int Andre> As stupid as this maybe math-wise Yes, that's what I meant to write. So I should static_cast, right? And is abs always

Re: Compiler warnings

2003-01-10 Thread Andre Poenitz
On Fri, Jan 10, 2003 at 11:35:37AM +0100, Jean-Marc Lasgouttes wrote: > ../../../../lyx-devel/src/frontends/xforms/xscreen.C: In method `void >XScreen::draw(LyXText *, BufferView *, unsigned int)': > ../../../../lyx-devel/src/frontends/xforms/xscreen.C:218: call of overloaded `abs >(unsigned int)

Re: Compiler warnings

2003-01-10 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | Lars> That's why I posted here. However, I fail to see how the two Lars> forms I | posted may have any difference. Am I blind? Lars> It looks correct to me as well. I feel stu

Re: Compiler warnings

2003-01-08 Thread Andre Poenitz
On Wed, Jan 08, 2003 at 06:18:55PM +0100, Jean-Marc Lasgouttes wrote: > posted may have any difference. Am I blind? I don't see a difference either. Andre' (usually in need of glasses) -- Those who desire to give up Freedom in order to gain Security, will not have, nor do they deserve, either o

Re: Compiler warnings

2003-01-08 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | That's why I posted here. However, I fail to see how the two forms I | posted may have any difference. Am I blind? It looks correct to me as well. -- Lgb

Re: Compiler warnings

2003-01-08 Thread John Levon
On Wed, Jan 08, 2003 at 06:18:55PM +0100, Jean-Marc Lasgouttes wrote: > >> // is any optimization possible? if (std::abs(y - old_first) < > >> owner_.workHeight()) > > That's why I posted here. However, I fail to see how the two forms I > posted may have any difference. Am I blind? Probably not,

Re: Compiler warnings

2003-01-08 Thread Jean-Marc Lasgouttes
> "John" == John Levon <[EMAIL PROTECTED]> writes: John> On Wed, Jan 08, 2003 at 05:26:48PM +0100, Jean-Marc Lasgouttes John> wrote: >> This one is weird: the code reads >> >> // is any optimization possible? if ((y - old_first) < >> owner_.workHeight() && (old_first - y) < owner_.workHeight(

Re: Compiler warnings

2003-01-08 Thread John Levon
On Wed, Jan 08, 2003 at 05:26:48PM +0100, Jean-Marc Lasgouttes wrote: > This one is weird: the code reads > > // is any optimization possible? > if ((y - old_first) < owner_.workHeight() > && (old_first - y) < owner_.workHeight()) > > is there a reason why the test is not >

Re: Compiler warnings

2003-01-08 Thread Jean-Marc Lasgouttes
> "Michael" == Michael Schmitt <[EMAIL PROTECTED]> writes: Michael> Hi, please find the current list of compiler warnings below Michael> (gcc-3.2, xforms frontend). I think a few of them are easily Michael> fixable. Michael> xscreen.C: In member function Michael> `virtual void XScreen::draw(L

Re: Compiler warnings

2002-03-01 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> Any good reasons to not do this: OK, I did it. JMarc

Re: Compiler warnings

2002-02-28 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> | vspace.C:447: warning: control reaches end of non-void Lars> function | `VSpace::asLatexCommand(const BufferParams &) const' Lars> Easy to fix... _but_ as I have no (clean)[1] lyx tree checked Lars> out at the moment I will

Re: Compiler warnings

2002-02-28 Thread Angus Leeming
On Wednesday 27 February 2002 8:34 pm, Lars Gullik Bjønnes wrote: > Michael Schmitt <[EMAIL PROTECTED]> writes: > > | ../../src/insets/insetgraphics.h:122: warning: member initializers for > | `const class string InsetGraphics::graphic_label' > | ../../src/insets/insetgraphics.h:125: warning: a

Re: Compiler warnings

2002-02-27 Thread Martin Vermeer
On Wed, Feb 27, 2002 at 09:34:06PM +0100, Lars Gullik Bjønnes wrote: ... > [1] I have three trees, one with the paragraph changes, one with > layout as string and a third with textclass as string in addition to > layout as string. > > -- > Lgb What about just putting either of the last

Re: Compiler warnings

2001-08-16 Thread John Levon
On Thu, Aug 16, 2001 at 01:06:04PM +0200, Michael Schmitt wrote: > Could you please fix these warnings even if they are harmless? my bad. will fix. john -- "Not to laugh, not to lament, not to curse, but to understand." - Spinoza

Re: Compiler warnings

1999-02-05 Thread Jochen Kuepper
On Fre, 05 Feb 1999 Jean-Marc Lasgouttes wrote: >> "Robert" == Robert Harley <[EMAIL PROTECTED]> writes: > >Robert> I built Lyx 1.0.0 on a Alpha running RedHat Linux 4.2 + >Robert> updates. I compiled with g++ from the egcs 1.1.1 >Robert> distribution. The following warnings were a little wo

Re: Compiler warnings

1999-02-05 Thread Jean-Marc Lasgouttes
> "Robert" == Robert Harley <[EMAIL PROTECTED]> writes: Robert> I built Lyx 1.0.0 on a Alpha running RedHat Linux 4.2 + Robert> updates. I compiled with g++ from the egcs 1.1.1 Robert> distribution. The following warnings were a little worrying: Yes, they are a bit strange indeed. Are you

Re: Compiler warnings

1999-02-03 Thread Tim Waugh
On Wed, 3 Feb 1999, Robert Harley wrote: > g++ -c -g -O2 -I. -I. -I../images -I/usr/X11R6/include FileInfo.C > FileInfo.C: In method `time_t & FileInfo::getModificationTime()': > FileInfo.C:287: warning: initializing non-const `time_t &' with `long unsigned int' >will use a temporary > FileIn