wrapper.js: reset attempts and log exit code

[clausreinke]

currently, the following lines are dead code as child is null when monitor is called

node-windows/lib/wrapper.js

Lines 120 to 125 in accfbd0

	} else {
	// reset attempts and wait time
	attempts = 0;
	wait = argv.w * 1000;
	}
	};

This suggests that monitor may not be used as originally intended, and it means that even successful restarts suffer from accumulated delays, keeping the child offline without reason.

Given the ressource limitations on this project, I would suggest a minor fix: after the fork (

node-windows/lib/wrapper.js

Line 162 in accfbd0

child = fork(script,args,opts);

) add

 if (child && child.pid) {
    // reset attempts and wait time
    attempts = 0;
    wait = argv.w * 1000;
  }

Also, please don't hide the child's exit code - it is sometimes the only available clue to what went wrong.

node-windows/lib/wrapper.js

Lines 165 to 166 in accfbd0

	child.on('exit',function(code){
	log.warn(argv.f+' stopped running.');

log.warn(argv.f+' stopped running with code: '+code.toString(16));