seems ok here, but in general, should initialize all variables to something