should this be in a finally {} clause ? don't know how important it is to close() these