Replacing ap_get_client_block()

[P. Dwayne Miller]

I was trying to replace a call to ap_get_client_block() in a module with 
appropriate apr*brigade*() calls.  Several readers posted some code and 
I thank them for that.  But, when trying to figure out all of the 
details, I tracked down the source of ap_get_client_block() and it is 
already using *brigade*() calls.  There are some comments that say that 
it should not be used, but the details of why are not obvious to me.  I 
was hoping someone could shed some light on this for me.