[Date Prev][Date Next][Thread Prev][][Date Index][Thread Index]

Re: error upon load and tab navigation [PATCH included]



On 2017-07-10 06:34, Boruch Baum wrote:
> On 2017-07-10 19:06, Katsumi Yamaoka wrote:
> > In [emacs-w3m : No.12738]
> > On Mon, 10 Jul 2017 05:42:21 -0400, Boruch Baum wrote:
> > > I did that work while tracing the problem for the patch. The culprit
> > > is `w3m-print-this-url', ie. remove it from the hook list, and the
> > > issue disappears.
> >
> > Thanks.  So, what should be fixed is `w3m-print-this-url', not
> > `w3m-replace-in-string', isn't it?
>
> Yes. I did look in `w3m-print-this-url' and did not see where to make
> a fix. Later, it seemed desirable / sensible for the function
> `w3m-replace-in-string' to fail silently and return nil instead of
> returning an error when a input string was unbound.

Following up, I see that this bug isn't just a matter of being a
nuisance message, but it does cause me failures.

For example, `C-x m' (browse-url-at-point) does not work without the
patch, but does with the patch.

--
hkp://keys.gnupg.net
CA45 09B5 5351 7C11 A9D1  7286 0036 9E45 1595 8BC0