Fix code review comments.

Indentation changed to universally use two spaces.
Removed superfluous " \" at the end of wrapped lines.
Cosmetic changes.

Added a space after some commas (where missing).  Other minor code layout changes
to keep lines < 80 characters where practical.
Minor bug fixes within the generated cleanup resource file.

The windows pathname needs to have the '\' separators within the path
escaped.  The kill command cannot be used to terminate the agent
process as it is liable to change (e.g. after a system restart).
Cosmetic changes.

Inserted a space after commas (where it was missing).  Ensured
that line length < 80 characters where practical.
Minor documentation changes in function comments.

Clarified the return value.
Minor bug fix within generated cleanup resource file.

The windows pathname needs to have the '\' separators within the path
escaped.  Also clarified the return status from the "target_exec"
function; returning true/nil seems potentially counter intuitive so
changed to true/false.  The comment was also incorrect (no attempt was
made to return the pid), and the invoking code was checking for the
'nil', which whilst safe re-enforced the belief that a non-nil PID
value might be expected upon success.