Remove all #warning statements

3 files changed, 29 insertions, 19 deletions

diff --git a/CarpetDev/CarpetIOF5/src/input.cc b/CarpetDev/CarpetIOF5/src/input.cc
index d42cbd076..b7f25b59f 100644
--- a/CarpetDev/CarpetIOF5/src/input.cc
+++ b/CarpetDev/CarpetIOF5/src/input.cc
@@ -117,7 +117,7 @@ namespace CarpetIOF5 {
       
       F5iterate_grids(path, NULL, grid_iterator, this, NULL, NULL);
       
-#warning "TODO: synchronise all read grid functions"
+      // TODO: synchronise all read grid functions
     }
   
     void read_grid(F5Path *const path)
@@ -191,8 +191,8 @@ namespace CarpetIOF5 {
       cout << indent << "field=" << fieldname << "\n";
       
       interpret_fieldname(cctkGH, fieldname, varindex);
-#warning "TODO: check all variables in the group"
-#warning "TODO: loop over all variables in the group"
+      // TODO: check all variables in the group
+      // TODO: loop over all variables in the group
       if (varindex>=0 and input_var.at(varindex)) {
 
         int major_version, minor_version, release_version;
@@ -205,7 +205,9 @@ namespace CarpetIOF5 {
         assert(iret);
         
         // Do we need to iterate over fragments?
-#warning "TODO: Should instead check whether attribute FIBER_HDF5_TYPEID_ATTRIB exists (existence indicates fragmentation)"
+        // TODO: Should instead check whether attribute
+        // FIBER_HDF5_TYPEID_ATTRIB exists (existence indicates
+        // fragmentation)
         int const is_fragmented = F5Fis_fragmented(path, fieldname);
         cout << indent
              << (is_fragmented ? "fragmented" : "not fragmented") << "\n";
@@ -221,8 +223,10 @@ namespace CarpetIOF5 {
         indent_t indent;
         cout << indent << "ignoring this field\n";
       }
-#warning "TODO: keep track of which fields have been read, and complain about unread ones"
-#warning "TODO: keep track of which part of a field has been read, and complain about unread parts"
+      // TODO: keep track of which fields have been read, and complain
+      // about unread ones
+      // TODO: keep track of which part of a field has been read, and
+      // complain about unread parts
     }
     
     void read_fragment(F5Path *const path)
@@ -440,7 +444,7 @@ namespace CarpetIOF5 {
       fragdesc.reflevel = reflevel;
       fragdesc.map = map;
       fragdesc.component = component;
-#warning "TODO: set timelevel correctly"
+      // TODO: set timelevel correctly
       fragdesc.timelevel = 0;
       fragdesc.imin = fbox.lower();
       fragdesc.imax = fbox.upper();
@@ -599,7 +603,7 @@ namespace CarpetIOF5 {
     // Grid structure
     string gs;
     ReadLargeAttribute(group, grid_structure, gs);
-#warning "TODO: set grid structure"
+    // TODO: set grid structure
     
     herr = H5Gclose(group);
     assert(not herr);
@@ -614,7 +618,7 @@ namespace CarpetIOF5 {
              bool const input_metadata,
              scatter_t& scatter)
   {
-#warning "TODO: not yet implemented"
+    // TODO: not yet implemented
     assert (not input_metadata);
     input_iterator_t iterator(cctkGH, input_var,
                               input_past_timelevels, input_metadata,
diff --git a/CarpetDev/CarpetIOF5/src/iof5.cc b/CarpetDev/CarpetIOF5/src/iof5.cc
index 01033b24b..c9ebb028f 100644
--- a/CarpetDev/CarpetIOF5/src/iof5.cc
+++ b/CarpetDev/CarpetIOF5/src/iof5.cc
@@ -474,7 +474,8 @@ namespace CarpetIOF5 {
       // Keep track of which files could be read, and which could not
       int foundproc = -1, notfoundproc = -1;
       
-#warning "TODO: Store how many processes contributed to the output, and expect exactly that many files"
+      // TODO: Store how many processes contributed to the output, and
+      // expect exactly that many files
       int const myproc = CCTK_MyProc(cctkGH);
       int const nprocs = CCTK_nProcs(cctkGH);
       // Loop over all (possible) files
@@ -501,7 +502,7 @@ namespace CarpetIOF5 {
         
         // Iterate over all time slices
         bool const input_past_timelevels = in_recovery;
-#warning "TODO: read metadata when recoverying parameters"
+        // TODO: read metadata when recoverying parameters
         bool const input_metadata = false;
         input(cctkGH, file, input_var, input_past_timelevels, input_metadata,
               scatter);
diff --git a/CarpetDev/CarpetIOF5/src/output.cc b/CarpetDev/CarpetIOF5/src/output.cc
index 6b7d95b30..c0e8436dd 100644
--- a/CarpetDev/CarpetIOF5/src/output.cc
+++ b/CarpetDev/CarpetIOF5/src/output.cc
@@ -148,7 +148,8 @@ namespace CarpetIOF5 {
           imin[d] = 0;
           imax[d] = lssh[d];
           if (not output_ghost_points) {
-#warning "TODO: Don't output ghosts on refinement boundaries; only output ghosts for inter-process boundaries"
+            // TODO: Don't output ghosts on refinement boundaries;
+            // only output ghosts for inter-process boundaries
             int const overlap = min(ughosts[d], minimum_component_overlap);
             imin[d] += lghosts[d];
             imax[d] -= ughosts[d] - overlap;
@@ -241,7 +242,7 @@ namespace CarpetIOF5 {
         int const do_every =
           group_type == CCTK_GF ? maxtimereflevelfact / timereffacts.AT(rl) : 1;
         if (cctkGH->cctk_iteration % do_every == 0) {
-#warning "TODO: don't switch modes"
+          // TODO: don't switch modes
           ENTER_LEVEL_MODE(cctkGH, rl) {
             DECLARE_CCTK_ARGUMENTS;
             
@@ -264,7 +265,9 @@ namespace CarpetIOF5 {
                   (file, cctk_time, gridname.c_str(), &v2h(reffact)[0],
                    chartname.c_str());
                 assert(globalpath);
-#warning "TODO: Probably must not call this for cell-centred AMR; this probably makes the call to F5Rcreate_coordinate_topology below fail"
+                // TODO: Probably must not call this for cell-centred
+                // AMR; this probably makes the call to
+                // F5Rcreate_coordinate_topology below fail
                 FAILWARN(F5Rlink_default_vertex_topology(globalpath,
                                                          &v2h(reffact)[0]));
                 
@@ -336,7 +339,7 @@ namespace CarpetIOF5 {
                                           topologyname.c_str(),
                                           indexdepth,
                                           mi.dim, mi.dim, &v2h(reffact)[0]);
-#warning "TODO: how should these two topologies be linked?"
+          // TODO: how should these two topologies be linked?
           // assert(mi.dim == 3);
           // assert(all(reffact == 1));
           // path =
@@ -358,7 +361,8 @@ namespace CarpetIOF5 {
       assert(coordpath);
       assert(path);
       
-#warning "TODO: Attach the number of I/O processes to the topology. WB sent email with suggestions."
+      // TODO: Attach the number of I/O processes to the topology. WB
+      // sent email with suggestions.
       
       // Define default topology (once per grid)
       if (group_type == CCTK_GF and reflevel == 0 and timelevel == 0) {
@@ -440,7 +444,7 @@ namespace CarpetIOF5 {
                                  mi.dim, &v2h(mi.gsh)[0],
                                  type,
                                  &vlower, &vdelta));
-#warning "TODO: path and chart don't match"
+        // TODO: path and chart don't match
         FAILWARN(F5Fset_range(path, &vlower, &vupper));
       }
       
@@ -545,8 +549,9 @@ namespace CarpetIOF5 {
       
       assert(groupdata.stagtype == 0);
       
-#warning "TODO: Do not output symmetry zones (unless requested by the user)"
-#warning "TODO: Do not output buffer zones (is that easily possible?)"
+      // TODO: Do not output symmetry zones (unless requested by the
+      // user)
+      // TODO: Do not output buffer zones (is that easily possible?)
       
       int const will_cover_complete_domain =
         (group_type != CCTK_GF or not is_multipatch) and reflevel==0;