Fix portability issue with ArrayRef initializer overloading
authorErik Lindahl <erik@kth.se>
Wed, 30 Jul 2014 12:46:57 +0000 (14:46 +0200)
committerGerrit Code Review <gerrit@gerrit.gromacs.org>
Thu, 7 Aug 2014 17:59:16 +0000 (19:59 +0200)
commit61ca37ff7b4b8aa9c436818c1e04f1d63ea5dd0d
tree65a19f33f203ee275ebd6729e1598b907a13e85e
parent9635099c5aa3122110e84ba8ea3b999342011ba5
Fix portability issue with ArrayRef initializer overloading

Some compilers (in particular the Fujitsu compilers that are
derived from an earlier version of Clang) will not allow
overloading functions where one version uses a pointer to a
type, and the other a vector iterator to the same type - likely
because they have implemented iterators with pointers.
Regardless of what the C++ standard says, we need this working
on K computer, so this patch replaces the overloaded
initializers with non-member functions that create an ArrayRef
or ConstArrayRef either from pointers, an array,
or iterators.

Change-Id: I4c4e327c869920cc08e3f955e88cb3a5b28c7e87
src/gromacs/analysisdata/arraydata.cpp
src/gromacs/analysisdata/dataframe.cpp
src/gromacs/analysisdata/datastorage.cpp
src/gromacs/commandline/pargs.cpp
src/gromacs/options/filenameoption.cpp
src/gromacs/selection/nbsearch.cpp
src/gromacs/selection/selection.h
src/gromacs/utility/arrayref.h