[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