Hmmm... While looking over some proxy changes, for some
reason my eyes were pulled to apr_temp_dir_get(). Note
that the comments state:

    "Our goal is to find a temporary directory suitable for writing
     into.  We'll only pay the price once if we're successful -- we
     cache our successful find."

Except we don't (cache); neither in 0.9.x or trunk.

So is the comment broken or the code? Methinks the code,
but I'm sure one reason was due to potential thread
safety issues... ?

Reply via email to