> >> parse_string(r, tag_val, parsed_string, sizeof(parsed_string), 0);
> >> if (!find_file(r, "flastmod", tag, parsed_string, &finfo)) {
> >> char *t_val;
> >>
> >> t_val = ap_ht_time(r->pool, finfo.mtime, ctx->time_str, 0);
> >> t_len = strlen(t_val);
> >>
> >> tmp_buck = ap_bucket_create_transient(t_val, t_len);
> >> AP_BUCKET_INSERT_BEFORE(head_ptr, tmp_buck);
> >> if (*inserted_head == NULL) {
> >> *inserted_head = tmp_buck;
> >> }
> >>
> [snip]
> > CREATE_ERROR_BUCKET is brokecn because of the transient problem.
>
> Yes, I agree that you can't use a transient bucket (because a transient
> bucket means data on the stack that must be passed down right away or
> setaside (ie, copied into a heap bucket). But you *can* use an immortal
> bucket. Any disagreement with that statement, Ryan?
I haven't reviewed patch itself, but the concepts are correct.
Ryan
_______________________________________________________________________________
Ryan Bloom [EMAIL PROTECTED]
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------
