Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:

>>> I don't know `let-alist`, but looks good, however keeping
>>> `ffap-gopher-regexp` instead of hardcoding it and returning nil when the
>>> regexp is nil would allow users to disable the feature.
>>
>> Makes sense, here's an update:
>
> Thanks, however AFAIU your patch, when `ffap-gopher-regexp` is nil,
> `ffap-gopher-at-point` will return with an error but not nil isn't it ?

Yes, although that's also true before my patch.  So `ffap-gopher-regexp'
never actually supported being changed.  I'm not sure if it's a good
idea to add support for that.