From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: Renaming non-X x_* procedures in xdisp.c (and elsewhere) Date: Sat, 23 Mar 2019 21:32:47 +0200 Message-ID: <83wokpo080.fsf@gnu.org> References: <87wokp4okn.fsf@gmail.com> <83ef6xpo6b.fsf@gnu.org> <0f4be9a6-6e09-f55d-9f58-2a15aef264cd@cs.ucla.edu> <837ecpplw8.fsf@gnu.org> <87imw94hi2.fsf@gmail.com> <831s2xpiba.fsf@gnu.org> <87a7hl4e01.fsf@gmail.com> Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="55193"; mail-complaints-to="usenet@blaine.gmane.org" Cc: eggert@cs.ucla.edu, emacs-devel@gnu.org To: Alex Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Mar 23 20:55:30 2019 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1h7mjp-000Dlk-8W for ged-emacs-devel@m.gmane.org; Sat, 23 Mar 2019 20:55:29 +0100 Original-Received: from localhost ([127.0.0.1]:47372 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1h7mjo-0000pM-8n for ged-emacs-devel@m.gmane.org; Sat, 23 Mar 2019 15:55:28 -0400 Original-Received: from eggs.gnu.org ([209.51.188.92]:53118) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1h7mOD-0006MA-Ky for emacs-devel@gnu.org; Sat, 23 Mar 2019 15:33:11 -0400 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:38059) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1h7mOD-0006Rc-2a; Sat, 23 Mar 2019 15:33:09 -0400 Original-Received: from [176.228.60.248] (port=3413 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1h7mOA-000526-6E; Sat, 23 Mar 2019 15:33:08 -0400 In-reply-to: <87a7hl4e01.fsf@gmail.com> (message from Alex on Sat, 23 Mar 2019 12:55:26 -0600) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.21 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.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.org gmane.emacs.devel:234664 Archived-At: > From: Alex > Cc: eggert@cs.ucla.edu, emacs-devel@gnu.org > Date: Sat, 23 Mar 2019 12:55:26 -0600 > > > But what you propose in the patch stops short of that goal, it just > > renames the functions that are explicitly called from xdisp.c. It > > doesn't rename x_* functions in files unrelated to X. I'm not sure > > this partial renaming is worth the trouble. > > It can be considered as just a step towards that goal. As I mentioned, I > don't see why all the work has to happen at the same time. IMO the RIF > x_* and the multiply-defined x_* are separate, even if related, issues; > the RIF x_* being much easier to solve (as demonstrated). I tend to prefer to do it in one go. The reason is that in the past we've seen once or twice someone who made such initial steps towards a goal that we considered important, but then didn't follow up with the rest, and we were left with a slightly less readable and/or slightly less familiar code, and no real gains. So now I'm less inclined to support such partial steps which in themselves have little gains to offer. Of course, it's possible to accumulate the changes piecemeal on a separate branch, if you want to work on this in several increments, I'm only talking about landing them on master. > If you're referring to another class of x_* procedures to be renamed, > then those can be done in a later commit. I think I see them all as a single class. That some of them are used through redisplay_interface is not an important distinction, IMO. > >> P.S. Should x_clear_window_mouse_face instead be renamed to > >> clear_window_mouse_face since it doesn't depend on HAVE_WINDOW_SYSTEM > >> like the others do? > > > > It does depend on the window-system, albeit somewhat subtly: it is > > only invoked for some terminal types. > > If it's not invoked for non-GUI Emacs, then the gui_* prefix would > indeed be appropriate. When the mouse is involved, the separation between GUI and non-GUI tends to be blurry.