Current CVS broken? (SortedArray.h)

From: Michael Mazack (michael_at_mazack.org)
Date: Tue Feb 05 2013 - 12:52:26 CST

Hi. I ran a CVS checkout on the namd2 source-tree yesterday, tried to
rebuild and was given a compiling error:

g++ -m64 -O3
-I/home/michael/work/code/namd2/charm-6.5.0/multicore-linux64/include
-DCMK_OPTIMIZE=1 -Isrc -Iinc -Iplugins/include -DSTATIC_PLUGIN
-I.rootdir/tcl-threaded/include -DNAMD_TCL -I.rootdir/fftw/include
-DNAMD_FFTW -DNAMD_VERSION=\"2.9\"
-DNAMD_PLATFORM=\"Linux-x86_64-multicore\"
-DREMOVE_PROXYRESULTMSG_EXTRACOPY -DNODEAWARE_PROXY_SPANNINGTREE
-DUSE_NODEPATCHMGR -fexpensive-optimizations -ffast-math -o
obj/ComputeMoa.o -c src/ComputeMoa.C
In file included from src/HomePatchList.h:15:0,
                 from src/PatchMap.h:11,
                 from src/ComputeMoa.C:27:
src/SortedArray.h: In member function ‘int SortedArray<Elem>::index(const
Elem&)’:
src/SortedArray.h:77:42: error: ‘found’ was not declared in this scope
make: *** [obj/ComputeMoa.o] Error 1

---
Using diff, it looks like "int found" was previously above "int isSorted"
as a protected variable in the declaration of template <class Elem> class
SortedArray: public SortableResizeArray<Elem> near the top of the old
SortedArray.h. With the new update in SortedArray.h, "int found" is not
declared as a class variable, but only as a local variable for a few
functions. Re-adding "int found" seems to make the code build fine, and it
seems to run this way too. However, the diff shows some significant
differences between the old SortedArray.h and new SortedArray.h, so I'm
unsure of any side effects by this "fix". Can anyone else confirm this?
Michael
-- 
Michael J. M. Mazack
Scientific Computation Program
University of Minnesota
Minneapolis, MN 55455
E-mail: michael_at_mazack.org
Web: http://mazack.org

This archive was generated by hypermail 2.1.6 : Wed Dec 31 2014 - 23:20:54 CST