Re: [Bug-apl] SVN 839 doesn't compile
Hi David,
thanks, hopefully fixed in SVN 840. I did not get these
warnings with g++ 4.8.4.
-Werror=misleading-indentation? They must be kidding.
/// Jürgen
On 01/09/2017 03:11 AM, David B.
Lamkins wrote:
GNU APL SVN 839 does not compile. Tested with gcc 6.3.1 and clang 3.8.0 using ./configure; make clean all .
Here are the compilation errors. I can post the entire make transcript if it'd help.
gcc
===
g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT apl-Common.o -MD -MP -MF .deps/apl-Common.Tpo -c -o apl-Common.o `test -f 'Common.cc' || echo './'`Common.cc
Command.cc: In static member function ‘static void Command::lib_common(std::ostream&, const UCS_string&, int)’:
Command.cc:1022:17: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
if (col == (col_width.size() - 1) || c == (directories.size() - 1))
^
g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT apl-Executable.o -MD -MP -MF .deps/apl-Executable.Tpo -c -o apl-Executable.o `test -f 'Executable.cc' || echo './'`Executable.cc
Error.cc: In member function ‘void Error::print(std::ostream&) const’:
Error.cc:60:9: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation]
if (symbol_name.size())
^~
Error.cc:63:27: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
out << " Thrown at: " << throw_loc << endl
^~~
g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT apl-FloatCell.o -MD -MP -MF .deps/apl-FloatCell.Tpo -c -o apl-FloatCell.o `test -f 'FloatCell.cc' || echo './'`FloatCell.cc
Executable.cc: In member function ‘UCS_string Executable::extract_lambda_text(Fun_signature, int) const’:
Executable.cc:691:19: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
if (tidx >= text.size())
~^~
cc1plus: all warnings being treated as errors
clang
=
clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Assert.o -MD -MP -MF .deps/apl-Assert.Tpo -c -o apl-Assert.o `test -f 'Assert.cc' || echo './'`Assert.cc
In file included from Archive.cc:41:
In file included from ./Macro.hh:24:
In file included from ./UserFunction.hh:29:
./Symbol.hh:39:1: error: 'ValueStackItem' defined as a struct here but previously declared as a class [-Werror,-Wmismatched-tags]
struct ValueStackItem
^
./Archive.hh:40:1: note: did you mean struct here?
class ValueStackItem;
^
struct
In file included from Assert.cc:28:
In file included from ./Workspace.hh:30:
In file included from ./Quad_RL.hh:24:
./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides overloaded virtual function [-Werror,-Woverloaded-virtual]
virtual void push_label(int label) {}
^
./Symbol.hh:175:17: note: hidden overloaded virtual function 'Symbol::push_label' declared here: type mismatch at 1st parameter ('Function_Line' vs 'int')
virtual void push_label(Function_Line label);
clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Avec.o -MD -MP -MF .deps/apl-Avec.Tpo -c -o apl-Avec.o `test -f 'Avec.cc' || echo './'`Avec.cc
In file included from Archive.cc:51:
In file included from ./Workspace.hh:30:
In file included from ./Quad_RL.hh:24:
./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides overloaded virtual function [-Werror,-Woverloaded-virtual]
virtual void push_label(int label) {}
^
./Symbol.hh:175:17: note: hidden overloaded virtual function 'Symbol::push_label' declared here: type mismatch at 1st parameter ('Function_Line' vs 'int')
virtual void push_label(Function_Line label);
^
In file included from main.cc:42:
In file included from ./Workspace.hh:30:
In file included from ./Quad_RL.hh:24:
./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides overloaded virtual function [-Werror,-Woverloaded-virtual]
virtual void push_label(int label) {}
^
./Symbol.hh:175:17: note: hidden overloaded virtual function 'Symbol::push_label' declared here: type mismatch at 1st parameter ('Function_Line' vs 'int')
virtual void push_label(Function_Line label);
^
1 error generated.
^
Re: [Bug-apl] SVN 839 doesn't compile
Compile errors on Fedora 25 at svn revision 840, Xtian. g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql-rdynamic -g -O2 -MT apl-LApack.o -MD -MP -MF .deps/apl-LApack.Tpo -c -o apl-LApack.o `test -f 'LApack.cc' || echo './'`LApack.cc InputFile.cc: In static member function ‘static void InputFile::randomize_files()’: InputFile.cc:91:31: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (size_t done = 0; done < 4*files_todo.size();) ~^ mv -f .deps/apl-IndexIterator.Tpo .deps/apl-IndexIterator.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql-rdynamic -g -O2 -MT apl-LibPaths.o -MD -MP -MF .deps/apl-LibPaths.Tpo -c -o apl-LibPaths.o `test -f 'LibPaths.cc' || echo './'`LibPaths.cc mv -f .deps/apl-IntCell.Tpo .deps/apl-IntCell.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql-rdynamic -g -O2 -MT apl-LineInput.o -MD -MP -MF .deps/apl-LineInput.Tpo -c -o apl-LineInput.o `test -f 'LineInput.cc' || echo './'`LineInput.cc cc1plus: all warnings being treated as errors Makefile:2260: recipe for target 'apl-InputFile.o' failed make[3]: *** [apl-InputFile.o] Error 1 make[3]: *** Waiting for unfinished jobs LineInput.cc: In member function ‘void LineHistory::save_history(const char*)’: LineInput.cc:198:31: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (size_t p = put + 1; p < hist_lines.size(); ++p) ~~^~~ LineInput.cc: In member function ‘void LineHistory::print_history(std::ostream&)’: LineInput.cc:226:31: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] for (size_t p = put + 1; p < hist_lines.size(); ++p) ~~^~~ mv -f .deps/apl-LibPaths.Tpo .deps/apl-LibPaths.Po mv -f .deps/apl-LApack.Tpo .deps/apl-LApack.Po cc1plus: all warnings being treated as errors Makefile:2316: recipe for target 'apl-LineInput.o' failed make[3]: *** [apl-LineInput.o] Error 1 make[3]: Leaving directory '/home/xtian/gnuapl/src' Makefile:3162: recipe for target 'all-recursive' failed make[2]: *** [all-recursive] Error 1 make[2]: Leaving directory '/home/xtian/gnuapl/src' Makefile:520: recipe for target 'all-recursive' failed make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory '/home/xtian/gnuapl' Makefile:408: recipe for target 'all' failed make: *** [all] Error 2 On 2017-01-09 07:13, Juergen Sauermann wrote: Hi David, thanks, hopefully fixed in *SVN 840*. I did not get these warnings with *g++ 4.8.4*. *-Werror=misleading-indentation*? They must be kidding. /// Jürgen On 01/09/2017 03:11 AM, David B. Lamkins wrote: GNU APL SVN 839 does not compile. Tested with gcc 6.3.1 and clang 3.8.0 using ./configure; make clean all . Here are the compilation errors. I can post the entire make transcript if it'd help. gcc === g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT apl-Common.o -MD -MP -MF .deps/apl-Common.Tpo -c -o apl-Common.o `test -f 'Common.cc' || echo './'`Common.cc Command.cc: In static member function ‘static void Command::lib_common(std::ostream&, const UCS_string&, int)’: Command.cc:1022:17: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (col == (col_width.size() - 1) || c == (directories.size() - 1)) ^ g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT apl-Executable.o -MD -MP -MF .deps/apl-Executable.Tpo -c -o apl-Executable.o `test -f 'Executable.cc' || echo './'`Executable.cc Error.cc: In member function ‘void Error::print(std::ostream&) const’: Error.cc:60:9: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation] if (symbol_name.size()) ^~ Error.cc:63:27: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’ out << " Thrown at: " << throw_loc << endl ^~~ g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT apl-FloatCell.o -MD -MP -MF .deps/apl-FloatCell.Tpo -c -o apl-FloatCell.o `test -f 'FloatCell.cc' || echo './'`FloatCell.cc Executable.cc: In member function ‘UCS_string Executable::extract_lambda_text(Fun_signature, int) const’: Executable.cc:691:19: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (tidx >= text.size()) ~^~ cc1plus: all warnings being treated as errors clang = clang++ -DHAVE_CONFIG_H -I
Re: [Bug-apl] SVN 839 doesn't compile
I must've missed some error when excerpting the 839 transcript. Sorry.
I've attached make transcripts with the remaining errors.
On Mon, Jan 9, 2017 at 4:13 AM, Juergen Sauermann
wrote:
> Hi David,
>
> thanks, hopefully fixed in SVN 840. I did not get these warnings with g++
> 4.8.4.
>
> -Werror=misleading-indentation? They must be kidding.
>
> /// Jürgen
>
>
> On 01/09/2017 03:11 AM, David B. Lamkins wrote:
>
> GNU APL SVN 839 does not compile. Tested with gcc 6.3.1 and clang 3.8.0
> using ./configure; make clean all .
>
> Here are the compilation errors. I can post the entire make transcript if
> it'd help.
>
> gcc
> ===
>
> g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> apl-Common.o -MD -MP -MF .deps/apl-Common.Tpo -c -o apl-Common.o `test -f
> 'Common.cc' || echo './'`Common.cc
> Command.cc: In static member function ‘static void
> Command::lib_common(std::ostream&, const UCS_string&, int)’:
> Command.cc:1022:17: error: comparison between signed and unsigned integer
> expressions [-Werror=sign-compare]
> if (col == (col_width.size() - 1) || c == (directories.size() - 1))
> ^
>
> g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> apl-Executable.o -MD -MP -MF .deps/apl-Executable.Tpo -c -o apl-Executable.o
> `test -f 'Executable.cc' || echo './'`Executable.cc
> Error.cc: In member function ‘void Error::print(std::ostream&) const’:
> Error.cc:60:9: error: this ‘if’ clause does not guard...
> [-Werror=misleading-indentation]
> if (symbol_name.size())
> ^~
> Error.cc:63:27: note: ...this statement, but the latter is misleadingly
> indented as if it is guarded by the ‘if’
>out << " Thrown at: " << throw_loc << endl
>^~~
>
> g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> apl-FloatCell.o -MD -MP -MF .deps/apl-FloatCell.Tpo -c -o apl-FloatCell.o
> `test -f 'FloatCell.cc' || echo './'`FloatCell.cc
> Executable.cc: In member function ‘UCS_string
> Executable::extract_lambda_text(Fun_signature, int) const’:
> Executable.cc:691:19: error: comparison between signed and unsigned integer
> expressions [-Werror=sign-compare]
> if (tidx >= text.size())
> ~^~
> cc1plus: all warnings being treated as errors
>
>
>
> clang
> =
>
> clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Assert.o -MD
> -MP -MF .deps/apl-Assert.Tpo -c -o apl-Assert.o `test -f 'Assert.cc' || echo
> './'`Assert.cc
> In file included from Archive.cc:41:
> In file included from ./Macro.hh:24:
> In file included from ./UserFunction.hh:29:
> ./Symbol.hh:39:1: error: 'ValueStackItem' defined as a struct here but
> previously declared as a class [-Werror,-Wmismatched-tags]
> struct ValueStackItem
> ^
> ./Archive.hh:40:1: note: did you mean struct here?
> class ValueStackItem;
> ^
> struct
> In file included from Assert.cc:28:
> In file included from ./Workspace.hh:30:
> In file included from ./Quad_RL.hh:24:
> ./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides
> overloaded virtual function [-Werror,-Woverloaded-virtual]
>virtual void push_label(int label) {}
> ^
> ./Symbol.hh:175:17: note: hidden overloaded virtual function
> 'Symbol::push_label' declared here: type mismatch at 1st parameter
> ('Function_Line' vs 'int')
>virtual void push_label(Function_Line label);
>
> clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Avec.o -MD -MP
> -MF .deps/apl-Avec.Tpo -c -o apl-Avec.o `test -f 'Avec.cc' || echo
> './'`Avec.cc
> In file included from Archive.cc:51:
> In file included from ./Workspace.hh:30:
> In file included from ./Quad_RL.hh:24:
> ./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides
> overloaded virtual function [-Werror,-Woverloaded-virtual]
>virtual void push_label(int label) {}
> ^
> ./Symbol.hh:175:17: note: hidden overloaded virtual function
> 'Symbol::push_label' declared here: type mismatch at 1st parameter
> ('Function_Line' vs 'int')
>virtual void push_label(Function_Line label);
> ^
> In file included from main.cc:42:
> In file included from ./Workspace.hh:30:
> In file included from ./Quad_RL.hh:24:
> ./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides
> overloaded virtual function [-Werror,-Woverloaded-virtual]
>virtual void push_label(int label) {}
> ^
> ./Symbol.hh:175:17: note: hidden overloaded virtual function
> 'Symbol::push_label' declared here: type mismatch at 1st parameter
> ('Funct
Re: [Bug-apl] SVN 839 doesn't compile
Hi,
fixed in SVN 841.
I guess what happens is that g++ compiles a handful of
files in parallel (according to the
-J option for make) and only shows the errors in those
files.
If you run make several time (until Workspace.cc ia
being compiled) then you will see more
and more errors as make proceeds.
The reasons why these warnings show up only now is that I have
changed the normal build
to do -Wall in order to get a cleaner build. The
strange thing is that my g++ does not show
these warnings even though I use the same -W options as
you do.
/// Jürgen
On 01/09/2017 07:19 PM, David Lamkins
wrote:
I must've missed some error when excerpting the 839 transcript. Sorry.
I've attached make transcripts with the remaining errors.
On Mon, Jan 9, 2017 at 4:13 AM, Juergen Sauermann
wrote:
Hi David,
thanks, hopefully fixed in SVN 840. I did not get these warnings with g++
4.8.4.
-Werror=misleading-indentation? They must be kidding.
/// Jürgen
On 01/09/2017 03:11 AM, David B. Lamkins wrote:
GNU APL SVN 839 does not compile. Tested with gcc 6.3.1 and clang 3.8.0
using ./configure; make clean all .
Here are the compilation errors. I can post the entire make transcript if
it'd help.
gcc
===
g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
-I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
apl-Common.o -MD -MP -MF .deps/apl-Common.Tpo -c -o apl-Common.o `test -f
'Common.cc' || echo './'`Common.cc
Command.cc: In static member function ‘static void
Command::lib_common(std::ostream&, const UCS_string&, int)’:
Command.cc:1022:17: error: comparison between signed and unsigned integer
expressions [-Werror=sign-compare]
if (col == (col_width.size() - 1) || c == (directories.size() - 1))
^
g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
-I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
apl-Executable.o -MD -MP -MF .deps/apl-Executable.Tpo -c -o apl-Executable.o
`test -f 'Executable.cc' || echo './'`Executable.cc
Error.cc: In member function ‘void Error::print(std::ostream&) const’:
Error.cc:60:9: error: this ‘if’ clause does not guard...
[-Werror=misleading-indentation]
if (symbol_name.size())
^~
Error.cc:63:27: note: ...this statement, but the latter is misleadingly
indented as if it is guarded by the ‘if’
out << " Thrown at: " << throw_loc << endl
^~~
g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
-I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
apl-FloatCell.o -MD -MP -MF .deps/apl-FloatCell.Tpo -c -o apl-FloatCell.o
`test -f 'FloatCell.cc' || echo './'`FloatCell.cc
Executable.cc: In member function ‘UCS_string
Executable::extract_lambda_text(Fun_signature, int) const’:
Executable.cc:691:19: error: comparison between signed and unsigned integer
expressions [-Werror=sign-compare]
if (tidx >= text.size())
~^~
cc1plus: all warnings being treated as errors
clang
=
clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
-I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Assert.o -MD
-MP -MF .deps/apl-Assert.Tpo -c -o apl-Assert.o `test -f 'Assert.cc' || echo
'./'`Assert.cc
In file included from Archive.cc:41:
In file included from ./Macro.hh:24:
In file included from ./UserFunction.hh:29:
./Symbol.hh:39:1: error: 'ValueStackItem' defined as a struct here but
previously declared as a class [-Werror,-Wmismatched-tags]
struct ValueStackItem
^
./Archive.hh:40:1: note: did you mean struct here?
class ValueStackItem;
^
struct
In file included from Assert.cc:28:
In file included from ./Workspace.hh:30:
In file included from ./Quad_RL.hh:24:
./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides
overloaded virtual function [-Werror,-Woverloaded-virtual]
virtual void push_label(int label) {}
^
./Symbol.hh:175:17: note: hidden overloaded virtual function
'Symbol::push_label' declared here: type mismatch at 1st parameter
('Function_Line' vs 'int')
virtual void push_label(Function_Line label);
clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
-I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Avec.o -MD -MP
-MF .deps/apl-Avec.Tpo -c -o apl-Avec.o `test -f 'Avec.cc' || echo
'./'`Avec.cc
In file included from Archive.cc:51:
In file included from ./Workspace.hh:30:
In file included from ./Quad_RL.hh:24:
./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides
overloaded virtual function [-Werror,-Woverloaded-virtual]
virtual void push_label(int label) {}
^
./Symbol.hh:175:17: note: hidden overloaded virtual function
'Symbol::push_label' decla
Re: [Bug-apl] SVN 839 doesn't compile
You're right. I forgot about the fail-on-first error nature of make;
the -j misled me to think that it was reporting all errors. This time
I've done a make -k; this ought to catch more errors. See attached.
A new version of gcc is always an adventure. :\
On Mon, Jan 9, 2017 at 11:05 AM, Juergen Sauermann
wrote:
> Hi,
>
> fixed in SVN 841.
>
> I guess what happens is that g++ compiles a handful of files in parallel
> (according to the
> -J option for make) and only shows the errors in those files.
>
> If you run make several time (until Workspace.cc ia being compiled) then you
> will see more
> and more errors as make proceeds.
>
> The reasons why these warnings show up only now is that I have changed the
> normal build
> to do -Wall in order to get a cleaner build. The strange thing is that my
> g++ does not show
> these warnings even though I use the same -W options as you do.
>
> /// Jürgen
>
>
> On 01/09/2017 07:19 PM, David Lamkins wrote:
>
> I must've missed some error when excerpting the 839 transcript. Sorry.
> I've attached make transcripts with the remaining errors.
>
> On Mon, Jan 9, 2017 at 4:13 AM, Juergen Sauermann
> wrote:
>
> Hi David,
>
> thanks, hopefully fixed in SVN 840. I did not get these warnings with g++
> 4.8.4.
>
> -Werror=misleading-indentation? They must be kidding.
>
> /// Jürgen
>
>
> On 01/09/2017 03:11 AM, David B. Lamkins wrote:
>
> GNU APL SVN 839 does not compile. Tested with gcc 6.3.1 and clang 3.8.0
> using ./configure; make clean all .
>
> Here are the compilation errors. I can post the entire make transcript if
> it'd help.
>
> gcc
> ===
>
> g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> apl-Common.o -MD -MP -MF .deps/apl-Common.Tpo -c -o apl-Common.o `test -f
> 'Common.cc' || echo './'`Common.cc
> Command.cc: In static member function ‘static void
> Command::lib_common(std::ostream&, const UCS_string&, int)’:
> Command.cc:1022:17: error: comparison between signed and unsigned integer
> expressions [-Werror=sign-compare]
> if (col == (col_width.size() - 1) || c == (directories.size() - 1))
> ^
>
> g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> apl-Executable.o -MD -MP -MF .deps/apl-Executable.Tpo -c -o apl-Executable.o
> `test -f 'Executable.cc' || echo './'`Executable.cc
> Error.cc: In member function ‘void Error::print(std::ostream&) const’:
> Error.cc:60:9: error: this ‘if’ clause does not guard...
> [-Werror=misleading-indentation]
> if (symbol_name.size())
> ^~
> Error.cc:63:27: note: ...this statement, but the latter is misleadingly
> indented as if it is guarded by the ‘if’
>out << " Thrown at: " << throw_loc << endl
>^~~
>
> g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> apl-FloatCell.o -MD -MP -MF .deps/apl-FloatCell.Tpo -c -o apl-FloatCell.o
> `test -f 'FloatCell.cc' || echo './'`FloatCell.cc
> Executable.cc: In member function ‘UCS_string
> Executable::extract_lambda_text(Fun_signature, int) const’:
> Executable.cc:691:19: error: comparison between signed and unsigned integer
> expressions [-Werror=sign-compare]
> if (tidx >= text.size())
> ~^~
> cc1plus: all warnings being treated as errors
>
>
>
> clang
> =
>
> clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Assert.o -MD
> -MP -MF .deps/apl-Assert.Tpo -c -o apl-Assert.o `test -f 'Assert.cc' || echo
> './'`Assert.cc
> In file included from Archive.cc:41:
> In file included from ./Macro.hh:24:
> In file included from ./UserFunction.hh:29:
> ./Symbol.hh:39:1: error: 'ValueStackItem' defined as a struct here but
> previously declared as a class [-Werror,-Wmismatched-tags]
> struct ValueStackItem
> ^
> ./Archive.hh:40:1: note: did you mean struct here?
> class ValueStackItem;
> ^
> struct
> In file included from Assert.cc:28:
> In file included from ./Workspace.hh:30:
> In file included from ./Quad_RL.hh:24:
> ./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides
> overloaded virtual function [-Werror,-Woverloaded-virtual]
>virtual void push_label(int label) {}
> ^
> ./Symbol.hh:175:17: note: hidden overloaded virtual function
> 'Symbol::push_label' declared here: type mismatch at 1st parameter
> ('Function_Line' vs 'int')
>virtual void push_label(Function_Line label);
>
> clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> -I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Avec.o -MD -MP
> -MF .deps/apl-Avec.Tpo -c -o apl-Avec.o `test -f 'Avec.cc' || echo
> './'`Avec.cc
> In file included from Archive.cc:51:
Re: [Bug-apl] SVN 839 doesn't compile
They should have frozen gcc dev at 2.95 :)
On Mon, 9 Jan 2017 11:45:23 -0800
David Lamkins wrote:
> You're right. I forgot about the fail-on-first error nature of make;
> the -j misled me to think that it was reporting all errors. This time
> I've done a make -k; this ought to catch more errors. See attached.
>
> A new version of gcc is always an adventure. :\
>
> On Mon, Jan 9, 2017 at 11:05 AM, Juergen Sauermann
> wrote:
> > Hi,
> >
> > fixed in SVN 841.
> >
> > I guess what happens is that g++ compiles a handful of files in parallel
> > (according to the
> > -J option for make) and only shows the errors in those files.
> >
> > If you run make several time (until Workspace.cc ia being compiled) then you
> > will see more
> > and more errors as make proceeds.
> >
> > The reasons why these warnings show up only now is that I have changed the
> > normal build
> > to do -Wall in order to get a cleaner build. The strange thing is that my
> > g++ does not show
> > these warnings even though I use the same -W options as you do.
> >
> > /// Jürgen
> >
> >
> > On 01/09/2017 07:19 PM, David Lamkins wrote:
> >
> > I must've missed some error when excerpting the 839 transcript. Sorry.
> > I've attached make transcripts with the remaining errors.
> >
> > On Mon, Jan 9, 2017 at 4:13 AM, Juergen Sauermann
> > wrote:
> >
> > Hi David,
> >
> > thanks, hopefully fixed in SVN 840. I did not get these warnings with g++
> > 4.8.4.
> >
> > -Werror=misleading-indentation? They must be kidding.
> >
> > /// Jürgen
> >
> >
> > On 01/09/2017 03:11 AM, David B. Lamkins wrote:
> >
> > GNU APL SVN 839 does not compile. Tested with gcc 6.3.1 and clang 3.8.0
> > using ./configure; make clean all .
> >
> > Here are the compilation errors. I can post the entire make transcript if
> > it'd help.
> >
> > gcc
> > ===
> >
> > g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> > -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> > apl-Common.o -MD -MP -MF .deps/apl-Common.Tpo -c -o apl-Common.o `test -f
> > 'Common.cc' || echo './'`Common.cc
> > Command.cc: In static member function ‘static void
> > Command::lib_common(std::ostream&, const UCS_string&, int)’:
> > Command.cc:1022:17: error: comparison between signed and unsigned integer
> > expressions [-Werror=sign-compare]
> > if (col == (col_width.size() - 1) || c == (directories.size() - 1))
> > ^
> >
> > g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> > -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> > apl-Executable.o -MD -MP -MF .deps/apl-Executable.Tpo -c -o apl-Executable.o
> > `test -f 'Executable.cc' || echo './'`Executable.cc
> > Error.cc: In member function ‘void Error::print(std::ostream&) const’:
> > Error.cc:60:9: error: this ‘if’ clause does not guard...
> > [-Werror=misleading-indentation]
> > if (symbol_name.size())
> > ^~
> > Error.cc:63:27: note: ...this statement, but the latter is misleadingly
> > indented as if it is guarded by the ‘if’
> >out << " Thrown at: " << throw_loc << endl
> >^~~
> >
> > g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> > -I/usr/include -I sql -I/usr/include -I sql -rdynamic -g -O2 -MT
> > apl-FloatCell.o -MD -MP -MF .deps/apl-FloatCell.Tpo -c -o apl-FloatCell.o
> > `test -f 'FloatCell.cc' || echo './'`FloatCell.cc
> > Executable.cc: In member function ‘UCS_string
> > Executable::extract_lambda_text(Fun_signature, int) const’:
> > Executable.cc:691:19: error: comparison between signed and unsigned integer
> > expressions [-Werror=sign-compare]
> > if (tidx >= text.size())
> > ~^~
> > cc1plus: all warnings being treated as errors
> >
> >
> >
> > clang
> > =
> >
> > clang++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing
> > -I/usr/include -I sql -I/usr/include -I sql -g -O2 -MT apl-Assert.o -MD
> > -MP -MF .deps/apl-Assert.Tpo -c -o apl-Assert.o `test -f 'Assert.cc' || echo
> > './'`Assert.cc
> > In file included from Archive.cc:41:
> > In file included from ./Macro.hh:24:
> > In file included from ./UserFunction.hh:29:
> > ./Symbol.hh:39:1: error: 'ValueStackItem' defined as a struct here but
> > previously declared as a class [-Werror,-Wmismatched-tags]
> > struct ValueStackItem
> > ^
> > ./Archive.hh:40:1: note: did you mean struct here?
> > class ValueStackItem;
> > ^
> > struct
> > In file included from Assert.cc:28:
> > In file included from ./Workspace.hh:30:
> > In file included from ./Quad_RL.hh:24:
> > ./SystemVariable.hh:82:17: error: 'NL_SystemVariable::push_label' hides
> > overloaded virtual function [-Werror,-Woverloaded-virtual]
> >virtual void push_label(int label) {}
> > ^
> > ./Symbol.hh:175:17: note: hidden overloaded virtual function
> > 'Symbol::push_label' declared here: type mismatch at 1st parameter
[Bug-apl] Errors building 841
On 64 bit Linux using GCC-4.8.4 (Same system I've been using for a couple of years.) ... g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-PrimitiveOperator.o -MD -MP -MF .deps/apl-PrimitiveOperator.Tpo -c -o apl-PrimitiveOperator.o `test -f 'PrimitiveOperator.cc' || echo './'`PrimitiveOperator.cc mv -f .deps/apl-PointerCell.Tpo .deps/apl-PointerCell.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-PrintBuffer.o -MD -MP -MF .deps/apl-PrintBuffer.Tpo -c -o apl-PrintBuffer.o `test -f 'PrintBuffer.cc' || echo './'`PrintBuffer.cc mv -f .deps/apl-PrimitiveOperator.Tpo .deps/apl-PrimitiveOperator.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-QuadFunction.o -MD -MP -MF .deps/apl-QuadFunction.Tpo -c -o apl-QuadFunction.o `test -f 'QuadFunction.cc' || echo './'`QuadFunction.cc mv -f .deps/apl-Prefix.Tpo .deps/apl-Prefix.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-ProcessorID.o -MD -MP -MF .deps/apl-ProcessorID.Tpo -c -o apl-ProcessorID.o `test -f 'ProcessorID.cc' || echo './'`ProcessorID.cc mv -f .deps/apl-PrintBuffer.Tpo .deps/apl-PrintBuffer.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-Quad_CR.o -MD -MP -MF .deps/apl-Quad_CR.Tpo -c -o apl-Quad_CR.o `test -f 'Quad_CR.cc' || echo './'`Quad_CR.cc mv -f .deps/apl-ProcessorID.Tpo .deps/apl-ProcessorID.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-Quad_DLX.o -MD -MP -MF .deps/apl-Quad_DLX.Tpo -c -o apl-Quad_DLX.o `test -f 'Quad_DLX.cc' || echo './'`Quad_DLX.cc mv -f .deps/apl-Quad_DLX.Tpo .deps/apl-Quad_DLX.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-Quad_FIO.o -MD -MP -MF .deps/apl-Quad_FIO.Tpo -c -o apl-Quad_FIO.o `test -f 'Quad_FIO.cc' || echo './'`Quad_FIO.cc mv -f .deps/apl-QuadFunction.Tpo .deps/apl-QuadFunction.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-Quad_FX.o -MD -MP -MF .deps/apl-Quad_FX.Tpo -c -o apl-Quad_FX.o `test -f 'Quad_FX.cc' || echo './'`Quad_FX.cc mv -f .deps/apl-PrimitiveFunction.Tpo .deps/apl-PrimitiveFunction.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-Quad_RL.o -MD -MP -MF .deps/apl-Quad_RL.Tpo -c -o apl-Quad_RL.o `test -f 'Quad_RL.cc' || echo './'`Quad_RL.cc Quad_FIO.cc: In member function ‘virtual Token Quad_FIO::eval_XB(Value_P, Value_P)’: Quad_FIO.cc:1205:53: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (fd > 8*sizeof(fd_set)) DOMAIN_ERROR; ^ Quad_FIO.cc:1220:53: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (fd > 8*sizeof(fd_set)) DOMAIN_ERROR; ^ Quad_FIO.cc:1235:53: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (fd > 8*sizeof(fd_set)) DOMAIN_ERROR; ^ mv -f .deps/apl-Quad_CR.Tpo .deps/apl-Quad_CR.Po g++ -DHAVE_CONFIG_H -I. -I..-Werror -Wall -Wno-strict-aliasing -I/usr/include -I sql -I/usr/include/postgresql -I sql -rdynamic -g -O2 -MT apl-Quad_SQL.o -MD -MP -MF .deps/apl-Quad_SQL.Tpo -c -o apl-Quad_SQL.o `test -f 'Quad_SQL.cc' || echo './'`Quad_SQL.cc cc1plus: all warnings being treated as errors make[3]: *** [apl-Quad_FIO.o] Error 1 make[3]: *** Waiting for unfinished jobs mv -f .deps/apl-Quad_RL.Tpo .deps/apl-Quad_RL.Po mv -f .deps/apl-Quad_FX.Tpo .deps/apl-Quad_FX.Po mv -f .deps/apl-Quad_SQL.Tpo .deps/apl-Quad_SQL.Po make[3]: Leaving directory `/home/blake/Backup/apl/src' make[2]: *** [all-recursive] Error 1 make[2]: Leaving directory `/home/blake/Backup/apl/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/blake/Backup/apl' make: *** [all] Error 2
