Merge "Improve ProgramInfo and use it everywhere."

Merge "Fix most doxygen warnings."

Remove unused FFT functions

Change-Id: Ic3fd21579f6dcf7e57084c5a9beed4637a8cae6a

Fix most doxygen warnings.

This makes it much more convenient to work locally with Doxygen.  Even
though Jenkins ignored these warnings, if you ran Doxygen locally, it
produced a long list of warnings in the log file, making it difficult to
spot if new warnings had appeared.

Also improved the Doxygen configuration files to have common settings in
a shared file that is then included from the actual configuration files.

Change-Id: Ic57576e2cb89f6d0e5929981553b48a61435c26c

Fixes for FFT libraries

- Updated documentation
- Added gmx_fft_cleanup for any cleanup after using FFT
    (removing memory leak otherwise caused by FFTW)
- Replaced TMPI_THREAD_MUTEX_INITIALIZER with tMPI::mutex
    (otherwise tmpi leaks memory)

Change-Id: If3e9012da14ebf74d87d4d4647c2f17bd8380fc1

Improve ProgramInfo and use it everywhere.

- The ProgramInfo class now stores the full command line.
  Moved functionality to add quotes to arguments with spaces from
  gmx::test::CommandLine to ProgramInfo.
- ProgramInfo initialization is now protected by a mutex for
  completeness.
- oenv.c and statutil.c now use ProgramInfo internally to store/return
  the binary name and command line.  Removes duplicate implementations
  of this functionality.  Required changing the return values of
  ProgramInfo methods from std::string values to const references.
- Removed many unnecessary #include directives from these two files.
- Fixed warnings that were produced from these files when switching them
  to C++ compilation (one cppcheck warning suppressed for now).
- Add a temporary hack for Windows to try to make get_libdir() work for
  tests that are run through CTest.

Helps with #950.

Change-Id: I1bfd4231b8b7055d0a014b41be67a7c1c99e36b0

Error/exception handling improvements.

- Change formatErrorMessage() to printFatalErrorMessage() that directly
  prints the error message for an exception into a FILE (typically
  stderr) instead of creating a std::string.  This makes it possible to
  handle out-of-memory errors internally within the function.
  The function is much easier to use if we can guarantee that it never
  throws.
- Add basic handling of std::bad_alloc into the above function.
- Resolve clang warnings in the function.
- Add a macro for conveniently catching all exceptions at C++ - C
  boundary and reporting them as fatal errors.
- Initialize ProgramInfo in all C++ binaries.

Related to #838.

Change-Id: I77c355480485f563c57b6c6ea9b07a8ce7f433a8

Always build thread_mpi threading into libgromacs.

This allows using basic thread synchronization primitives without
conditional compilation.
Adjusted the build system to build only the basic thread synchronization
files instead of full thread_mpi if GMX_THREAD_MPI is off.
This required splitting tMPI_Malloc and friends from tmpi_init.c into a
separate file (since tmpi_init.c depends on most other files), and also
moving TMPI_COMM_WORLD to the new file.
Also removed a few unused variables from atomic.h to avoid cppcheck
warnings.

Part of #948.

Change-Id: I3c6bd2cf084acd4a4e881b99a07bc3a4c878727e

Remove more underscore-prefixed C++ symbols.

More of the same as in e39fb5f (I7f4bf42e), no manual editing this time.
Now only trajectoryanalysis/modules/ contains these, will change that
separately as doing it here would create conflicts with I67e123f8.

Change-Id: Ic730a5f906b365212cd20da7b27d9c7b79316053

Merge "Make C++11 test quiet after the first cmake run."

Merge "Use VERSION_LESS in the version detection code."

Use VERSION_LESS in the version detection code.

There was a comment in the cmake script that suggested this, and it is
already used elsewhere in the cmake code.

Change-Id: Ic63c52510d37b864f2f39c8bc2f3806783405cc2

Fix for boost version check added by 6b4c9bc

Change-Id: I96fdece894d0e0c767aee156f5e00016bf14a0e1

Update CPack package version.

Updated also the CPack package version to 5.0.

Change-Id: I7ba4af3fb697527e5007c5bc09df8543e5f371d5

Make C++11 test quiet after the first cmake run.

Change-Id: I08b03aad4980d748d67acc6cc48150afea6f6deb

Merge "Make sure that found Boost version is new enough"

