GNU bug report logs - #58021
29.0.50; Remove old font.c code

Previous Next

Package: emacs;

Reported by: Manuel Giraud <manuel <at> ledu-giraud.fr>

Date: Fri, 23 Sep 2022 09:12:02 UTC

Severity: normal

Tags: moreinfo, wontfix

Found in version 29.0.50

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 58021 in the body.
You can then email your comments to 58021 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#58021; Package emacs. (Fri, 23 Sep 2022 09:12:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Manuel Giraud <manuel <at> ledu-giraud.fr>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Fri, 23 Sep 2022 09:12:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Manuel Giraud <manuel <at> ledu-giraud.fr>
To: bug-gnu-emacs <at> gnu.org
Subject: 29.0.50; Remove old font.c code
Date: Fri, 23 Sep 2022 11:11:22 +0200
[Message part 1 (text/plain, inline)]
Hi,

Here is patch that removes some old "#if 0" code from font.c.  Most of
this code seems to be "#if 0" since 2006 (when font.c was introduced)
and was only "updated" for overall interfaces change.

Maybe it was kept as hints for future readers but I found this code to
be a bit misleading as it has drifted along from how things are actually
done now.

[0001-src-font.c-remove-old-if-0-code.patch (text/x-patch, attachment)]
[Message part 3 (text/plain, inline)]
Best regards,


In GNU Emacs 29.0.50 (build 2, x86_64-unknown-openbsd7.2, cairo version
 1.17.6) of 2022-09-23 built on elite.giraud
Repository revision: 94380420e2cba7e2821b22fd8a46cd5b04c985ef
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12101004
System Description: OpenBSD elite.giraud 7.2 GENERIC.MP#739 amd64

Configured using:
 'configure --prefix=/home/manuel/emacs --bindir=/home/manuel/bin
 --with-x-toolkit=no --without-sound --without-compress-install
 CPPFLAGS=-I/usr/local/include LDFLAGS=-L/usr/local/lib'

Configured features:
CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GSETTINGS HARFBUZZ JPEG JSON
LCMS2 LIBOTF LIBXML2 MODULES NOTIFY KQUEUE OLDXMENU PDUMPER PNG RSVG
SQLITE3 THREADS TIFF WEBP X11 XDBE XIM XINPUT2 XPM ZLIB

Important settings:
  value of $LC_ALL: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Change Log

Minor modes in effect:
  global-git-commit-mode: t
  magit-auto-revert-mode: t
  icomplete-mode: t
  display-time-mode: t
  display-battery-mode: t
  shell-dirtrack-mode: t
  global-so-long-mode: t
  repeat-mode: t
  global-eldoc-mode: t
  show-paren-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  line-number-mode: t
  auto-fill-function: do-auto-fill
  indent-tabs-mode: t
  transient-mark-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t

Load-path shadows:
/home/manuel/.emacs.d/elpa/transient-20220806.2224/transient hides /home/manuel/emacs/share/emacs/29.0.50/lisp/transient

