From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Amin Bandali Newsgroups: gmane.emacs.devel Subject: Re: master 081eb52: Switch the order of ERC's '/whois' arguments sent to the server Date: Wed, 29 Sep 2021 22:30:12 -0400 Message-ID: <87lf3eu763.fsf@gnu.org> References: <20210929050358.32228.41978@vcs0.savannah.gnu.org> <20210929050359.D07CD207F5@vcs0.savannah.gnu.org> <878rzf7stj.fsf@gmx.de> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="17290"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) Cc: emacs-devel@gnu.org To: Michael Albinus Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Thu Sep 30 04:32:02 2021 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mVlre-0004KI-Kz for ged-emacs-devel@m.gmane-mx.org; Thu, 30 Sep 2021 04:32:02 +0200 Original-Received: from localhost ([::1]:38236 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mVlrc-0003CO-Gd for ged-emacs-devel@m.gmane-mx.org; Wed, 29 Sep 2021 22:32:00 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:53256) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mVlqT-0002Q0-Oj for emacs-devel@gnu.org; Wed, 29 Sep 2021 22:30:49 -0400 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:54430) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mVlqR-0001XK-I5; Wed, 29 Sep 2021 22:30:49 -0400 Original-Received: from [2607:fea8:3fdf:f2d9:30cf:3cbd:d5ed:c84c] (port=34570 helo=langa) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mVlpt-00022b-La; Wed, 29 Sep 2021 22:30:21 -0400 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.io gmane.emacs.devel:275843 Archived-At: Michael Albinus writes: > bandali@gnu.org (Amin Bandali) writes: > > Hi Amin, Hi Michael, >> diff --git a/etc/NEWS b/etc/NEWS >> index 6d039ba..bf942cb 100644 >> --- a/etc/NEWS >> +++ b/etc/NEWS >> @@ -2853,6 +2853,11 @@ The new '/opme' convenience command asks ChanServ to set the operator >> status for the current nick in the current channel, and '/deopme' >> unsets it. >> >> +--- >> +*** Fix the order of '/whois' arguments sent to the server. >> +Per RFC 1459 and RFC 2812, when given, the optional 'server' argument >> +for the WHOIS command must come before the 'user' argument, not after. >> + > > Isn't this rather an implementation detail, not needed to be announced > in etc/NEWS? Yes, but it was a breaking behaviour change in a fairly commonly used, long-standing, user-facing function -- granted, the optional `server' argument isn't used nearly as often -- I thought it'd be appropriate to announce it in etc/NEWS so that people relying on the old behaviour would know to update their usage. That said, today I thought of a better way to approach this which preserves the old (and more intuitive, coming from other IRC clients) behaviour of the /whois command people have come to get used to, by restoring the old behaviour and instead changing the argument names to be more accurate, and properly documented them in the doc string. I removed the etc/NEWS entry I'd added, as it's not needed anymore. :) > Best regards, Michael. > Best, amin -- https://bndl.org