"M. Levinson" <[EMAIL PROTECTED]> writes:

> While writing this, I noticed that the Python API documentation doesn't
> mention the effect of returning an empty string from goto_url_hook or
> follow_url_hook. Here's a trivial patch to remedy that.

OK, applied.

While reviewing the patch, I noticed a something odd in the
documentation of goto-url in doc/events.txt:

| If a URL other than @url should be followed, the old one should be freed
| and the new one should be assigned to @url. @url must not be assigned
| NULL and must remain freeable.
| Valid values for @url are:
| - unchanged, if the original URL should be followed;
| - a new, dynamically allocated URL to be followed instead; or
| - NULL or an empty, dynamically allocated string if no URL should be followed.

What can "@url must not be assigned NULL" mean if NULL is
a valid value for @url?  The follow-url hook has no such
restriction.

This contradiction does not hurt the Python interface because
replace_with_python_string never sets *dest = NULL.

Attachment: pgp90IM3v7WUU.pgp
Description: PGP signature

_______________________________________________
elinks-dev mailing list
[email protected]
http://linuxfromscratch.org/mailman/listinfo/elinks-dev

Reply via email to