[debbugs-tracker] bug#36428: closed (Non-necessary progn afer with-eval-after-load in tramp in recent change)

[GNU bug Tracking System]

Your message dated Sat, 29 Jun 2019 12:23:16 +0200
with message-id <address@hidden>
and subject line Re: bug#36428: Non-necessary progn afer with-eval-after-load 
in tramp in recent change
has caused the debbugs.gnu.org bug report #36428,
regarding Non-necessary progn afer with-eval-after-load in tramp in recent 
change
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden.)


-- 
36428: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=36428
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: Non-necessary progn afer with-eval-after-load in tramp in recent change Date: Sat, 29 Jun 2019 11:49:56 +0800

Hi,

In some recent changes to tramp, the 'eval-after-load' was replaced by 'with-eval-after-load',

however, the unnecessary 'progn' function for the 'eval-after-load' is no removed.

Best Regards,

Shuguang Sun

tramp_progn_after_with-eval-after-load.diff
Description: Binary data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#36428: Non-necessary progn afer with-eval-after-load in tramp in recent change
>
>
>
>
> Date: 
>
> Sat, 29 Jun 2019 12:23:16 +0200
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)
>
>
>
>
> Version: 27.1
>
> Eli Zaretskii <address@hidden> writes:
>
> >> However, the real change touches just 10 lines, everything else are just
> >> indentation (white space) changes. So IMHO, the patch could be applied
> >> w/o signed papers.
> >>
> >> Eli?
> >
> > Yes, this needs no legal paperwork.
>
> Thanks, I've pushed it to master. Closing the bug.
>
> > Thanks.
>
> Best regards, Michael.
>
>
> --- End Message ---