EPICS Controls Argonne National Laboratory

Experimental Physics and
Industrial Control System

2002  2003  2004  2005  2006  2007  2008  2009  2010  <20112012  2013  2014  2015  2016  2017  2018  2019  2020  2021  2022  2023  2024  Index 2002  2003  2004  2005  2006  2007  2008  2009  2010  <20112012  2013  2014  2015  2016  2017  2018  2019  2020  2021  2022  2023  2024 
<== Date ==> <== Thread ==>

Subject: RE: [Merge] lp:~epics-core/epics-base/rebased-atomics into lp:epics-base
From: "Jeff Hill" <[email protected]>
To: "'Andrew Johnson'" <[email protected]>, "'EPICS core-talk'" <[email protected]>
Date: Mon, 12 Sep 2011 16:05:57 -0600
Hi Andrew,

I am surprised that there were no warning messages on 32 bit systems as that test code was changed to a c++ template and it is used with both integer and unsigned types. It's going to be difficult to use the C based testOk stdio format strings with the new tests which are now implemented by C++ templates so the best solution is probably going to be to just remove the format strings. I did this while also attempting to improve the test output at the same time, and pushed the changes to lp.

There are currently some issues with both of the win32-x86 and win32-x86-mingw builds on the R3.15 branch. Therefore, I did not try the windows64 build. likewise we don’t have that many if any 64 bit linux systems here so I didn’t try a 64 bit build.

