anchor.diff:      public :: dump_restart_hdf5, read_restart_hdf5, restart_info_global
   
       type restart_info_t
  -!REVIEW-AJWT:  The comments in parse_input are not helpful in this regard!  More please.
  +! [review] -  AJWT: The comments in parse_input are not helpful in this regard!  More please.
           ! See comments in parse_input regarding the read_id and write_id.
           integer :: write_id ! ID number to write to.
           integer :: read_id  ! ID number to write to.
  
positive.code: 
      ! The HDF5 structure we use is:
  
  !REVIEW-AJWT: Does order matter?
  !REVIEW-AJWT: How do we keep this specification consistent with what is actually done?
      ! /                                # ROOT/
      !
      !  metadata/
      !           restart version        # Version of restart module used to produce the restart file.
      !           hande version          # git sha1 hash.  For info only (not used).
  !REVIEW-AJWT: Probably best to say 'not currently used on read-in'
      !           date                   # For info only (not used).
      !           calc_type              # Calculation type (as given by a parameter in calc).
      !           nprocs                 # Number of processors used in calculation.
  
negative.code:                 call read_integer(group_id, dcalc, calc_type_restart)
  
  
  ! [review] -  AJWT: While bit strings are nice, I think the code below lacks modularity.
  !              I foresee a time when the calc_type format will change, and the code
  !              below will be a pain.  Perhaps some sort of interface for this.
  !             In particular, I don't think the restart_read should be dealing with this sort of thing.
  