Merge "Merge remote-tracking branch 'origin/release-4-6'"

Make sure that found Boost version is new enough

FindBoost only checks the first time cmake is run that the
available boost version is at least the required boost version.
If Boost_INCLUDE_DIR is already set from a prior cmake run,
the version isn't checked again. And thus it tries to use the
older external version.

Change-Id: I9897b49c895bb0263d7013b74afb54b921d077d6

Merge "Merge remote-tracking branch 'gerrit/release-4-6'"

Merge remote-tracking branch 'origin/release-4-6'

Change-Id: I0c181ad8eb86d0cd130ea27b0cd896f5db581ab5

Merge "Fix mdp_opt formatting of expanded ensemble" into release-4-6

Merge remote-tracking branch 'gerrit/release-4-6'

Moved files added to include/ to src/gromacs/legacyheaders/ and
files added to src/gmxlib/ to src/gromacs/gmxlib/.

Conflicts:
    CMakeLists.txt
    src/config.h.cmakein

Conflicts for files removed from master:
    cmake/gmxTestInlineASM.cmake (resurrected)
    cmake/TestInlineASM_gcc_x86.c (no conflict, resurrected)
    src/gmxlib/CMakeLists.txt
        (changes applied to src/gromacs/CMakeLists.txt and
        src/gromacs/gmxlib/CMakeLists.txt)

