Why is timeToNxtFeed a double on line 9? It should be a time_t
Why are you calling timeToNextFeed() twice (lines 22 and 23)? Line 23 should be feedOn = timeToNxtFeed + time(NULL);
But really, consider the logic here. timeToNextFeed() computes the time of the next feeding. Then it subtracts time(0) to get the number of seconds until the next feeding. nextFeed() then adds time(0) back in to get the time of the next feeding. It would be better to rename timeToNextFeed() to be timeOfNextFeed() and have it return the actual feed time. This has the added advantage of avoiding the intermittent bug of having time(0) return different values on line 8 and line 23, which could extend the feed time by 1 second.