Features:
(shadow sort mail-extr emacsbug whitespace gnus-dired magit-patch
sh-script smie magit-extras log-view executable vc-annotate vc misearch
multi-isearch vc-git vc-dispatcher vc-svn bug-reference pulse face-remap
magit-bookmark magit-submodule magit-obsolete magit-blame magit-stash
magit-reflog magit-bisect magit-push magit-pull magit-fetch magit-clone
magit-remote magit-commit magit-sequence magit-notes magit-worktree
magit-tag magit-merge magit-branch magit-reset magit-files magit-refs
magit-status magit magit-repos magit-apply magit-wip magit-log
which-func imenu magit-diff smerge-mode diff diff-mode git-commit
log-edit pcvs-util add-log magit-core magit-autorevert autorevert
filenotify magit-margin magit-transient magit-process with-editor
magit-mode transient magit-git magit-base magit-section dash compat-27
compat-26 compat compat-macs paredit edmacro icomplete time battery
exwm-randr xcb-randr exwm-config exwm exwm-input xcb-keysyms xcb-xkb
exwm-manage exwm-floating xcb-cursor xcb-render exwm-layout
exwm-workspace exwm-core xcb-ewmh xcb-icccm xcb xcb-xproto xcb-types
xcb-debug kmacro server stimmung-themes modus-operandi-theme
modus-themes ytdious osm mingus libmpdee reporter edebug debug backtrace
transmission diary-lib diary-loaddefs color calc-bin calc-ext calc
calc-loaddefs rect calc-macs w3m-load mu4e mu4e-org mu4e-main mu4e-view
mu4e-headers mu4e-compose mu4e-draft mu4e-actions smtpmail mu4e-search
mu4e-lists mu4e-bookmarks mu4e-mark mu4e-message flow-fill mule-util
hl-line mu4e-contacts mu4e-update mu4e-folders mu4e-server mu4e-context
mu4e-vars mu4e-helpers mu4e-config bookmark ido supercite regi
ebdb-message ebdb-gnus gnus-msg gnus-art mm-uu mml2015 mm-view mml-smime
smime gnutls dig gnus-sum shr pixel-fill kinsoku url-file svg dom
gnus-group gnus-undo gnus-start gnus-dbus gnus-cloud nnimap nnmail
mail-source utf7 nnoo gnus-spec gnus-int gnus-range message sendmail
yank-media puny rfc822 mml mml-sec epa epg rfc6068 epg-config mm-decode
mm-bodies mm-encode mail-parse rfc2231 rfc2047 rfc2045 ietf-drums
gmm-utils mailheader gnus-win gnus nnheader gnus-util mail-utils range
mm-util mail-prsvr ebdb-mua ebdb-com crm ebdb-format ebdb mailabbrev
eieio-opt speedbar ezimage dframe eieio-base pcase timezone org ob
ob-tangle ob-ref ob-lob ob-table ob-exp org-macro org-footnote org-src
ob-comint org-pcomplete org-list org-faces org-entities org-version
ob-emacs-lisp ob-core ob-eval org-table oc-basic bibtex ol org-keys oc
org-compat org-macs org-loaddefs find-func cal-menu calendar
cal-loaddefs visual-basic-mode cl web-mode disp-table erlang-start
smart-tabs-mode skeleton cc-mode cc-fonts cc-guess cc-menus cc-cmds
cc-styles cc-align cc-engine cc-vars cc-defs slime-asdf grep slime-tramp
tramp tramp-loaddefs trampver tramp-integration cus-edit cus-load
wid-edit files-x tramp-compat rx shell pcomplete parse-time iso8601
time-date ls-lisp format-spec slime-fancy slime-indentation
slime-cl-indent cl-indent slime-trace-dialog slime-fontifying-fu
slime-package-fu slime-references slime-compiler-notes-tree
slime-scratch slime-presentations advice bridge slime-macrostep
macrostep slime-mdot-fu slime-enclosing-context slime-fuzzy
slime-fancy-trace slime-fancy-inspector slime-c-p-c
slime-editing-commands slime-autodoc slime-repl elp slime-parse slime
derived cl-extra help-mode lisp-mnt gud apropos compile
text-property-search etags fileloop generator xref project arc-mode
archive-mode noutline outline icons pp comint osc ansi-color ring
hyperspec thingatpt slime-autoloads dired-aux dired-x dired
dired-loaddefs so-long notifications dbus xml repeat easy-mmode
auctex-autoloads tex-site debbugs-autoloads hyperbole-autoloads
magit-autoloads git-commit-autoloads magit-section-autoloads
dash-autoloads paredit-autoloads rust-mode-autoloads
stimmung-themes-autoloads transient-autoloads with-editor-autoloads info
compat-autoloads ytdious-autoloads package browse-url url url-proxy
url-privacy url-expand url-methods url-history url-cookie
generate-lisp-file url-domsuf url-util mailcap url-handlers url-parse
auth-source cl-seq eieio eieio-core cl-macs password-cache json subr-x
map byte-opt gv bytecomp byte-compile cconv url-vars cl-loaddefs cl-lib
rmc iso-transl tooltip eldoc paren electric uniquify ediff-hook vc-hooks
lisp-float-type elisp-mode mwheel term/x-win x-win term/common-win x-dnd
tool-bar dnd fontset image regexp-opt fringe tabulated-list replace
newcomment text-mode lisp-mode prog-mode register page tab-bar menu-bar
rfn-eshadow isearch easymenu timer select scroll-bar mouse jit-lock
font-lock syntax font-core term/tty-colors frame minibuffer nadvice seq
simple cl-generic indonesian philippine cham georgian utf-8-lang
misc-lang vietnamese tibetan thai tai-viet lao korean japanese eucjp-ms
cp51932 hebrew greek romanian slovak czech european ethiopic indian
cyrillic chinese composite emoji-zwj charscript charprop case-table
epa-hook jka-cmpr-hook help abbrev obarray oclosure cl-preloaded button
loaddefs faces cus-face macroexp files window text-properties overlay
sha1 md5 base64 format env code-pages mule custom widget keymap
hashtable-print-readable backquote threads dbusbind kqueue lcms2
dynamic-setting system-font-setting font-render-setting cairo xinput2 x
multi-tty make-network-process emacs)

