scr: return false from need_checkpoint if user does not enable conditions

Author: adammoody

doc/rst/users/api.rst

@@ -271,6 +271,11 @@ Upon returning from :code:`SCR_Need_checkpoint`,
 and it is set to :code:`0` otherwise.
 The call returns the same value in :code:`flag` on all processes.
 
+Various configuration settings determine the conditions
+in which :code:`SCR_Need_checkpoint` sets :code:`flag` to 1.
+See :code:`SCR_CHECKPOINT_INTERVAL`, :code:`SCR_CHECKPOINT_SECONDS`,
+and :code:`SCR_CHECKPOINT_OVERHEAD` in :ref:`sec-config`.
+
 SCR_Start_output
 ^^^^^^^^^^^^^^^^
 

src/scr.c

@@ -3092,15 +3092,6 @@ int SCR_Need_checkpoint(int* flag)
         }
       }
     }
-
-    /* no way to determine whether we need to checkpoint, so always say yes */
-    if (!*flag &&
-        scr_checkpoint_interval <= 0 &&
-        scr_checkpoint_seconds  <= 0 &&
-        scr_checkpoint_overhead <= 0)
-    {
-      *flag = 1;
-    }
   }
 
   /* rank 0 broadcasts the decision */