Guardando il codice mi pare di capire che scrivendo
ledyellow(), LOW;
allarme(), LOW;
lui volesse/credesse in qualche modo di pilotare le void esterne al loop con il LOW e HIGH......dico questo perchè lo ripete più volte nel loop e lo utilizza sempre con le void esterne pensando magari che con LOW non entra nella void e che con HIGH invece si. Se così fosse non si scrive "ledyellow(),HIGH;" se si vuole entrare nella void ma semplicemente "ledyellow();".
In buona sostanza tutte le void esterne non vengono mai eseguite fino a quando non vengono chiamate dal loop principale e,una volta letta ed eseguito il codice scritto,si ritorna in automatico al loop principale continuando da dove aveva lasciato.