Hi,
also ich würde das:

if (debounce(&PINB, PB0)) /* Falls Taster an PIN PB0 gedrueckt.. */
PORTB = PINB ^ ( 1 << PB1 ); /* ..LED an Port PB1 an-
bzw. ausschalten */

in ne Endlosschleife setzen, weil dein Programm aus ist bevor du überhaupt gedrückt hast!
Des zweimal if ist ja net schlimm!
Des 0xFF kann man so lassen man kann auch 0x00FF schreiben des isch genauso wie wenn man 0010 statt 10 schreibt!
Gruß Michi