On Sep 1, 2011, at 8:59 AM, Joe Orton wrote:

> On Thu, Sep 01, 2011 at 02:47:19PM +0200, "Plüm, Rüdiger, VF-Group" wrote:
>>>> If we rip it out, we should replace it with ap_assert()s. And maybe 
>>>> only do it in 2.3?
>>> 
>>> It would seem odd to have ENOTIMPL as a "fatal" error but other 
>>> "real" errors non-fatal.  *No* error should occur here with unless 
>>> somebody has cooked up a really whacky bucket type.
>> 
>> So you would follow the rip-out-and-replace-with-asserts approach?
> 
> No, just remove the special case for ENOTIMPL and don't change anything 
> else.
> 

+1...

Reply via email to