Re: [Bug-apl] clang++ warning in SymbolTable.cc

2016-07-13 Thread Juergen Sauermann
Hi David, thanks,, fixed in SVN 779. /// Jürgen On 07/12/2016 06:55 PM, David B. Lamkins wrote: clang++ -DHAVE_CONFIG_H -I. -I.. -g -O2 -MT apl-SymbolTable.o -MD -MP -MF .deps/apl-SymbolTable.Tpo -c -o apl-SymbolTable.o `test -f 'SymbolTable.cc' || echo './'`SymbolTable.cc SymbolTable

Re: [Bug-apl] clang Warning

2014-04-23 Thread Juergen Sauermann
Hi, I see. I have changed the test in configure so that it checks whether the compiler is g++ rather than whether it accepts -rdynamic. SVN 221. That should also avoid the same warning in Solaris. /// Jürgen On 04/22/2014 07:01 PM, Peter Teeson wrote: Hi Jürgen: Gandalf:~ pteeson$ man gcc No

Re: [Bug-apl] clang Warning

2014-04-21 Thread Juergen Sauermann
Hi Peter, fixed in SVN 220. The -rdynamic is described in 'man g++'. /// Jürgen On 04/19/2014 06:22 PM, Peter Teeson wrote: /bin/sh ../../libtool --tag=CXX --mode=compile g++ -DHAVE_CONFIG_H -I. -I../.. -I/usr/local/include -rdynamic -g -O2 -MT file_io.lo -MD -MP -MF .deps/file_io.Tpo -c -

Re: [Bug-apl] clang Warning - revisited

2014-04-21 Thread Elias Mårtenson
There is a workaround to avoid the warning though, and that is to do a static_cast of the enum when you do the comparison. That was the recommendation when I was googling about this issue. Regards, Elias On 21 April 2014 18:24, Juergen Sauermann wrote: > Hi Vladimir, > > thanks. Not sure if yo

Re: [Bug-apl] clang Warning - revisited

2014-04-21 Thread Juergen Sauermann
Hi Vladimir, thanks. Not sure if you can do anything about the text of the warning, If so, then my proposal would be "comparison *may *always be true because all declarators in enum XXX are smaller than int 10" or so. /// Jürgen On 04/21/2014 09:19 AM, Vladimir Ur wrote: Juergen, hmm, ok. I a

Re: [Bug-apl] clang Warning - revisited

2014-04-20 Thread Juergen Sauermann
Hi Vladimir, I see. My point was just that the text of the warning, i.e. "comparison is always true" is wrong because it is, IMHO, not. And I hope that the optimizer in the compiler does not make the same assumption because that would lead to incorrect code. /// Jürgen On 04/20/2014 10:47 A

Re: [Bug-apl] clang Warning - revisited

2014-04-20 Thread Vladimir Ur
Juergen, I think whis warning is neither about compiler nor c standard. It is about programmers' convention to use enum type to hold limited set of values. Using enum for integer is not tricking the compiler, but tricking other programmers who follow this convention (or even the self). Most compile

Re: [Bug-apl] clang Warning - revisited

2014-04-19 Thread Juergen Sauermann
Hi Peter, I have changed it in SVN 215. I still disagree with the warning because in a comparison of an enumeration member with an integer should perform integer promotion of the enum member to int and then do integer comparison and NOT convert the int to an enum member (which it can't) and t

Re: [Bug-apl] clang warning

2014-04-18 Thread Juergen Sauermann
Hi Peter, thanks, fixed in SVN 212. /// Jürgen On 04/18/2014 02:40 AM, Peter Teeson wrote: Updated to revision 211. ../GNUAPL/apl-svn/src/UserPreferences.cc:378:1: Control may reach end of non-void function Peter

Re: [Bug-apl] clang warning

2014-04-17 Thread Peter Teeson
Updated to revision 211. ../GNUAPL/apl-svn/src/UserPreferences.cc:378:1: Control may reach end of non-void function Peter