Improve dryrun capabilty

[prckent]

Proposed changes

dryrun will exit cleanly (zero error code) rather than QMCPACK aborting. Small edits to docs and comments.

  Hamiltonian h0
  Kinetic         Kinetic energy
  LocalECP        CoulombPBCAB potential source: ion0
  NonLocalECP     NonLocalECPotential: ion0
  ElecElec        CoulombPBCAA potential: e_e
  IonIon          CoulombPBCAA potential: ion0_ion0

  dryrun == 1 : Skipping all QMC and loop elements 

Use --enable-timers=<value> command line option to increase or decrease level of timing information
Stack timer profile
Timer      Inclusive_time  Exclusive_time  Calls       Time_per_call
  Startup    54.0524    54.0524              1      54.052436786

QMCPACK execution completed successfully

What type(s) of changes does this code introduce?

• Bugfix
• New feature
• Documentation or build script changes

Does this introduce a breaking change?

• No

What systems has this change been tested on?

Ubuntu 22.04 gcc 11.4.0

Checklist

• Yes. This PR is up to date with current the current state of 'develop'
• Yes. Code added or changed in the PR has been clang-formatted
• No. This PR adds tests to cover any new code, or to catch a bug that is being fixed
• Yes. Documentation has been added (if appropriate)

[ye-luo]

Also please remove the commented lines of code.

[prckent]

Done

[ye-luo]

How about making an early return instead of keep a giant scope of 'else'.
I learned this from https://llvm.org/docs/CodingStandards.html#use-early-exits-and-continue-to-simplify-code

[prckent]

Sure, it is a matter of style. I prefer the early exit but some people do not.

[PDoakORNL]

If method is of sane size early exit should not present difficulty reading.

[prckent]

Test this please