Memory information:
((conses 16 825266 63855)
 (symbols 48 57333 3)
 (strings 32 172702 10415)
 (string-bytes 1 5701254)
 (vectors 16 97903)
 (vector-slots 8 1330069 88622)
 (floats 8 541 702)
 (intervals 56 39795 1082)
 (buffers 1000 26))

-- 
Manuel Giraud

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#58021; Package emacs. (Fri, 23 Sep 2022 10:08:02 GMT) Full text and rfc822 format available.

Message #8 received at 58021 <at> debbugs.gnu.org (full text, mbox):

From: Po Lu <luangruo <at> yahoo.com>
To: Manuel Giraud <manuel <at> ledu-giraud.fr>
Cc: 58021 <at> debbugs.gnu.org
Subject: Re: bug#58021: 29.0.50; Remove old font.c code
Date: Fri, 23 Sep 2022 18:07:06 +0800
Manuel Giraud <manuel <at> ledu-giraud.fr> writes:

> Hi,
>
> Here is patch that removes some old "#if 0" code from font.c.  Most of
> this code seems to be "#if 0" since 2006 (when font.c was introduced)
> and was only "updated" for overall interfaces change.
>
> Maybe it was kept as hints for future readers but I found this code to
> be a bit misleading as it has drifted along from how things are actually
> done now.

I think Eli insisted on keeping that piece of code.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#58021; Package emacs. (Fri, 23 Sep 2022 10:50:02 GMT) Full text and rfc822 format available.

Message #11 received at 58021 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Manuel Giraud <manuel <at> ledu-giraud.fr>
Cc: 58021 <at> debbugs.gnu.org
Subject: Re: bug#58021: 29.0.50; Remove old font.c code
Date: Fri, 23 Sep 2022 13:49:58 +0300
> From: Manuel Giraud <manuel <at> ledu-giraud.fr>
> Date: Fri, 23 Sep 2022 11:11:22 +0200
> 
> Here is patch that removes some old "#if 0" code from font.c.  Most of
> this code seems to be "#if 0" since 2006 (when font.c was introduced)
> and was only "updated" for overall interfaces change.
> 
> Maybe it was kept as hints for future readers but I found this code to
> be a bit misleading as it has drifted along from how things are actually
> done now.

FWIW, I see no reason to remove this.  Someone someday may decide to
implement these ideas.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#58021; Package emacs. (Fri, 23 Sep 2022 15:42:02 GMT) Full text and rfc822 format available.

Message #14 received at 58021 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 58021 <at> debbugs.gnu.org, Manuel Giraud <manuel <at> ledu-giraud.fr>
Subject: Re: bug#58021: 29.0.50; Remove old font.c code
Date: Fri, 23 Sep 2022 17:41:43 +0200
Eli Zaretskii <eliz <at> gnu.org> writes:

> FWIW, I see no reason to remove this.  Someone someday may decide to
> implement these ideas.

Perhaps we could pick it out and place on a branch?  Because it's a lot
of deaf stuff to wade through when looking at font.c.




Added tag(s) moreinfo. Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Fri, 23 Sep 2022 15:43:01 GMT) Full text and rfc822 format available.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#58021; Package emacs. (Fri, 23 Sep 2022 15:54:02 GMT) Full text and rfc822 format available.