timerQueuePassive.cpp
cl -c               /nologo /D__STDC__=0 /D_CRT_SECURE_NO_DEPRECATE /D_CRT_NONSTDC_NO_DEPRECATE   /Ox /GL   /W3 /w44355
si\\compiler\\default -I. -I..\\..\\..\\src\\libCom\\osi\\os\\WIN32 -I..\\..\\..\\src\\libCom\\osi\\os\\default -I.. -I..
c\\libCom\\cvtFast -I..\\..\\..\\src\\libCom\\cppStd -I..\\..\\..\\src\\libCom\\cxxTemplates -I..\\..\\..\\src\\libCom\\d
\..\\src\\libCom\\fdmgr -I..\\..\\..\\src\\libCom\\flex -I..\\..\\..\\src\\libCom\\freeList -I..\\..\\..\\src\\libCom\\gp
\\..\\src\\libCom\\misc -I..\\..\\..\\src\\libCom\\osi -I..\\..\\..\\src\\libCom\\ring -I..\\..\\..\\src\\libCom\\taskwd
\\include\\compiler\\msvc -I..\\..\\..\\include\\os\\WIN32 -I..\\..\\..\\include        ..\\..\\..\\src\\libCom\\tsDefs\\
tsDefs.c
cl -c               /nologo /D__STDC__=0 /D_CRT_SECURE_NO_DEPRECATE /D_CRT_NONSTDC_NO_DEPRECATE   /Ox /GL   /W3 /w44355
si\\compiler\\default -I. -I..\\..\\..\\src\\libCom\\osi\\os\\WIN32 -I..\\..\\..\\src\\libCom\\osi\\os\\default -I.. -I..
c\\libCom\\cvtFast -I..\\..\\..\\src\\libCom\\cppStd -I..\\..\\..\\src\\libCom\\cxxTemplates -I..\\..\\..\\src\\libCom\\d
\..\\src\\libCom\\fdmgr -I..\\..\\..\\src\\libCom\\flex -I..\\..\\..\\src\\libCom\\freeList -I..\\..\\..\\src\\libCom\\gp
\\..\\src\\libCom\\misc -I..\\..\\..\\src\\libCom\\osi -I..\\..\\..\\src\\libCom\\ring -I..\\..\\..\\src\\libCom\\taskwd
\\include\\compiler\\msvc -I..\\..\\..\\include\\os\\WIN32 -I..\\..\\..\\include        ..\\..\\..\\src\\libCom\\osi\\os\
epicsGetopt.c
cl /nologo /EHsc /GR -c               /nologo /D__STDC__=0 /D_CRT_SECURE_NO_DEPRECATE /D_CRT_NONSTDC_NO_DEPRECATE   /Ox /
.\\..\\src\\libCom\\osi\\compiler\\default -I. -I..\\..\\..\\src\\libCom\\osi\\os\\WIN32 -I..\\..\\..\\src\\libCom\\osi\\
calc -I..\\..\\..\\src\\libCom\\cvtFast -I..\\..\\..\\src\\libCom\\cppStd -I..\\..\\..\\src\\libCom\\cxxTemplates -I..\\.
bCom\\error -I..\\..\\..\\src\\libCom\\fdmgr -I..\\..\\..\\src\\libCom\\flex -I..\\..\\..\\src\\libCom\\freeList -I..\\..
bCom\\macLib -I..\\..\\..\\src\\libCom\\misc -I..\\..\\..\\src\\libCom\\osi -I..\\..\\..\\src\\libCom\\ring -I..\\..\\..\
om\\yacc -I..\\..\\..\\include\\compiler\\msvc -I..\\..\\..\\include\\os\\WIN32 -I..\\..\\..\\include        ..\\..\\..\\
setThreadName.cpp
cl /nologo /EHsc /GR -c               /nologo /D__STDC__=0 /D_CRT_SECURE_NO_DEPRECATE /D_CRT_NONSTDC_NO_DEPRECATE   /Ox /
.\\..\\src\\libCom\\osi\\compiler\\default -I. -I..\\..\\..\\src\\libCom\\osi\\os\\WIN32 -I..\\..\\..\\src\\libCom\\osi\\
calc -I..\\..\\..\\src\\libCom\\cvtFast -I..\\..\\..\\src\\libCom\\cppStd -I..\\..\\..\\src\\libCom\\cxxTemplates -I..\\.
bCom\\error -I..\\..\\..\\src\\libCom\\fdmgr -I..\\..\\..\\src\\libCom\\flex -I..\\..\\..\\src\\libCom\\freeList -I..\\..
bCom\\macLib -I..\\..\\..\\src\\libCom\\misc -I..\\..\\..\\src\\libCom\\osi -I..\\..\\..\\src\\libCom\\ring -I..\\..\\..\
om\\yacc -I..\\..\\..\\include\\compiler\\msvc -I..\\..\\..\\include\\os\\WIN32 -I..\\..\\..\\include        ..\\..\\..\\
forceBadAllocException.cpp
rc -l 0x409 -I. -I../O.Common -I. -I../../../src/libCom/osi/compiler/msvc -I../../../src/libCom/osi/compiler/default -I.
./src/libCom/bucketLib -I../../../src/libCom/calc -I../../../src/libCom/cvtFast -I../../../src/libCom/cppStd -I../../../s
./../src/libCom/error -I../../../src/libCom/fdmgr -I../../../src/libCom/flex -I../../../src/libCom/freeList -I../../../sr
rc/libCom/misc -I../../../src/libCom/osi -I../../../src/libCom/ring -I../../../src/libCom/taskwd -I../../../src/libCom/ti
/os/WIN32 -I../../../include        -fo Com.res ../Com.rc
Microsoft (R) Windows (R) Resource Compiler Version 6.1.7600.16385
Copyright (C) Microsoft Corporation.  All rights reserved.

