bug#56907: closed ([PATCH] * eww.el (eww-browse): Raise error if there are no arguments)

[GNU bug Tracking System]

Your message dated Thu, 04 Aug 2022 12:22:00 +0000
with message-id <877d3o9p1z.fsf@posteo.net>
and subject line Re: bug#56907: [PATCH] * eww.el (eww-browse): Raise error if 
there are no arguments
has caused the debbugs.gnu.org bug report #56907,
regarding [PATCH] * eww.el (eww-browse): Raise error if there are no arguments
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
56907: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=56907
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH] * eww.el (eww-browse): Raise error if there are no arguments Date: Wed, 03 Aug 2022 10:37:40 +0000

Tags: patch


If there are no command line arguments, then nil is passed to eww, and
eventually it is given to string-trim, that raises an apparently
unrelated error due to the missing string.

An alternative to this patch would be to unmark it as interactive, since
it appears to be meant for batch use.  I had initially confused it with
`eww-open-file'.


In GNU Emacs 29.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.34, cairo 
version 1.17.6)
 of 2022-08-01 built on rhea
Repository revision: 47f1cae83c269ea43d6b208e055ce536c017856f
Repository branch: feature/package+vc
System Description: Fedora Linux 36 (Workstation Edition)

Configured using:
 'configure --with-pgtk --with-native-compilation --with-imagemagick'

0001-eww.el-eww-browse-Raise-error-if-there-are-no-argume.patch
Description: Text Data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#56907: [PATCH] * eww.el (eww-browse): Raise error if there are no arguments
>
>
>
>
> Date: 
>
> Thu, 04 Aug 2022 12:22:00 +0000
>
>
>
>
> Lars Ingebrigtsen <larsi@gnus.org> writes:
>
> > Philip Kaludercic <philipk@posteo.net> writes:
> >
> >> Subject: [PATCH] * eww.el (eww-browse): Raise error if there are no arguments
> >
> > Looks good to me; please go ahead and push.
>
> Done.
>
>
> --- End Message ---