bad indentation in loop