]>
Commit | Line | Data |
---|---|---|
1b39caaa | 1 | |
582c2af2 | 2 | #include "squid.h" |
4299f876 | 3 | #include "base/AsyncJobCalls.h" |
3d93a84d | 4 | #include "base/TextException.h" |
1b39caaa | 5 | #include "BodyPipe.h" |
6 | ||
7 | CBDATA_CLASS_INIT(BodyPipe); | |
8 | ||
e7352f30 | 9 | // BodySink is a BodyConsumer class which just consume and drops |
26ac0430 AJ |
10 | // data from a BodyPipe |
11 | class BodySink: public BodyConsumer | |
12 | { | |
e7352f30 | 13 | public: |
5a856d1e AR |
14 | BodySink(const BodyPipe::Pointer &bp): AsyncJob("BodySink"), body_pipe(bp) {} |
15 | virtual ~BodySink() { assert(!body_pipe); } | |
26ac0430 | 16 | |
e7352f30 | 17 | virtual void noteMoreBodyDataAvailable(BodyPipe::Pointer bp) { |
26ac0430 AJ |
18 | size_t contentSize = bp->buf().contentSize(); |
19 | bp->consume(contentSize); | |
e7352f30 | 20 | } |
21 | virtual void noteBodyProductionEnded(BodyPipe::Pointer bp) { | |
5a856d1e | 22 | stopConsumingFrom(body_pipe); |
e7352f30 | 23 | } |
24 | virtual void noteBodyProducerAborted(BodyPipe::Pointer bp) { | |
5a856d1e | 25 | stopConsumingFrom(body_pipe); |
e7352f30 | 26 | } |
5a856d1e AR |
27 | bool doneAll() const {return !body_pipe && AsyncJob::doneAll();} |
28 | ||
29 | private: | |
30 | BodyPipe::Pointer body_pipe; ///< the pipe we are consuming from | |
31 | ||
e7352f30 | 32 | CBDATA_CLASS2(BodySink); |
33 | }; | |
34 | ||
35 | CBDATA_CLASS_INIT(BodySink); | |
36 | ||
37 | // The BodyProducerDialer is an AsyncCall class which used to schedule BodyProducer calls. | |
26ac0430 | 38 | // In addition to a normal AsyncCall checks if the BodyProducer is still the producer of |
e7352f30 | 39 | // the BodyPipe passed as argument |
40 | class BodyProducerDialer: public UnaryMemFunT<BodyProducer, BodyPipe::Pointer> | |
41 | { | |
42 | public: | |
26ac0430 | 43 | typedef UnaryMemFunT<BodyProducer, BodyPipe::Pointer> Parent; |
e7352f30 | 44 | |
4299f876 | 45 | BodyProducerDialer(const BodyProducer::Pointer &aProducer, |
4cb2536f A |
46 | Parent::Method aHandler, BodyPipe::Pointer bp): |
47 | Parent(aProducer, aHandler, bp) {} | |
e7352f30 | 48 | |
26ac0430 | 49 | virtual bool canDial(AsyncCall &call); |
e7352f30 | 50 | }; |
51 | ||
52 | // The BodyConsumerDialer is an AsyncCall class which used to schedule BodyConsumer calls. | |
26ac0430 | 53 | // In addition to a normal AsyncCall checks if the BodyConsumer is still the reciptient |
e7352f30 | 54 | // of the BodyPipe passed as argument |
55 | class BodyConsumerDialer: public UnaryMemFunT<BodyConsumer, BodyPipe::Pointer> | |
56 | { | |
57 | public: | |
26ac0430 | 58 | typedef UnaryMemFunT<BodyConsumer, BodyPipe::Pointer> Parent; |
e7352f30 | 59 | |
4299f876 | 60 | BodyConsumerDialer(const BodyConsumer::Pointer &aConsumer, |
4cb2536f A |
61 | Parent::Method aHandler, BodyPipe::Pointer bp): |
62 | Parent(aConsumer, aHandler, bp) {} | |
e7352f30 | 63 | |
26ac0430 | 64 | virtual bool canDial(AsyncCall &call); |
e7352f30 | 65 | }; |
66 | ||
67 | bool | |
26ac0430 AJ |
68 | BodyProducerDialer::canDial(AsyncCall &call) |
69 | { | |
70 | if (!Parent::canDial(call)) | |
71 | return false; | |
72 | ||
4299f876 | 73 | const BodyProducer::Pointer &producer = job; |
26ac0430 AJ |
74 | BodyPipe::Pointer pipe = arg1; |
75 | if (!pipe->stillProducing(producer)) { | |
76 | debugs(call.debugSection, call.debugLevel, HERE << producer << | |
77 | " no longer producing for " << pipe->status()); | |
78 | return call.cancel("no longer producing"); | |
79 | } | |
e7352f30 | 80 | |
26ac0430 | 81 | return true; |
e7352f30 | 82 | } |
83 | ||
84 | bool | |
26ac0430 AJ |
85 | BodyConsumerDialer::canDial(AsyncCall &call) |
86 | { | |
87 | if (!Parent::canDial(call)) | |
88 | return false; | |
89 | ||
4299f876 | 90 | const BodyConsumer::Pointer &consumer = job; |
26ac0430 AJ |
91 | BodyPipe::Pointer pipe = arg1; |
92 | if (!pipe->stillConsuming(consumer)) { | |
93 | debugs(call.debugSection, call.debugLevel, HERE << consumer << | |
94 | " no longer consuming from " << pipe->status()); | |
95 | return call.cancel("no longer consuming"); | |
96 | } | |
e7352f30 | 97 | |
26ac0430 | 98 | return true; |
e7352f30 | 99 | } |
100 | ||
101 | ||
102 | /* BodyProducer */ | |
103 | ||
1b39caaa | 104 | // inform the pipe that we are done and clear the Pointer |
105 | void BodyProducer::stopProducingFor(RefCount<BodyPipe> &pipe, bool atEof) | |
106 | { | |
26ac0430 AJ |
107 | debugs(91,7, HERE << this << " will not produce for " << pipe << |
108 | "; atEof: " << atEof); | |
109 | assert(pipe != NULL); // be strict: the caller state may depend on this | |
110 | pipe->clearProducer(atEof); | |
111 | pipe = NULL; | |
1b39caaa | 112 | } |
113 | ||
e7352f30 | 114 | |
115 | ||
116 | /* BodyConsumer */ | |
117 | ||
1b39caaa | 118 | // inform the pipe that we are done and clear the Pointer |
119 | void BodyConsumer::stopConsumingFrom(RefCount<BodyPipe> &pipe) | |
120 | { | |
26ac0430 AJ |
121 | debugs(91,7, HERE << this << " will not consume from " << pipe); |
122 | assert(pipe != NULL); // be strict: the caller state may depend on this | |
123 | pipe->clearConsumer(); | |
124 | pipe = NULL; | |
1b39caaa | 125 | } |
126 | ||
e7352f30 | 127 | |
1b39caaa | 128 | /* BodyPipe */ |
129 | ||
130 | BodyPipe::BodyPipe(Producer *aProducer): theBodySize(-1), | |
26ac0430 AJ |
131 | theProducer(aProducer), theConsumer(0), |
132 | thePutSize(0), theGetSize(0), | |
abe286b8 | 133 | mustAutoConsume(false), abortedConsumption(false), isCheckedOut(false) |
1b39caaa | 134 | { |
26ac0430 AJ |
135 | // TODO: teach MemBuf to start with zero minSize |
136 | // TODO: limit maxSize by theBodySize, when known? | |
137 | theBuf.init(2*1024, MaxCapacity); | |
138 | debugs(91,7, HERE << "created BodyPipe" << status()); | |
1b39caaa | 139 | } |
140 | ||
141 | BodyPipe::~BodyPipe() | |
142 | { | |
26ac0430 AJ |
143 | debugs(91,7, HERE << "destroying BodyPipe" << status()); |
144 | assert(!theProducer); | |
145 | assert(!theConsumer); | |
146 | theBuf.clean(); | |
1b39caaa | 147 | } |
148 | ||
47f6e231 | 149 | void BodyPipe::setBodySize(uint64_t aBodySize) |
1b39caaa | 150 | { |
26ac0430 | 151 | assert(!bodySizeKnown()); |
26ac0430 | 152 | assert(thePutSize <= aBodySize); |
1b39caaa | 153 | |
26ac0430 AJ |
154 | // If this assert fails, we need to add code to check for eof and inform |
155 | // the consumer about the eof condition via scheduleBodyEndNotification, | |
156 | // because just setting a body size limit may trigger the eof condition. | |
157 | assert(!theConsumer); | |
1b39caaa | 158 | |
26ac0430 AJ |
159 | theBodySize = aBodySize; |
160 | debugs(91,7, HERE << "set body size" << status()); | |
1b39caaa | 161 | } |
162 | ||
47f6e231 | 163 | uint64_t BodyPipe::bodySize() const |
1b39caaa | 164 | { |
26ac0430 AJ |
165 | assert(bodySizeKnown()); |
166 | return static_cast<uint64_t>(theBodySize); | |
1b39caaa | 167 | } |
168 | ||
47f6e231 | 169 | bool BodyPipe::expectMoreAfter(uint64_t offset) const |
1b39caaa | 170 | { |
26ac0430 AJ |
171 | assert(theGetSize <= offset); |
172 | return offset < thePutSize || // buffer has more now or | |
173 | (!productionEnded() && mayNeedMoreData()); // buffer will have more | |
1b39caaa | 174 | } |
175 | ||
176 | bool BodyPipe::exhausted() const | |
177 | { | |
26ac0430 | 178 | return !expectMoreAfter(theGetSize); |
1b39caaa | 179 | } |
180 | ||
610d8f3b | 181 | uint64_t BodyPipe::unproducedSize() const |
1b39caaa | 182 | { |
26ac0430 | 183 | return bodySize() - thePutSize; // bodySize() asserts that size is known |
1b39caaa | 184 | } |
185 | ||
83c51da9 CT |
186 | void BodyPipe::expectProductionEndAfter(uint64_t size) |
187 | { | |
188 | const uint64_t expectedSize = thePutSize + size; | |
189 | if (bodySizeKnown()) | |
190 | Must(bodySize() == expectedSize); | |
191 | else | |
192 | theBodySize = expectedSize; | |
193 | } | |
194 | ||
1b39caaa | 195 | void |
196 | BodyPipe::clearProducer(bool atEof) | |
197 | { | |
4299f876 | 198 | if (theProducer.set()) { |
26ac0430 | 199 | debugs(91,7, HERE << "clearing BodyPipe producer" << status()); |
4299f876 | 200 | theProducer.clear(); |
26ac0430 AJ |
201 | if (atEof) { |
202 | if (!bodySizeKnown()) | |
203 | theBodySize = thePutSize; | |
e1381638 AJ |
204 | else if (bodySize() != thePutSize) |
205 | debugs(91,3, HERE << "aborting on premature eof" << status()); | |
26ac0430 AJ |
206 | } else { |
207 | // asserta that we can detect the abort if the consumer joins later | |
208 | assert(!bodySizeKnown() || bodySize() != thePutSize); | |
209 | } | |
210 | scheduleBodyEndNotification(); | |
211 | } | |
1b39caaa | 212 | } |
213 | ||
214 | size_t | |
350e2aec | 215 | BodyPipe::putMoreData(const char *aBuffer, size_t size) |
1b39caaa | 216 | { |
26ac0430 | 217 | if (bodySizeKnown()) |
d85c3078 | 218 | size = min((uint64_t)size, unproducedSize()); |
26ac0430 AJ |
219 | |
220 | const size_t spaceSize = static_cast<size_t>(theBuf.potentialSpaceSize()); | |
d85c3078 | 221 | if ((size = min(size, spaceSize))) { |
350e2aec | 222 | theBuf.append(aBuffer, size); |
26ac0430 AJ |
223 | postAppend(size); |
224 | return size; | |
225 | } | |
226 | return 0; | |
1b39caaa | 227 | } |
228 | ||
229 | bool | |
4299f876 | 230 | BodyPipe::setConsumerIfNotLate(const Consumer::Pointer &aConsumer) |
1b39caaa | 231 | { |
26ac0430 | 232 | assert(!theConsumer); |
4299f876 | 233 | assert(aConsumer.set()); // but might be invalid |
26ac0430 AJ |
234 | |
235 | // TODO: convert this into an exception and remove IfNotLate suffix | |
236 | // If there is something consumed already, we are in an auto-consuming mode | |
237 | // and it is too late to attach a real consumer to the pipe. | |
238 | if (theGetSize > 0) { | |
239 | assert(mustAutoConsume); | |
240 | return false; | |
241 | } | |
1b39caaa | 242 | |
abe286b8 AR |
243 | Must(!abortedConsumption); // did not promise to never consume |
244 | ||
26ac0430 AJ |
245 | theConsumer = aConsumer; |
246 | debugs(91,7, HERE << "set consumer" << status()); | |
247 | if (theBuf.hasContent()) | |
248 | scheduleBodyDataNotification(); | |
249 | if (!theProducer) | |
250 | scheduleBodyEndNotification(); | |
1b39caaa | 251 | |
26ac0430 | 252 | return true; |
1b39caaa | 253 | } |
254 | ||
255 | void | |
26ac0430 AJ |
256 | BodyPipe::clearConsumer() |
257 | { | |
4299f876 | 258 | if (theConsumer.set()) { |
26ac0430 | 259 | debugs(91,7, HERE << "clearing consumer" << status()); |
4299f876 | 260 | theConsumer.clear(); |
abe286b8 AR |
261 | // do not abort if we have not consumed so that HTTP or ICAP can retry |
262 | // benign xaction failures due to persistent connection race conditions | |
263 | if (consumedSize()) | |
264 | expectNoConsumption(); | |
265 | } | |
266 | } | |
267 | ||
268 | void | |
269 | BodyPipe::expectNoConsumption() | |
270 | { | |
b84d327b AR |
271 | // We may be called multiple times because multiple jobs on the consumption |
272 | // chain may realize that there will be no more setConsumer() calls (e.g., | |
273 | // consuming code and retrying code). It is both difficult and not really | |
274 | // necessary for them to coordinate their expectNoConsumption() calls. | |
275 | ||
276 | // As a consequence, we may be called when we are auto-consuming already. | |
277 | ||
abe286b8 | 278 | if (!abortedConsumption && !exhausted()) { |
b84d327b | 279 | // Before we abort, any regular consumption should be over and auto |
d8eddc48 | 280 | // consumption must not be started. |
b84d327b AR |
281 | Must(!theConsumer); |
282 | ||
e6f9e263 A |
283 | AsyncCall::Pointer call= asyncCall(91, 7, |
284 | "BodyProducer::noteBodyConsumerAborted", | |
285 | BodyProducerDialer(theProducer, | |
286 | &BodyProducer::noteBodyConsumerAborted, this)); | |
287 | ScheduleCallHere(call); | |
288 | abortedConsumption = true; | |
b84d327b AR |
289 | |
290 | // in case somebody enabled auto-consumption before regular one aborted | |
291 | if (mustAutoConsume) | |
292 | startAutoConsumption(); | |
26ac0430 | 293 | } |
1b39caaa | 294 | } |
295 | ||
296 | size_t | |
350e2aec | 297 | BodyPipe::getMoreData(MemBuf &aMemBuffer) |
1b39caaa | 298 | { |
26ac0430 AJ |
299 | if (!theBuf.hasContent()) |
300 | return 0; // did not touch the possibly uninitialized buf | |
1b39caaa | 301 | |
350e2aec FC |
302 | if (aMemBuffer.isNull()) |
303 | aMemBuffer.init(); | |
304 | const size_t size = min(theBuf.contentSize(), aMemBuffer.potentialSpaceSize()); | |
305 | aMemBuffer.append(theBuf.content(), size); | |
26ac0430 AJ |
306 | theBuf.consume(size); |
307 | postConsume(size); | |
308 | return size; // cannot be zero if we called buf.init above | |
1b39caaa | 309 | } |
310 | ||
311 | void | |
312 | BodyPipe::consume(size_t size) | |
313 | { | |
26ac0430 AJ |
314 | theBuf.consume(size); |
315 | postConsume(size); | |
1b39caaa | 316 | } |
317 | ||
26ac0430 | 318 | // In the AutoConsumption mode the consumer has gone but the producer continues |
e7352f30 | 319 | // producing data. We are using a BodySink BodyConsumer which just discards the produced data. |
1b39caaa | 320 | void |
26ac0430 AJ |
321 | BodyPipe::enableAutoConsumption() |
322 | { | |
323 | mustAutoConsume = true; | |
324 | debugs(91,5, HERE << "enabled auto consumption" << status()); | |
325 | if (!theConsumer && theBuf.hasContent()) | |
326 | startAutoConsumption(); | |
5121d48e AR |
327 | } |
328 | ||
329 | // start auto consumption by creating body sink | |
330 | void | |
331 | BodyPipe::startAutoConsumption() | |
332 | { | |
26ac0430 AJ |
333 | Must(mustAutoConsume); |
334 | Must(!theConsumer); | |
5a856d1e | 335 | theConsumer = new BodySink(this); |
26ac0430 AJ |
336 | debugs(91,7, HERE << "starting auto consumption" << status()); |
337 | scheduleBodyDataNotification(); | |
1b39caaa | 338 | } |
339 | ||
340 | MemBuf & | |
26ac0430 AJ |
341 | BodyPipe::checkOut() |
342 | { | |
343 | assert(!isCheckedOut); | |
344 | isCheckedOut = true; | |
345 | return theBuf; | |
1b39caaa | 346 | } |
347 | ||
348 | void | |
349 | BodyPipe::checkIn(Checkout &checkout) | |
350 | { | |
26ac0430 AJ |
351 | assert(isCheckedOut); |
352 | isCheckedOut = false; | |
353 | const size_t currentSize = theBuf.contentSize(); | |
354 | if (checkout.checkedOutSize > currentSize) | |
355 | postConsume(checkout.checkedOutSize - currentSize); | |
e1381638 AJ |
356 | else if (checkout.checkedOutSize < currentSize) |
357 | postAppend(currentSize - checkout.checkedOutSize); | |
1b39caaa | 358 | } |
359 | ||
360 | void | |
361 | BodyPipe::undoCheckOut(Checkout &checkout) | |
362 | { | |
26ac0430 AJ |
363 | assert(isCheckedOut); |
364 | const size_t currentSize = theBuf.contentSize(); | |
365 | // We can only undo if size did not change, and even that carries | |
366 | // some risk. If this becomes a problem, the code checking out | |
367 | // raw buffers should always check them in (possibly unchanged) | |
368 | // instead of relying on the automated undo mechanism of Checkout. | |
369 | // The code can always use a temporary buffer to accomplish that. | |
87728a5b | 370 | Must(checkout.checkedOutSize == currentSize); |
1b39caaa | 371 | } |
372 | ||
373 | // TODO: Optimize: inform consumer/producer about more data/space only if | |
374 | // they used the data/space since we notified them last time. | |
375 | ||
376 | void | |
26ac0430 AJ |
377 | BodyPipe::postConsume(size_t size) |
378 | { | |
379 | assert(!isCheckedOut); | |
380 | theGetSize += size; | |
381 | debugs(91,7, HERE << "consumed " << size << " bytes" << status()); | |
382 | if (mayNeedMoreData()) { | |
383 | AsyncCall::Pointer call= asyncCall(91, 7, | |
384 | "BodyProducer::noteMoreBodySpaceAvailable", | |
385 | BodyProducerDialer(theProducer, | |
386 | &BodyProducer::noteMoreBodySpaceAvailable, this)); | |
387 | ScheduleCallHere(call); | |
388 | } | |
1b39caaa | 389 | } |
390 | ||
391 | void | |
26ac0430 AJ |
392 | BodyPipe::postAppend(size_t size) |
393 | { | |
394 | assert(!isCheckedOut); | |
395 | thePutSize += size; | |
396 | debugs(91,7, HERE << "added " << size << " bytes" << status()); | |
1b39caaa | 397 | |
26ac0430 AJ |
398 | if (mustAutoConsume && !theConsumer && size > 0) |
399 | startAutoConsumption(); | |
5121d48e | 400 | |
26ac0430 AJ |
401 | // We should not consume here even if mustAutoConsume because the |
402 | // caller may not be ready for the data to be consumed during this call. | |
403 | scheduleBodyDataNotification(); | |
1b39caaa | 404 | |
26ac0430 AJ |
405 | if (!mayNeedMoreData()) |
406 | clearProducer(true); // reached end-of-body | |
1b39caaa | 407 | } |
408 | ||
409 | ||
6c56baf6 | 410 | void |
411 | BodyPipe::scheduleBodyDataNotification() | |
412 | { | |
4299f876 | 413 | if (theConsumer.valid()) { // TODO: allow asyncCall() to check this instead |
26ac0430 AJ |
414 | AsyncCall::Pointer call = asyncCall(91, 7, |
415 | "BodyConsumer::noteMoreBodyDataAvailable", | |
416 | BodyConsumerDialer(theConsumer, | |
417 | &BodyConsumer::noteMoreBodyDataAvailable, this)); | |
418 | ScheduleCallHere(call); | |
419 | } | |
6c56baf6 | 420 | } |
421 | ||
1b39caaa | 422 | void |
423 | BodyPipe::scheduleBodyEndNotification() | |
424 | { | |
4299f876 | 425 | if (theConsumer.valid()) { // TODO: allow asyncCall() to check this instead |
26ac0430 AJ |
426 | if (bodySizeKnown() && bodySize() == thePutSize) { |
427 | AsyncCall::Pointer call = asyncCall(91, 7, | |
428 | "BodyConsumer::noteBodyProductionEnded", | |
429 | BodyConsumerDialer(theConsumer, | |
430 | &BodyConsumer::noteBodyProductionEnded, this)); | |
431 | ScheduleCallHere(call); | |
432 | } else { | |
433 | AsyncCall::Pointer call = asyncCall(91, 7, | |
434 | "BodyConsumer::noteBodyProducerAborted", | |
435 | BodyConsumerDialer(theConsumer, | |
436 | &BodyConsumer::noteBodyProducerAborted, this)); | |
437 | ScheduleCallHere(call); | |
438 | } | |
439 | } | |
1b39caaa | 440 | } |
441 | ||
1b39caaa | 442 | // a short temporary string describing buffer status for debugging |
443 | const char *BodyPipe::status() const | |
444 | { | |
350e2aec FC |
445 | static MemBuf outputBuffer; |
446 | outputBuffer.reset(); | |
1b39caaa | 447 | |
350e2aec | 448 | outputBuffer.append(" [", 2); |
1b39caaa | 449 | |
c91ca3ce | 450 | outputBuffer.Printf("%" PRIu64 "<=%" PRIu64, theGetSize, thePutSize); |
1b39caaa | 451 | if (theBodySize >= 0) |
c91ca3ce | 452 | outputBuffer.Printf("<=%" PRId64, theBodySize); |
26ac0430 | 453 | else |
350e2aec | 454 | outputBuffer.append("<=?", 3); |
1b39caaa | 455 | |
350e2aec | 456 | outputBuffer.Printf(" %d+%d", (int)theBuf.contentSize(), (int)theBuf.spaceSize()); |
1b39caaa | 457 | |
350e2aec | 458 | outputBuffer.Printf(" pipe%p", this); |
4299f876 AR |
459 | if (theProducer.set()) |
460 | outputBuffer.Printf(" prod%p", theProducer.get()); | |
461 | if (theConsumer.set()) | |
462 | outputBuffer.Printf(" cons%p", theConsumer.get()); | |
1b39caaa | 463 | |
26ac0430 | 464 | if (mustAutoConsume) |
350e2aec | 465 | outputBuffer.append(" A", 2); |
abe286b8 AR |
466 | if (abortedConsumption) |
467 | outputBuffer.append(" !C", 3); | |
26ac0430 | 468 | if (isCheckedOut) |
350e2aec | 469 | outputBuffer.append(" L", 2); // Locked |
1b39caaa | 470 | |
350e2aec | 471 | outputBuffer.append("]", 1); |
1b39caaa | 472 | |
350e2aec | 473 | outputBuffer.terminate(); |
1b39caaa | 474 | |
350e2aec | 475 | return outputBuffer.content(); |
1b39caaa | 476 | } |
477 | ||
478 | ||
479 | /* BodyPipeCheckout */ | |
480 | ||
481 | BodyPipeCheckout::BodyPipeCheckout(BodyPipe &aPipe): pipe(aPipe), | |
26ac0430 AJ |
482 | buf(aPipe.checkOut()), offset(aPipe.consumedSize()), |
483 | checkedOutSize(buf.contentSize()), checkedIn(false) | |
1b39caaa | 484 | { |
485 | } | |
486 | ||
487 | BodyPipeCheckout::~BodyPipeCheckout() | |
488 | { | |
87728a5b AR |
489 | if (!checkedIn) { |
490 | // Do not pipe.undoCheckOut(*this) because it asserts or throws | |
491 | // TODO: consider implementing the long-term solution discussed at | |
492 | // http://www.mail-archive.com/squid-dev@squid-cache.org/msg07910.html | |
493 | debugs(91,2, HERE << "Warning: cannot undo BodyPipeCheckout"); | |
494 | pipe.checkIn(*this); | |
495 | } | |
1b39caaa | 496 | } |
497 | ||
498 | void | |
499 | BodyPipeCheckout::checkIn() | |
500 | { | |
26ac0430 AJ |
501 | assert(!checkedIn); |
502 | pipe.checkIn(*this); | |
503 | checkedIn = true; | |
1b39caaa | 504 | } |
505 | ||
506 | ||
507 | BodyPipeCheckout::BodyPipeCheckout(const BodyPipeCheckout &c): pipe(c.pipe), | |
26ac0430 AJ |
508 | buf(c.buf), offset(c.offset), checkedOutSize(c.checkedOutSize), |
509 | checkedIn(c.checkedIn) | |
1b39caaa | 510 | { |
26ac0430 | 511 | assert(false); // prevent copying |
1b39caaa | 512 | } |
513 | ||
514 | BodyPipeCheckout & | |
515 | BodyPipeCheckout::operator =(const BodyPipeCheckout &) | |
516 | { | |
26ac0430 AJ |
517 | assert(false); // prevent assignment |
518 | return *this; | |
1b39caaa | 519 | } |