divide openConnection in the wrapper function openConnect and the
internal openConnectionWithoutCloseOnError. the former calls the latter
and automatically calls closeConnection if an error was returned. the
internal function no longer calls closeConnection at all

also cleaned up style oddities while stepping through the code (fewer
line breaking, fewer useless var assignment, no longer `else if` after
break/return)