Adds some redundant logic, but makes CLANG reported problem go away.

Now that the TRES logic is working, remove vestigial burst buffer
  configuration parameters: JobSizeLimit, UserSizeLimit, PrioBoostAlloc,
  and PrioBoostUse.
Also remove the PrivateData configuration parameter and use a Flag
  value of PrivateData instead.
Stop printing a bunch of configuration parameters that are not
  applicable to the cray plugin (e.g. CreateBuffer script).

job_mgr.c for keeping memory from another job.

acct_policy_job_runnable_post_select to validate TRES like bb and lic

This could probably be moved to the pre_select but this was easy and
quick for the time being.

limits on startup.

instead of always working on the job_ptr->tres_req_array

The sacctmgr now accept "resource" and "reservation" as arguments,
  and some calls in the test just used "res", which now failed.
  Expanded some function names for clarity and changed the sacctmgr
  arguments to spell out "resource" rather than just "res".

Change some code format from Linux kernel to K&R, for better or worse...

of the whole type

duplicate code.