updates file read to close file more quickly

cleaned up the returns which are unneccsary and changed the file read so it pulls out all the lines and then closes the file.  I did this because the scans could take a while and that file would be left open for quite a while.  This approach also isolates the file read to one line, vs it reading one for each iteration.  It may or may not be much better, but whatever, I think it's easier to look at and debug.  Let me know what you think.  I'm sorry for the additional PR.  I can def merge after this.  These changes aren't major in anyway, just idiomatic things.
Oh, I also wrapped one seriously long line and add a couple words to the description to reflect the behavior you altered after my last PR and I shortened some of the 'if result' lines.