Cleanup of header files

Removed several not-necessary inclusions

src/Action.h

@@ -3,14 +3,13 @@
 #include <vector>
 #include <string>
 #include <set>
-#include "Log.h"
 #include "Tools.h"
-
-#include "PlumedCommunicator.h"
+#include "Log.h"
 
 namespace PLMD{
 
 class PlumedMain;
+class PlumedCommunicator;
 
 /// This class is used to bring the relevant information to the Action constructor.
 /// Only Action and ActionRegister class can access to its content, which is 

src/ActionWithArguments.h

 #ifndef __PLUMED_ActionWithArguments_h
 #define __PLUMED_ActionWithArguments_h
 
-#include "Action.h"
 #include <map>
 #include <vector>
 #include <cassert>
-
-#include "ActionWithValue.h"
+#include "Action.h"
+#include "Value.h"
 
 namespace PLMD{
 

src/ActionWithValue.h

 #ifndef __PLUMED_ActionWithValue_h
 #define __PLUMED_ActionWithValue_h
 
-#include "Action.h"
 #include <map>
 #include <vector>
 #include <cassert>
+#include "Action.h"
 #include "Value.h"
 
 namespace PLMD{

src/DumpDerivatives.cpp

 #include "ActionPilot.h"
 #include "ActionWithArguments.h"
 #include "ActionRegister.h"
+#include "PlumedCommunicator.h"
 
 using namespace std;
 

src/Log.cpp

 #include "Log.h"
+#include "PlumedCommunicator.h"
 #include <cstdarg>
 #include <cassert>
 #include <cstring>

src/Log.h

@@ -3,10 +3,11 @@
 #include <cstdio>
 #include <string>
 #include <sstream>
-#include "PlumedCommunicator.h"
 
 namespace PLMD{
 
+class PlumedCommunicator;
+
 /// Class containing the log stream.
 ///
 /// It is similar to a FILE stream. It allows a printf() function, and

src/Print.cpp

 #include "ActionPilot.h"
 #include "ActionWithArguments.h"
 #include "ActionRegister.h"
+#include "PlumedCommunicator.h"
 
 using namespace std;
 

src/Tools.cpp

 #include "Tools.h"
 #include <cstring>
-#include <assert.h>
+#include <cassert>
 #include <sstream>
 
 using namespace PLMD;