gmake[1]: *** No rule to make target `..\\..\\..\\bin\\win32-x86\\antelope.exe', needed by `asLib.c'.  Stop.
gmake[1]: Leaving directory `C:/hill/epicsInBazaar/rebased-atomics/trunk/src/libCom/O.win32-x86'
gmake: *** [install.win32-x86] Error 2

gcc -c             -D_MINGW     -O3   -Wall      -m32    -D_DLL   -MMD -I. -I../O.Common -I. -I../../../src/libCom/osi/compiler/gcc -I../../.
Com/osi/os/posix -I../../../src/libCom/osi/os/default -I.. -I../../../src/libCom/as -I../../../src/libCom/bucketLib -I../../../src/libCom/cal
ates -I../../../src/libCom/dbmf -I../../../src/libCom/ellLib -I../../../src/libCom/env -I../../../src/libCom/error -I../../../src/libCom/fdmg
../../../src/libCom/iocsh -I../../../src/libCom/log -I../../../src/libCom/macLib -I../../../src/libCom/misc -I../../../src/libCom/osi -I../..
/libCom/tsDefs -I../../../src/libCom/yacc -I../../../include/compiler/gcc -I../../../include/os/WIN32 -I../../../include        ../../../src/
../../../src/libCom/flex/misc.c:38:13: warning: 'rcsid' defined but not used
""
gcc -c             -D_MINGW     -O3   -Wall      -m32    -D_DLL   -MMD -I. -I../O.Common -I. -I../../../src/libCom/osi/compiler/gcc -I../../.
Com/osi/os/posix -I../../../src/libCom/osi/os/default -I.. -I../../../src/libCom/as -I../../../src/libCom/bucketLib -I../../../src/libCom/cal
ates -I../../../src/libCom/dbmf -I../../../src/libCom/ellLib -I../../../src/libCom/env -I../../../src/libCom/error -I../../../src/libCom/fdmg
../../../src/libCom/iocsh -I../../../src/libCom/log -I../../../src/libCom/macLib -I../../../src/libCom/misc -I../../../src/libCom/osi -I../..
/libCom/tsDefs -I../../../src/libCom/yacc -I../../../include/compiler/gcc -I../../../include/os/WIN32 -I../../../include        ../../../src/
../../../src/libCom/flex/nfa.c:38:13: warning: 'rcsid' defined but not used
""
gcc -c             -D_MINGW     -O3   -Wall      -m32    -D_DLL   -MMD -I. -I../O.Common -I. -I../../../src/libCom/osi/compiler/gcc -I../../.
Com/osi/os/posix -I../../../src/libCom/osi/os/default -I.. -I../../../src/libCom/as -I../../../src/libCom/bucketLib -I../../../src/libCom/cal
ates -I../../../src/libCom/dbmf -I../../../src/libCom/ellLib -I../../../src/libCom/env -I../../../src/libCom/error -I../../../src/libCom/fdmg
../../../src/libCom/iocsh -I../../../src/libCom/log -I../../../src/libCom/macLib -I../../../src/libCom/misc -I../../../src/libCom/osi -I../..
/libCom/tsDefs -I../../../src/libCom/yacc -I../../../include/compiler/gcc -I../../../include/os/WIN32 -I../../../include        ../../../src/
../../../src/libCom/flex/sym.c:38:13: warning: 'rcsid' defined but not used
""
gcc -c             -D_MINGW     -O3   -Wall      -m32    -D_DLL   -MMD -I. -I../O.Common -I. -I../../../src/libCom/osi/compiler/gcc -I../../.
Com/osi/os/posix -I../../../src/libCom/osi/os/default -I.. -I../../../src/libCom/as -I../../../src/libCom/bucketLib -I../../../src/libCom/cal
ates -I../../../src/libCom/dbmf -I../../../src/libCom/ellLib -I../../../src/libCom/env -I../../../src/libCom/error -I../../../src/libCom/fdmg
../../../src/libCom/iocsh -I../../../src/libCom/log -I../../../src/libCom/macLib -I../../../src/libCom/misc -I../../../src/libCom/osi -I../..
/libCom/tsDefs -I../../../src/libCom/yacc -I../../../include/compiler/gcc -I../../../include/os/WIN32 -I../../../include        ../../../src/
../../../src/libCom/flex/tblcmp.c:38:13: warning: 'rcsid' defined but not used
perl -MExtUtils::Command -e rm_f parse.tab.c
perl -MExtUtils::Command -e rm_f parse.tab.h
../../../bin/win32-x86-mingw/antelope.exe -bparse -l -d ../../../src/libCom/flex/parse.y
perl -MExtUtils::Command -e mv parse.tab.c parse.c
perl -MExtUtils::Command -e mv parse.tab.h parse.h
""
gcc -c          -DDEFAULT_SKELETON_FILE=include/flex.skel.static   -D_MINGW     -O3   -Wall      -m32    -D_DLL   -MMD -I. -I../O.Common -I.
../../src/libCom/osi/os/WIN32 -I../../../src/libCom/osi/os/posix -I../../../src/libCom/osi/os/default -I.. -I../../../src/libCom/as -I../../.
src/libCom/cppStd -I../../../src/libCom/cxxTemplates -I../../../src/libCom/dbmf -I../../../src/libCom/ellLib -I../../../src/libCom/env -I../.
/libCom/freeList -I../../../src/libCom/gpHash -I../../../src/libCom/iocsh -I../../../src/libCom/log -I../../../src/libCom/macLib -I../../../s
askwd -I../../../src/libCom/timer -I../../../src/libCom/tsDefs -I../../../src/libCom/yacc -I../../../include/compiler/gcc -I../../../include/
parse.c:36:13: warning: 'rcsid' defined but not used
scan.l:780:22: warning: 'yy_last_accepting_state' defined but not used
scan.l:781:17: warning: 'yy_last_accepting_cpos' defined but not used
scan.l:812:12: warning: 'input' defined but not used
In file included from parse.c:347:0:
scan.l: In function 'flexscan':
scan.l:844:12: warning: 'indented_code' may be used uninitialized in this function
scan.l:844:12: note: 'indented_code' was declared here
scan.l:844:27: warning: 'checking_used' may be used uninitialized in this function
scan.l:844:27: note: 'checking_used' was declared here
scan.l:844:42: warning: 'new_xlation' may be used uninitialized in this function
scan.l:844:42: note: 'new_xlation' was declared here
g++ -o e_flex.exe         -m32               epicsTempFile.o ccl.o dfa.o ecs.o gen.o misc.o nfa.o sym.o tblcmp.o parse.o
"Installing created file ../../../bin/win32-x86-mingw/e_flex.exe"
../../../bin/win32-x86-mingw/e_flex.exe -S../../../include/flex.skel.static -8 -I -t ../../../src/libCom/as/asLib_lex.l > asLib_lex.yy.c
'..' is not recognized as an internal or external command,
operable program or batch file.
gmake[3]: *** [asLib_lex.c] Error 1
gmake[3]: Leaving directory `C:/hill/epicsInBazaar/rebased-atomics/trunk/src/libCom/O.win32-x86-mingw'
gmake[2]: *** [install.win32-x86-mingw] Error 2
gmake[2]: Leaving directory `C:/hill/epicsInBazaar/rebased-atomics/trunk/src/libCom'
gmake[1]: *** [libCom.install] Error 2
gmake[1]: Leaving directory `C:/hill/epicsInBazaar/rebased-atomics/trunk/src'
gmake: *** [src.install] Error 2