Conflicts for files removed from release-4-6:
    src/gromacs/gmxlib/nonbonded/nb_kernel_ia32_sse/*
    src/gromacs/gmxlib/nonbonded/nb_kernel_ia32_sse2/*
    src/gromacs/gmxlib/nonbonded/nb_kernel_ia64_double/*
    src/gromacs/gmxlib/nonbonded/nb_kernel_ia64_single/*
    src/gromacs/gmxlib/nonbonded/nb_kernel_ppc_altivec/*
    src/gromacs/gmxlib/nonbonded/nb_kernel_x86_64_sse/*
    src/gromacs/gmxlib/nonbonded/nb_kernel_x86_64_sse2/*
    src/gromacs/legacyheaders/gmx_sse2_double.h
    src/gromacs/legacyheaders/gmx_sse2_single.h

Change-Id: I6202a89a802a8fd3e1788df95eac408cf60f3456

Fix mdp_opt formatting of expanded ensemble

Was introduced with c7a82654f2e. Broke the build of the manual
because dl tag wasn't closed.

Change-Id: I86afd4a2bdc638fb910cb0afa827b3f56447bb36

Fix illegal usage of MPI_Comm_create

Bug was introduced by 8cd25f0d45567
Illegal to call MPI_Comm_create with different group arguments

Change-Id: Ia70f81862fa50b03ab127e8796226369f6732942

Merge "Simple patch for openmm_wrapper.cpp" into release-4-6

Increase coverate of selection unit tests.

Add some tests and adjust existing tests to cover parts of the selection
compiler and evaluation functions that the coverage run showed as not
covered.
Fixed one issue found (can be backported to 4.5 if necessary).

Part of #651.

Change-Id: If92b5b1eb8cf6b3acb510ba43f64fc82b63ab82a

Fixed gcc inline assembly issue with PIC and older gcc compilers

Some gcc versions had problems with the recently introduced inline
assembly for cpuid (clobbering of ebx register), and since we used
a slightly different inline assembly instruction test in cmake we
did not detect it properly. This patch both makes the inline asm
itself much more portable, and we also test with exactly the same
instruction in cmake so we don't set the flag for supporting inline
asm unless we know this instruction works.

Change-Id: Ib479e2706024abb5e3f66d0249291ce55f1257b1

Make the selection parser exception-proof.

The parser now catches and stores any exception that occurs during
parsing, terminates the parser cleanly, and rethrows the exception.

The functions called from the parser are still not exception-safe, and
would require quite some work to make so, because they are code written
originally in C and do a lot of dynamic memory allocation, returning
some of the allocated memory to the caller.  But the generated code
should now work with exceptions.

Fixes #880.

Change-Id: I2d534d106a6fe20d949b3f24b0d7f9980c545fcf

Use push parser for selection parsing.

The push parser is new in Bison 2.4 (current parser.cpp files generated
with 2.5), and has two benefits in our case:
- Clearer control flow, since the interactive prompt loop is now outside
  the parser, so it doesn't need to behave like a state machine.
- Allows better exception handling: since it is possible to directly
  return from Flex actions, we should also be able to throw exceptions.
  But without this change, they would go through the generated part of
  the Bison parser, causing memory leaks since the code is generated for
  C.  With this change, the Flex scanner is called directly from C++
  code, eliminating this problem.

Since the Bison and Flex parsers are now decoupled, it should also be
possible to clarify the code further, by having a separate data
structure for parser-specific data instead of using yyscan_t.  For now,
kept the changes to a minimum.

Related to #880.

Change-Id: I43ea11561d3dd0577dd95eae3794ca621953d038

Add string utility functions for suffix handling.

Added endsWith() and stripSuffixIfPresent() functions to stringutil.h.
Allows for much clearer and easier-to-write string manipulation code
where this kind of functionality is needed.

Change-Id: I853234ce3d8a11b88ee50225a66e061302e10de7

Rename format.h to more generic stringutil.h.

There were already some string helper functions not directly related to
formatting, and this header is a natural place to add more of such
things.

Change-Id: I46f7bd46393d27e49d46589b8d7b5c2362bcdc8c

Clean up option and selection flags.

Rename the enums used for flags such that they are more likely unique.
Also a bit of other clean-up to limit the use of the option flags more
into the generic parts.  Removed a few unnecessary flags.  Made
efClearOnNextSet work more correctly for classes that don't use
OptionStorageTemplate, although currently it didn't cause any issues
(noticed some issues while developing SelectionFileOption, but the final
version worked without these fixes).

Change-Id: I78c243c4f73b87b285eb54d6ff3ae42699678962

Remove (part of) underscore-prefixed C++ symbols.

This commit is mostly the result of (using GNU sed syntax)
  sed -i -re 's/\<_([a-zA-Z]+)\>/\1_/g'
on the affected files. Some additional manual renaming was done.
optionsassigner.cpp was manually edited a bit more to update comments
and to remove unnecessary use of bit flags.
Also added a few 'explicit' keywords to constructors.

selection/ and trajectoryanalysis/ directories still contain a lot of
these, but this commit is large enough as it stands.

Change-Id: I7f4bf42ec5d9d762fefcff289e38b6b6b231b82b

gmx_dyecoupl: check selected groups from user

check if the user accidentally selected the same group twice. Abort
with error, since this will not produce any meaningful results.

Change-Id: Iace69c473e5f1b815e35d20a39845533297ad208

New CPU detection & AVX/SSE code, removed raw assembly files.

Removed all raw assembly files and deprecated altivec support.
Removed support for NASM and other assemblers, and replaced
previous SSE detection code with a new module using CPUID instead.
Added detection for SSE2, SSE4.1, AVX 128-bit with FMA, and AVX 256-bit.
Added Cmake detection of build platform based on CPUID, and output this
to the log file. The executables now compare the compile-time platform
and selected acceleration with the run-time platform and most suitable
acceleration and warns the user if they do not match. The compiler
detection code has also been reordered slightly to produce more readable
warnings when OpenMP is not available, and correctly disable pragma
warnings.

Added intrinsics code and math functions for SSE2, SSE4.1, AVX128/256
both in single and double precision. All math functions and permutation
code have been tested & verified. Single precision math functions are
correct apart from the least significant bit, and double precision has
roughly twice the accuracy.

This has forced me to temporarily disable the SSE & Fortran acceleration.
SSE will be added back soon based on new intrinsics-only kernels currently
in testing, and we will test if Fortran still makes sense then.

Finally, the patch includes a modification to gmx_rmsdist where
a regression issue was introduced recently by using sqrtf() for
the norm function. This caused the intel compiler to produce slightly
different results at high optimization leves, which got evident here.

Closes #926 - Raw assembly code has been removed.
Refs #923 - Old kernels removed, new will be added shortly.
Fixes #914 - Cmake now does architecture-speficic optimization.
Fixes #912, #913
Fixes #857 - We detect rdtscp support with CPUID and use it if possible.
Fixes #750
Closes #537, #574 - Altivec is now deprecated.

Change-Id: Icfca5a940762f8d82ae67b59c65b2d2ac683256d

Remove unnecessary "move" on shared pointer

Change-Id: I79d2d01307c014f4d40cfffc9dbd04067ed4f0fc

Simple patch for openmm_wrapper.cpp

The naming of several variables is changed to be consistent with the
latest free energy changes.  Allows mdrun-gpu to be compiled, which
otherwise fails (as reported on gmx-developers).

Change-Id: I497ba07ee6883ef78d23022450f6c3d971b2127e

Merge "Remove unnecessary *-impl.h files."

Updates to selection online help.

Some of the help topics were outdated after changes introduced by the
linked Redmine issues (some of them even before that).  Improved the
situation a bit and also did some misc. other documentation fixes.

Related to #656 and #666.

Change-Id: I7f18ca4ad8ce2aa5cd808281bab044253ca77f56

Remove unnecessary *-impl.h files.

Originally, I put all class declarations in headers, but for private
implementation classes used only from one source file, it just leads to
additional maintenance without much benefit.  Moved such class
declarations to the source files (quite a few private implementation
classes are already declared in the source files).

Removed some unnecessary destructors, and had to adjust some includes,
otherwise this change just copy-pastes code around.

Change-Id: Icfb30f88f593806971746d14e3fc0d9b5e7caa96

Merge "Merge branch 'release-4-6'"

Renamed TestTemporaryFileManager to TestFileManager

Made utility functions static members of class

Change-Id: I706a9c4b5d6e8fe4f121d1e50d229b03f492e49d

Testutils improvements for arrays and parameterized tests

- For parameterized tests replace / with _ to have all refdata
  xml files in one folder
- Print index in sequence in error message for value/refValue
  mismatch

Change-Id: Ic593b87f9c1e4cf2ea049044e48a705eca0aa839

Merge branch 'release-4-6'

Conflicts:
src/gromacs/gmxlib/dihres.c
src/gromacs/legacyheaders/dihre.h
src/gromacs/mdlib/force.c
src/gromacs/mdlib/mdebin.c
src/gromacs/mdlib/sim_util.c
src/programs/mdrun/md.c

Change-Id: I0c5e6d8e57f9f19da73a81737c3105dcd55488ec
Moved:
     src/mdlib/expanded.c ->
       src/gromacs/mdlib/expanded.c

Merging in free energy, exp. ensemble, & andersen t-control to 4.6

Conflicts:
src/gmxlib/mvdata.c
src/gmxlib/txtdump.c
src/kernel/readir.c
src/mdlib/sim_util.c
src/tools/gmx_membed.c
include/mdrun.h
src/kernel/md.c
src/kernel/md_openmm.c
src/kernel/md_openmm.h
src/kernel/runner.c
src/mdlib/minimize.c
src/mdlib/tpi.c
src/tools/gmx_membed.c
src/kernel/md.c

Change-Id: Ide69a8dbfb96072d5c55d0a051dc9db788cbe144

Merge release-4-6 into master

Conflicts:
src/mdlib/CMakeLists.txt
          change applied to src/gromacs/mdlib/CMakeLists.txt
src/tools/gmx_tune_pme.c
  trivial conflict

Moved:
src/mdlib/fftpack.c & src/mdlib/fftpack.h to src/external/fftpack

Fixed paths in COPYING-OTHER and added boost and gtest/gmock.

Moved the copyright header and list of modifications from fftpack.c into
new README file. Reduces modifications to original file.

Fixed header location in src/gromacs/mdlib/gmx_fft_fftpack.c.

Added macros.h to g_tune_pme (fixing missing min/max)

Change-Id: I9324b1695412e21d16bdcb4d2dc316c53356da5f

thread_mpi C++11 compatible mutex classes

This brings the master branch thread_mpi up to date, and introduces
a C++11 compatible API for mutexes and lock_guards (found in
thread_mpi/mutex.h).

Partially fixes http://redmine.gromacs.org/issues/948

Change-Id: Ibce48ae10e84bedee9fe641b94a21f6674efcd6b

Merge release-4-5-patches into release-4-6

Changes in 4.5 for removed lines/files in 4.6:
Makefile.am
include/domdec.h
include/domdec_network.h

Trivial conflict:
include/types/commrec.h

Fixed GMX_THREADS into GMX_THREAD_MPI for those added in 4.5 in:
        include/mdrun.h
        src/gmxlib/shift_util.c
        src/mdlib/gmx_fft_fftw3.c

Change-Id: Id37bfbbacc945a7cf184fa21313c05e2680e1fba

Merge "Use External BLAS/LAPACK by default" into release-4-6

Merge "Replace our fftpack version with Numpy's version" into release-4-5-patches

Replace our fftpack version with Numpy's version

Fixes incorrect results for vectors with length of multiples of 12

Fixes #946

Change-Id: I9a110de7334cbb96a72d0ced73ade4d55f665fee

More config.h cleanup.

Remove defines from config.h for standard types:
 - gid_t, never used.
 - uid_t, added an ifdef around the single use, should always be present
   if getuid() is.
 - off_t, was only used if HAVE_FSEEKO was set.

Also removed the code to check these types from CMake, as well as the
detection code for size_t (define already removed in Icc5bba87).

Change-Id: I2835b0bc0a524a30464230d65a2d873b611ed350

Make get_libdir() robust when run from build tree.

get_libdir() now uses the share/top/ directory in the original source
tree if the binary is run from the build tree.  Makes it easier to run
tests.  GMXLIB environment variable can still override this detection.

Change-Id: I74a5d3ba5ff02d68e35ab169b5466d411bad2ae1

Fix use after free in g_anaeig projection code.

PBC removal structure was deleted immediately after it was initialized,
and used afterwards.

Change-Id: I6126967b34e8de289370dede79cc50a9924ac34d

Fixed some memory leaks.

- gmx_rmpbc_done() is now symmetric to gmx_rmpbc_init() in that it frees
  also the gmx_rmpbc_t pointer.
- gmx_fio_fclose() frees the memory for the removed t_fileio structure.
- gmxfio functions free t_fileio::fn when they free the structure.

Change-Id: I0d2d667d915109f8a2670aea21326d714ae25adb

Removed dependency on windows.h through commrec.h, domdec.h.

Change-Id: If40506b521a4b4690506465a3f421dab93bc9ce6

Merge "Revised g_tune_pme" into release-4-6

Improve 'g_ana help'.

- 'g_ana help' now internally uses the new online help classes.
  Without any additional arguments, it still prints out the list of
  commands, and now also list of additional help topics.
- 'g_ana help <command>' prints help for the given command (the same as
  'g_ana <command> -h' for the trajectory analysis modules).
- 'g_ana help selections <subtopic>' can be used to access the general
  help for selections.

IssueID #666.

Change-Id: Id34044842b297591923bef242a93b4ae1ae9e14e

Generic handling of online help topics.

- Added a few simple classes for handling a tree of help topics
  (src/gromacs/onlinehelp/).
- Rewrote the selection help to use the new classes. Required changing
  the help parser slightly to pass all requested help topics in a single
  call to _gmx_sel_handle_help_cmd(). Although this changes the usage of
  the help slightly, did not yet update the help texts.
- Added a way to specify the bison and flex binaries to use for
  regenerate_parser.sh (useful, e.g., if multiple versions are
  installed).

Related to #666.

Change-Id: I318fc0fc42e4af39734a5aa65a503582302ecdd8

Additional features to TextTableFormatter.

Add possibility to print out a table without a header (if there are no
column titles) and possibility to indent the whole table.
Use the new features for formatting the list of commands in
CommandLineModuleManager.

IssueID #666

Change-Id: I2d5aaceaf1c5545ff3be215d27ec30df38fbacde

Separated string formatting used only for help.

Moved string formatting routines used only in help printing to
src/gromacs/onlinehelp/ from src/gromacs/utility/format.*.
Also moved one function from cmdlinehelpwriter.cpp to the new file.
Subsequent commits will add more content to the new directory and will
use the same string formatting routines.

Added NOMINMAX define for Windows to avoid min/max macros.

Related to #666.

Change-Id: I9b52012df2fc718bc3d68c697140661a241a1467

Reorganize linear algebra code.

- Moved gmx_arpack.* and internal BLAS/LAPACK implementations to
  src/gromacs/linearalgebra/.
- Moved all routines that directly depend on these to the same dir.
  Also moved integrally related routines (sparsematrix.*, mtxio.*).
- Some reformatting of headers.
- Adjusted Doxygen settings to resolve warnings in these files.
- Adjusted #include statements to make things compile.

With this, the internal BLAS/LAPACK/ARPACK is completely hidden within
libgromacs, and these headers don't need to be installed.  Removes
dependency on F77_FUNC (and hence on config.h) from installed headers.
Also makes it possible to hide the BLAS/LAPACK/ARPACK symbols if
necessary (not done in this change), such that Gromacs compiled with the
internal versions could be linked into a program that uses external
BLAS/LAPACK without symbol conflicts.

Change-Id: I9b2b1ea9832e307a29f5c5038afe1af5504223d9

Merge "Continuing to fix threading on Cygwin" into release-4-5-patches

Merge "Improve gmx::test::CommandLine."

Merge "Improve analysisdata reference checks."

Merge "Move common test code to testutils."

Merge "Simplify selection parsing methods signature."

Merge "Option parsing through SelectionOptionManager."

Merge "Simpler required selection input from a file."

Merge "Split option code away from SelectionCollection."

Merge "Merge release-4-6 (commit 'Ic142a690')"

Merge "Fixed 'g_ana select' broken by I8806b3e0."

Fix editconf real-bool misusage

Bool variable was declared as real. Caused invalid stack read
in double precision. Could have caused wrong value to be printed
for argument display but couldn't have caused wrong behavior.

Found with AddressSanitizer

Change-Id: I32553b1a898fb3a7f072ae177518dd63fad56dde

Merge release-4-6 (commit 'Ic142a690')

This merges changes more than resolving a conflict.
The reason is that Ic142a690 moved the files to a location
not suitable for master.

molfile_plugin.h, vmddlopen.h, vmdio.h, and vmdplugin.h
were in
- "include" before Ic142a690
- "src/gmxlib" after Ic142a690
- "src/gromacs/legacyheaders" in master

Moved all but vmdio.h to "src/external/vmd_molfile"
Moved vmdio.h to "src/gromacs/gmxlib"

Moved vmddlopen.c (in src/gmxlib or src/gromacs/gmxlib) also to
"src/external/vmd_molfile" (was not moved by Ic142a690).

Removed "#ifdef GMX_USE_PLUGINS" from vmddlopen.c
This makes the files in src/external/vmd_molfile identical to
VMD 1.8.6 besides the Copyright notice.

Instead of the #ifdef, only compile vmdio.c and vmddlopen.c if
GMX_USE_PLUGINS is set (in src/gromacs/gmxlib/CMakeLists.txt)

Trivial conflict in:
src/gromacs/mdlib/pull_rotation.c

Change-Id: If9bda639e4f680c09d785b046fc0382c16688beb

Merge "added explicit sqrtf to the float version of norm" into release-4-6

Fix Clang 3.1 warning

grid_spacing is an integer and thus it was always set to 0

Change-Id: Id10bccdadef8f2630631d638e94bd27d78774536

Continuing to fix threading on Cygwin

Fixes #833

Change-Id: I5d577a149ad18b295559f6d87c24dc42bffd281e

Make load_vmd_library static

Was already used only within the same file

Change-Id: I43276424d51021bc8247c04a3c3647f7fc1565f7

Made all vmdio headers private

- Made vmdio struct in trx a pointer
- No need for vmdio to be in the public API

Change-Id: Ic142a6900d0cbc803e8c24139efbfbc072b46c22

Merge "fixed [dd] and [TT] typos in documentation of mdrun and gmx_sans" into release-4-6

Merge "grompp now retains the order of bonds converted to constraints" into release-4-6

fixed [dd] and [TT] typos in documentation of mdrun and gmx_sans

Change-Id: Ia51f67b2b056f66072875d2c7a366ae4685bf9d4

Merge "Fix Clang 3.1 compile error"

Merge "Fix that sigaction mask wasn't zeroed" into release-4-6

Fix Clang 3.1 compile error

Change-Id: I3cdfed9168c8ea1b1ca29de0efba071e1f73b9e4

now mdrun always initializes MPI

MPI was only initialized with more than one process. This required
checks everywhere. Now MPI is also initialized with one process.

Change-Id: I4d0f543d5c1ae5a6136a0ba34f0f83ac27604b14

Merge "Code cleanup to avoid name collisions" into release-4-6

Improve gmx::test::CommandLine.

- Make the class properly copyable for easier use in tests that need to
  manipulate it.
- Make the class manage its memory using std::vector and prevent memory
  leaks if the argv array is modified outside the class (the internal
  state of the class is still not correctly maintained, though).
- Add methods to append arguments.

Needed for #920, but split from the main change to keep it smaller.

Change-Id: I66ebe9d48936d35bce9e0b4fc1d6246d3d5ec07d

Improve analysisdata reference checks.

The reference checking for AnalysisData objects now supports more
scenarios with multipoint data, i.e., cases where there are point sets
that do not cover all columns.

Supporting changes:
 - Add a TestReferenceChecker::checkPresent() method that allows to
   assign meaning to absence of a reference data item (and to check for
   that).
 - Split common analysisdata XML stylesheet elements to a separate xsl
   file.
 - Add a script for copying the XML stylesheets to the reference data
   directories for easier maintenance.  Master copies of shared
   stylesheets are now kept in src/testutils/.

Needed for #920, but split from the main change to keep it smaller.

Change-Id: I108c013833052c37a4f2e6c73be6161c0dae1c78

Move common test code to testutils.

- Move test fixture for AbstractAnalysisData classes from analysisdata/
  to testutils/.
- Some renaming to make file and class names more descriptive.
- Move gmx::test::CommandLine from commandline/ to testutils/.

Prerequisite for #920.

Change-Id: I2d4c8aaf87db03fe6c170856ba5b41eae05c483b

Revised g_tune_pme

1. Now, g_tune_pme only duplicates mdrun options where necessary,
options not understood by g_tune_pme are simply passed over to mdrun.
2. Users can now set minimum and maximum Coulomb radius iso scaling
factors for rcoulomb and the Fourier grid.
3. As a result of 1 and 2, some cleanup of the code was possible and
unneeded code was removed.
4. For -npme auto, the tool now derives a reasonable number of PME-
only nodes from the .tpr file settings and tests values around this
number.
Change-Id: I88af5d38a61e0409613f5f3fd4272f7f43823f64

Fix that sigaction mask wasn't zeroed

Change-Id: I84a1f77e6320538730cc367ed722eadf22046a68

Code cleanup to avoid name collisions

- Adding missing prepropessor guards
- Renaming functions defined using same name as other function
- Removing unused magic.h

Change-Id: I1375097eb7eced2dafb30222c9986ac24f45fb08

Use External BLAS/LAPACK by default

Check if Blas/Lapack is available and only use internal
if it isn't found

Updated FindLAPACK to 2.8.8. and included FindBLAS
so that it works without Fortran enabled.

Change-Id: I6d1df8a03196d9c14436d5b70c731d36b4ddf823

added explicit sqrtf to the float version of norm

Additionally, as inlining is not a concern nowadays, also cleaned up the
code by using iprod in norm.

Change-Id: Iabc237906380f848e16ed158809bf22b1de9ddd0

Fixed 'g_ana select' broken by I8806b3e0.

The loop for creating the output for -om is now even simpler and easier
to understand.

Change-Id: If1bb1eb09c40562a90effcfffb4da32de28d0a7a

grompp now retains the order of bonds converted to constraints

With the constraint mdp option grompp used to write tpr files where
the order of the constraints was reversed with respect to the order
of the bonds. This reordering has been removed.

Change-Id: I094c5001ccad1e80f46b61c4fbcfa400fd5c1e74

re-introduced fix for real/double issue in gmx_density

The float density histogram could overflow for large data sets.
A fix replaced 3 of the 4 pointers to floats by pointers to doubles, leading
to compilation issues. In two steps these replacements were reverted.
Now all 4 pointers have been replaced.

Change-Id: I0d8e38d27ac5c469480ff731ad191d869f5d890f

Remove unused defines from config.h, part 3.

- C89 defines size_t in stddef.h, even for free-standing
  implementations, we probably don't need to define our own fallback
  (and it probably doesn't work in any case).
- F77_FUNC_, the underscored version is not used anywhere.  Left it's
  definition in CMakeLists.txt as commented.
- TMPI_WINDOWS_NUMA_API, not used in source files nor in CMake.
- HAVE_LIBM and GMX_LARGEFILES, not used in any source file.
- HAVE_?MMINTRIN_H, not used anywhere.
- restrict, not used anywhere, and the check was broken.

Change-Id: Icc5bba87973c7a086a01be486a5785faee4c5cad

Remove unused defines from config.h, part 2.

Remove defines that were detected in CMake, but never used, not within
the source files nor within CMake.  Some more of this type remain, but
these seem clear.
Also removed TestRetSigType.c, since the related test is already gone.

Change-Id: I219f9d571b9da06844745a735de631d7c3dc33f9