On Wed, Mar 21, 2012 at 01:34, Randall Leeds <[email protected]>wrote:

> Thanks for cleaning up after me. Sloppy, Randall. Bad, Randall.
>

Well, I was mostly right...


> >>> In particular, the error is with the change to couch_log.erl.
>> >>
>> >> Whoops. I meant couch_file.erl.
>> >>
>> >> Randall, please see the attached diff, or see
>> >> http://friendpaste.com/2Hwjn9uj1oRM8pyRqAdbxc
>> >>
>> >> The pattern {Ref, Pid, {error, Reason}} does not match because the
>> >> message is {Ref, Pid, Reason}
>>
>
>
Only in one case, where we detect that the file exists.
I've changed it to return {error, eexist} in this case, which matches the
errors that come back from the file module in all other places and will
format the correct message.

Fixed on 1.1.x and 1.2.x.

Reply via email to