On Fri, Jan 21, 2011 at 4:34 PM, Filipe David Manana
<[email protected]> wrote:
> On Fri, Jan 21, 2011 at 3:29 PM, Benoit Chesneau <[email protected]> wrote:
>> On Fri, Jan 21, 2011 at 4:18 PM, Filipe David Manana
>> <[email protected]> wrote:
>>> -        ("httpd", "server_options") ->
>>> -            ?MODULE:stop();
>>>
>>> Benoît, I think you forgot to add a matching clause for the new
>>> couch_httpd:config_change/3 function.
>>>
>>
>> Sound like couch_config:register handle that: (couch_config.erl line 160) .
>
> I don't understand the reply.
> What I was talking about is that you're now ignoring new values for
> the "server_options" parameter. Your config_change/2 function is
> missing a clause like:
>
> config_change("httpd", "server_options") ->
>    ?MODULE:stop;
>
>
oh. Sorry was thinking you were speaking about function arity ... your
"/3" . Adding missing pattern.

-  benoît

Reply via email to