[PATCH] HTTP date parsing and formatting
Dridi Boukelmoune
dridi.boukelmoune at zenika.com
Wed Feb 25 15:23:24 CET 2015
On Wed, Feb 25, 2015 at 10:53 AM, Poul-Henning Kamp <phk at phk.freebsd.dk> wrote:
> Yeah, but now that we've rewritten the vtim.c code to not be locale
> dependent, that's sort of pointless ?
Pointless but convenient because I would not have to keep it around in
my working copy.
> By all means use it to find such trouble! I just don't see the point
> in adding it to the vmod on a permanent basis, unless we have code
> we specifically need to test.
Mainly for my very own convenience :)
Also to sort of "document" locale awareness in relevant test cases,
like regression tests would do with a bug that no longer exists.
> (Remember: Not all machines may have any given locale installed...)
That is indeed a problem. I suppose I could enforce a set of locales
at build time with autoconf, but it may not be welcomed.
I'll simply keep debug.setlocale around to find more issues.
Cheers,
Dridi
More information about the varnish-dev
mailing list