[FixRM #8312] - Fix file handle leaks

Fix file handle leaks for [SeeRM #8312]
What Luke said.

"You cannot, in general, place a variable declaration in a begin
scope and use it in the ensure scope unless you use nil?. It is
better to swap line 35 and line 34."
Make sure we close hashlist
Properly close hashlist
If exception hits, make sure it's closed.
Trivial grammar and word choice fixes for modules
Use the correct variable
Use the correct variable
[SeeRM #8313] - Print where files are stored

As an user, I want to be able to see where my file is stored when the
module I'm using runs a store_loot().
[FixRM #8314] - Use OptPath instead of OptString

These modules need to use OptPath to make sure the path is validated.
Typo
Land @2243, @wchen-r7's patch for [SeeRM #8312]
Merge pull request #2245 from todb-r7/grammar-and-such

Trivial grammar and word choice fixes for modules
Get rid of function that's never used

RPORT datastore option is deregistered, and is never used anywhere
in the module, so I don't why we need this rport() function here.
Module should use OptRegexp for regex pattern option

Instead of using OptString, OptRegexp should be used because this
datastore option is a regex pattern.
Use the correct var
Use print_line instead of print

These modules should be using print_line instead of print
Remove comments
Oh yeah, need to do this too
Patch for [SeeRM #8315]