[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug-apl] Fwd: Re: Bogus compiler warnings

From: Juergen Sauermann
Subject: [Bug-apl] Fwd: Re: Bogus compiler warnings
Date: Thu, 9 Mar 2017 23:22:14 +0100
User-agent: Mozilla/5.0 (X11; Linux i686; rv:45.0) Gecko/20100101 Thunderbird/45.2.0

Hi Alexey,

I believe it is not good if we have to introduce additional code just to suppress warnings.
In particular if the warning is obviously bogus.

-Werror is on purpose in order to improve the code quality of GNU APL.
If we would turn it off then that goal would be undermined.

I suppose that you can run configure like this:

CXXFLAGS="-Wnoerror" ./configure

If that doesn't work then configure.ac line 39 is place to change the CXXFLAGS.
After that autoreconf and a new ./configure is needed for the change to have an effect.

/// Jürgen

On 03/09/2017 11:29 AM, Alexey Veretennikov wrote:

I'm trying to compile GNU APL with GCC 3.4.6 armv5. There are couple of bogus compiler warnings, mainly about variable might be uninitialized.
Sometimes the situations are like this:

SomeObject * ptr = new SomeObject(arg);

- here compiler complains (sometimes) what ptr might be used uninitialized.
I've fixed it by separating variable declaration and initialization with the real value, like this:
In the beginning of the function:

SomeObject * ptr = 0;

ptr = new SomeObject();

So I can fix them by rearranging a code a little. It doesn't hurt as it seems.
But maybe it is not way to go, to complicate the code to satisfy the ancient compiler?

There are other issues, mainly related to HEX macro, where we supply pointer to this macro.
On 32bits machine the conversion int64_t(const char*) produces the warning.
I could solve them by introducing

It  makes code compile without warnings but looks not that pleasant.

But maybe there is a possibility to turn off -Werror with the ./configure option?


reply via email to

[Prev in Thread] Current Thread [Next in Thread]