Christian Couder <christian.couder@xxxxxxxxx> writes: > This function will be used in a following commit to get the expiration > time of the shared index files from the config, and it is generic > enough to be put in "config.c". Is it generic enough that a helper that sounds as if it can get any date string dies if it is given a future date? I somehow doubt it. At the minimum, it must be made clear that there is an artificial limitation that the current set of callers find useful in cache.h as a one-liner comment next to the added declaration. Then people with the same need (i.e. they want to reject future timestamps) can decide to use it, while others would stay away from it. If you can come up with a better word to use to encode that artificial limitation in its name, renaming it is even better.