If no file provided defer file.Close() causes panic.

[andyb]

I've moved the defer call after error check to prevent panic when file not found.

[mish15]

@andyb thanks for the PR. From memory I put it there for a reason. I think it's possible to have a file and an error (?), which would cause a mem leak. Will have a look in the morning. Cheers!

[andyb]

No problem, thanks for the useful package! If it needs to be there maybe a nil check will suffice to stop the panic. Let me know if you need me to change anything.

[mish15]

Yep no worries. Definitely need to get rid of the panic! Will have a play
in the morning.

On Thursday, 26 February 2015, Andy Britcliffe notifications@github.com
wrote:

No problem, thanks for the useful package! If it needs to be there maybe a
nil check will suffice to stop the panic. Let me know if you need me to
change anything.

—
Reply to this email directly or view it on GitHub
#6 (comment).

Hamish Ogilvy
Sajari Pty Ltd
_t: +61 (_0) 414 658 353 | e: *hogilvy@sajari.com
*w: www.sajari.com

[mish15]

Closing in favour of 61f9ca2