This commit contains changes automatically generated by plumedbot on Travis-ci.

cc: @gtribello

This commit contains changes automatically generated by plumedbot on Travis-ci.

This commit contains changes automatically generated by plumedbot on Travis-ci.

Using auto when creating a unique_ptr

Notice that this change required a bit of hacking.
setAveragingAction now takes a std::unique_ptr as an argument.

I think ownership is clearer now since it is explicitly declared.

valgrind reported usage of this as an uninitialized value, so I set
it. However, looking at the code below it looks harmless.

This commit contains changes automatically generated by plumedbot on Travis-ci.

I went through cppcheck warnings (style) and fixes those I understood.
Mostly issues related to scope of variables.

Using ...Args it is easy to forward any argument to the constructor.

This allows avoiding to explicitly put "new" statements when initializing
a ForwardDecl. In addition, all ForwardDecl that should be constructed
without arguments can be omitted from the initialization list, which
significantly simplifies the code.

removed extra comma

I changed all the forward references to unique_ptr
using an auxiliary class (ForwardDecl).

Notice that in this manner the order of destructors
is exactly the reverse of the order of constructors.
This forces me to move a variable up in the layout
of PlumedMain to avoid a segfault.

This change is necessary to make plumed exception-safe.

Regenerated after c40b96fe

Changed histogram so that it is calculated by default using Giovanni's formula for reweighted histogram

[makedocs]

Notice that almost all files were updated with the astyle commit
2813d759

Year in header is here updated to 2017

This is a huge commit, applying indentation to the whole
code. Notice that this commit is made on master branch
after having merged with -s ours from branch v2.3

See #215

This is a huge commit, applying indentation to the whole
code.

See #215