It is likely to fail when millis() rolls over to zero at about 49 days, but that may not be a problem in your project