Again, shouldn't you call give_break in the loop?