Jeff
______________________________________________________
Jeffrey O. Hill           Email        [email protected]
LANL MS H820              Voice        505 665 1831
Los Alamos NM 87545 USA   FAX          505 665 5107

Message content: TSPA

With sufficient thrust, pigs fly just fine. However, this is
not necessarily a good idea. It is hard to be sure where they
are going to land, and it could be dangerous sitting under them
as they fly overhead. -- RFC 1925


> -----Original Message-----
> From: [email protected] [mailto:[email protected]]
> On Behalf Of Andrew Johnson
> Sent: Friday, September 09, 2011 5:23 PM
> To: EPICS core-talk
> Subject: Re: [Merge] lp:~epics-core/epics-base/rebased-atomics into
> lp:epics-base
> 
> Hi Jeff,
> 
> On 2011-09-09 [email protected] wrote:
> > The proposal to merge lp:~epics-core/epics-base/rebased-atomics into
> >  lp:epics-base has been updated.
> >
> >     Status: Needs review => Merged
> 
> I merged the branch, but I am getting lots of warnings when I build on 64-
> bit
> system.  Maybe just cast the values to unsigned long and use %lu?
> 
> ../epicsAtomicTest.cpp: In function ‘void testIncrDecr() [with T = int]’:
> ../epicsAtomicTest.cpp:229:   instantiated from here
> ../epicsAtomicTest.cpp:137: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:229:   instantiated from here
> ../epicsAtomicTest.cpp:148: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp: In function ‘void testIncrDecr() [with T = long
> unsigned int]’:
> ../epicsAtomicTest.cpp:230:   instantiated from here
> ../epicsAtomicTest.cpp:134: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘long unsigned int’
> ../epicsAtomicTest.cpp:137: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:230:   instantiated from here
> ../epicsAtomicTest.cpp:148: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:151: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘long unsigned int’
> ../epicsAtomicTest.cpp: In function ‘void testAddSub() [with T = int]’:
> ../epicsAtomicTest.cpp:231:   instantiated from here
> ../epicsAtomicTest.cpp:171: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:231:   instantiated from here
> ../epicsAtomicTest.cpp:182: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp: In function ‘void testAddSub() [with T = long
> unsigned
> int]’:
> ../epicsAtomicTest.cpp:232:   instantiated from here
> ../epicsAtomicTest.cpp:168: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘long unsigned int’
> ../epicsAtomicTest.cpp:171: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:232:   instantiated from here
> ../epicsAtomicTest.cpp:182: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:185: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘long unsigned int’
> ../epicsAtomicTest.cpp: In function ‘void testCAS() [with T = int]’:
> ../epicsAtomicTest.cpp:233:   instantiated from here
> ../epicsAtomicTest.cpp:200: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:203: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:233:   instantiated from here
> ../epicsAtomicTest.cpp:216: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:219: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp: In function ‘void testCAS() [with T = long
> unsigned
> int]’:
> ../epicsAtomicTest.cpp:234:   instantiated from here
> ../epicsAtomicTest.cpp:200: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:203: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:234:   instantiated from here
> ../epicsAtomicTest.cpp:216: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:219: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp: In function ‘void testCAS() [with T = void*]’:
> ../epicsAtomicTest.cpp:235:   instantiated from here
> ../epicsAtomicTest.cpp:200: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:203: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:235:   instantiated from here
> ../epicsAtomicTest.cpp:216: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> ../epicsAtomicTest.cpp:219: warning: format ‘%u’ expects type ‘unsigned
> int’,
> but argument 3 has type ‘size_t’
> 
> - Andrew
> 
> PS: I'll be on vacation next week, so responses will be sporadic if any.
> --
> Optimization is the process of taking something that works and
> replacing it with something that almost works, but costs less.
> -- Roger Needham



References:
[Merge] lp:~epics-core/epics-base/rebased-atomics into lp:epics-base noreply
Re: [Merge] lp:~epics-core/epics-base/rebased-atomics into lp:epics-base Andrew Johnson

Navigate by Date:
Prev: Re: vxWorks 6.8 and "#define _VSB_CONFIG_FILE" Ernest L. Williams Jr.
Next: Re: EPICS and SMP: target board ==> ivme7210 ==> OS ==> vxWorks Ernest L. Williams Jr.
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  <20112012  2013  2014  2015  2016  2017  2018  2019  2020  2021  2022  2023  2024 
Navigate by Thread:
Prev: Re: [Merge] lp:~epics-core/epics-base/rebased-atomics into lp:epics-base Andrew Johnson
Next: extern "C" { #include <...> } Andrew Johnson
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  <20112012  2013  2014  2015  2016  2017  2018  2019  2020  2021  2022  2023  2024 
ANJ, 02 Feb 2012 Valid HTML 4.01! · Home · News · About · Base · Modules · Extensions · Distributions · Download ·
· Search · EPICS V4 · IRMIS · Talk · Bugs · Documents · Links · Licensing ·