Message #19 received at 58021 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 58021 <at> debbugs.gnu.org, manuel <at> ledu-giraud.fr
Subject: Re: bug#58021: 29.0.50; Remove old font.c code
Date: Fri, 23 Sep 2022 18:53:19 +0300
> From: Lars Ingebrigtsen <larsi <at> gnus.org>
> Cc: Manuel Giraud <manuel <at> ledu-giraud.fr>,  58021 <at> debbugs.gnu.org
> Date: Fri, 23 Sep 2022 17:41:43 +0200
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > FWIW, I see no reason to remove this.  Someone someday may decide to
> > implement these ideas.
> 
> Perhaps we could pick it out and place on a branch?  Because it's a lot
> of deaf stuff to wade through when looking at font.c.

Would it help if we move that to the end of the file?

My primary motivation for leaving this is to try to attract someone to
get involved in maintaining and developing font.c and fontset.c.
We've all but lost our only expert on this, and leaving this part of
Emacs without an active maintainer is bad for us.  I might even add a
comment there that encourages people to finish implementation of at
least some of that stuff, in the hope that this will bring on board
someone capable and motivated.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#58021; Package emacs. (Sat, 24 Sep 2022 09:26:01 GMT) Full text and rfc822 format available.

Message #22 received at 58021 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 58021 <at> debbugs.gnu.org, manuel <at> ledu-giraud.fr
Subject: Re: bug#58021: 29.0.50; Remove old font.c code
Date: Sat, 24 Sep 2022 11:25:38 +0200
Eli Zaretskii <eliz <at> gnu.org> writes:

> Would it help if we move that to the end of the file?
>
> My primary motivation for leaving this is to try to attract someone to
> get involved in maintaining and developing font.c and fontset.c.
> We've all but lost our only expert on this, and leaving this part of
> Emacs without an active maintainer is bad for us.  I might even add a
> comment there that encourages people to finish implementation of at
> least some of that stuff, in the hope that this will bring on board
> someone capable and motivated.

Adding some commentary to it to explain what the purpose of the code is
(and why it hasn't been enabled) would be helpful, at least.

But the code's been around since 2006, apparently, so at some point it's
probably time to admit defeat.  But perhaps adding some text to explain
why it's there will entice somebody to look at it?  And then if that
hasn't happened at the 20 year anniversary of the code, we can revisit
this issue.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#58021; Package emacs. (Mon, 26 Sep 2022 06:37:02 GMT) Full text and rfc822 format available.

Message #25 received at 58021 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 58021 <at> debbugs.gnu.org, manuel <at> ledu-giraud.fr
Subject: Re: bug#58021: 29.0.50; Remove old font.c code
Date: Mon, 26 Sep 2022 09:36:22 +0300
> From: Lars Ingebrigtsen <larsi <at> gnus.org>
> Cc: manuel <at> ledu-giraud.fr,  58021 <at> debbugs.gnu.org
> Date: Sat, 24 Sep 2022 11:25:38 +0200
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > Would it help if we move that to the end of the file?
> >
> > My primary motivation for leaving this is to try to attract someone to
> > get involved in maintaining and developing font.c and fontset.c.
> > We've all but lost our only expert on this, and leaving this part of
> > Emacs without an active maintainer is bad for us.  I might even add a
> > comment there that encourages people to finish implementation of at
> > least some of that stuff, in the hope that this will bring on board
> > someone capable and motivated.
> 
> Adding some commentary to it to explain what the purpose of the code is
> (and why it hasn't been enabled) would be helpful, at least.

Now done.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#58021; Package emacs. (Mon, 26 Sep 2022 10:59:02 GMT) Full text and rfc822 format available.

Message #28 received at 58021 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 58021 <at> debbugs.gnu.org, manuel <at> ledu-giraud.fr
Subject: Re: bug#58021: 29.0.50; Remove old font.c code
Date: Mon, 26 Sep 2022 12:58:00 +0200
Eli Zaretskii <eliz <at> gnu.org> writes:

> Now done.

Thanks.  So I think the conclusion here is that we don't want to remove
the commented-out code (at this time, at least), so I'm closing this bug
report.




Added tag(s) wontfix. Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Mon, 26 Sep 2022 10:59:02 GMT) Full text and rfc822 format available.

bug closed, send any further explanations to 58021 <at> debbugs.gnu.org and Manuel Giraud <manuel <at> ledu-giraud.fr> Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Mon, 26 Sep 2022 10:59:02 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Mon, 24 Oct 2022 11:24:05 GMT) Full text and rfc822 format available.

This bug report was last modified 1 year and 157 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.