Properly close listeners for asynchronous threads.
Signed-off-by: NOleg Nenashev <o.v.nenashev@gmail.com>

Signed-off-by: NOleg Nenashev <o.v.nenashev@gmail.com>

Resource leak may happen during the search of manifest file.
Signed-off-by: NOleg Nenashev <o.v.nenashev@gmail.com>

The issue is related to JENKINS-18351.
Signed-off-by: NOleg Nenashev <o.v.nenashev@gmail.com>

Signed-off-by: NOleg Nenashev <o.v.nenashev@gmail.com>

Signed-off-by: NOleg Nenashev <o.v.nenashev@gmail.com>

Signed-off-by: NOleg Nenashev <o.v.nenashev@gmail.com>

Adjust my prior change to better utilize structured form submission...

Adjust my prior change to better utilize structured form submission to accomodate a plugins that are not using p:config-assignedLabel

Adjust my prior change to better utilize structured form submission to accomodate a few plugins that are improperly submitting data through this processing logic

JNAerator ( https://code.google.com/p/jnaerator/ ) was helpful
for generating the Java code from the C struct.

Remove fields which are not part of struct _SHELLEXECUTEINFO.

fromExternalizableId should just return null, not throw IAE, if the job has since been renamed/move/deleted.
(It could already return null if the run has since been deleted; #1264 failed to mark this properly.)
Reserving IAE for inputs that getExternalizableId could not actually have produced,
and making sure to also throw this in case of non-numeric text after the hash.
Adding Javadoc for both methods.