Better formatting output

For a few actions that typically write long lists,
I fixed the format to write only 25 fields per line.

src/colvar/Gyration.cpp

@@ -149,10 +149,13 @@ Gyration::Gyration(const ActionOptions&ao):
   if(rg_type>TRACE) log<<"  Bibliography "<<plumed.cite("Jirí Vymetal and Jirí Vondrasek, J. Phys. Chem. A 115, 11455 (2011)"); log<<"\n";
 
   log.printf("  atoms involved : ");
-  for(unsigned i=0; i<atoms.size(); ++i) log.printf("%d ",atoms[i].serial());
+  for(unsigned i=0; i<atoms.size(); ++i) {
+    if(i%25==0) log<<"\n";
+    log.printf("%d ",atoms[i].serial());
+  }
   log.printf("\n");
 
-  if(!nopbc) {
+  if(nopbc) {
     log<<"  PBC will be ignored\n";
   } else {
     log<<"  broken molecules will be rebuilt assuming atoms are in the proper order\n";

src/generic/Group.cpp

@@ -175,11 +175,14 @@ Group::Group(const ActionOptions&ao):
   parseAtomList("REMOVE",remove);
   if(remove.size()>0) {
     std::vector<AtomNumber> notfound;
+    unsigned k=0;
     log<<"  removing these atoms from the list:";
     for(unsigned i=0; i<remove.size(); i++) {
       const auto it = find(atoms.begin(),atoms.end(),remove[i]);
       if(it!=atoms.end()) {
+        if(k%25==0) log<<"\n";
         log<<" "<<(*it).serial();
+        k++;
         atoms.erase(it);
       } else notfound.push_back(remove[i]);
     }
@@ -205,8 +208,11 @@ Group::Group(const ActionOptions&ao):
   }
 
   this->atoms.insertGroup(getLabel(),atoms);
-  log.printf("  list of atoms ");
-  for(unsigned i=0; i<atoms.size(); i++) log<<" "<<atoms[i].serial();
+  log.printf("  list of atoms:");
+  for(unsigned i=0; i<atoms.size(); i++) {
+    if(i%25==0) log<<"\n";
+    log<<" "<<atoms[i].serial();
+  }
   log.printf("\n");
 }
 

src/vatom/COM.cpp

@@ -94,7 +94,10 @@ COM::COM(const ActionOptions&ao):
   parseFlag("NOPBC",nopbc);
   checkRead();
   log.printf("  of atoms");
-  for(unsigned i=0; i<atoms.size(); ++i) log.printf(" %d",atoms[i].serial());
+  for(unsigned i=0; i<atoms.size(); ++i) {
+    if(i%25==0) log<<"\n";
+    log.printf(" %d",atoms[i].serial());
+  }
   log.printf("\n");
   if(!nopbc) {
     log<<"  PBC will be ignored\n";

src/vatom/Center.cpp

@@ -108,20 +108,29 @@ Center::Center(const ActionOptions&ao):
   parseFlag("MASS",weight_mass);
   parseFlag("NOPBC",nopbc);
   checkRead();
-  log.printf("  of atoms");
-  for(unsigned i=0; i<atoms.size(); ++i) log.printf(" %d",atoms[i].serial());
+  log.printf("  of atoms:");
+  for(unsigned i=0; i<atoms.size(); ++i) {
+     if(i%25==0) log<<"\n";
+     log.printf(" %d",atoms[i].serial());
+  }
+  log<<"\n";
   if(weight_mass) {
     log<<"  mass weighted\n";
     if(weights.size()!=0) error("WEIGHTS and MASS keywords should not be used simultaneously");
   } else {
     if( weights.size()==0) {
+      log<<" using the geometric center\n";
       weights.resize( atoms.size() );
       for(unsigned i=0; i<atoms.size(); i++) weights[i] = 1.;
+    } else {
+      log<<" with weights:";
+      if( weights.size()!=atoms.size() ) error("number of elements in weight vector does not match the number of atoms");
+      for(unsigned i=0; i<weights.size(); ++i) {
+        if(i%25==0) log<<"\n";
+        log.printf(" %f",weights[i]);
+      }
+      log.printf("\n");
     }
-    log<<" with weights";
-    if( weights.size()!=atoms.size() ) error("number of elements in weight vector does not match the number of atoms");
-    for(unsigned i=0; i<weights.size(); ++i) log.printf(" %f",weights[i]);
-    log.printf("\n");
   }
   if(!nopbc) {
     log<<"  PBC will be ignored\n";