]> git.ipfire.org Git - thirdparty/apache/httpd.git/commit
Revert about half of the last commit because of an oversight on my part
authorCliff Woolley <jwoolley@apache.org>
Sat, 21 Apr 2001 22:01:05 +0000 (22:01 +0000)
committerCliff Woolley <jwoolley@apache.org>
Sat, 21 Apr 2001 22:01:05 +0000 (22:01 +0000)
commit48ae6610b471b3fda58c7818ee301c74b0a3164e
tree362d87534342c6a7809d4db6240ab0daa1b278a2
parent4647e5f71a4b2d1d62238b9bce854a501b5477fb
Revert about half of the last commit because of an oversight on my part
that broke it.  Unfortunately, fixing the oversight "the right way" is
almost as ugly as the original code; it's easier to just go back to the way
it was, at least for now.

The problem is that the original code would delete buckets from foo through
bar in the brigade, and the patched code deletes *everything* up through
bar, which is bad.  I could have fixed it by doing two splits, but that
introduces too many palloc's for my taste.  It's also fixable with RING macros,
but I refuse to start using RING macros directly on brigades.  The best
solution would be if there were a brigade equivalent to a RING_UNSPLICE/
RING_INSERT_HEAD sequence (this sequence is also used interally by
apr_brigade_split(), btw), something like this:
APR_BRIGADE_TRANSFER_BUCKETS(oldbrigade,newbrigade,startbucket,endbucket);

Absent that, the affected parts of the patch are hereby reverted (grumble,
grumble).  If people liek the APR_BRIGADE_TRANSFER_BUCKETS() idea, I'll
implement that and repatch mod_include later.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@88909 13f79535-47bb-0310-9956-ffa450edef68
modules/filters/mod_include.c