Gnus development mailing list
 help / color / mirror / Atom feed
From: Lars Ingebrigtsen <larsi@gnus.org>
To: emacs-devel@gnu.org
Cc: ding@gnus.org
Subject: Re: Gnus cleanup
Date: Fri, 12 Feb 2016 15:19:25 +1100	[thread overview]
Message-ID: <87fuwyy302.fsf@gnus.org> (raw)
In-Reply-To: <87h9hied21.fsf@gnus.org>

I've now completed the first pass over the Gnus sources, removing almost
all XEmacs compat code.  It has, of course, not all been tested very
well, but it should be "obviously correct".  :-)  Virtually all of the
changes involve either removing the first part after
(if (featurep 'xemacs) ...) and leaving the "else" part in place.

The compat function removal can be a little bit more invasive, but it's
basically usually just replacing gnus-called-interactively-p with
called-interactively-p, so it should also be "obviously" correct.

But please do submit bug reports if this all blows up.

The next thing I was going to do is grep for "boundp" and look whether
the code looks like compat code or not.  This may introduce actual bugs,
because various builds of Emacs may have various things defined, but
I'll try to be careful.

But instead of jumping straight into that, I think I'm going to make the
Gnus build warning free first, which should help when doing cleanup, I
think.  But, again, these will be "real changes", and may introduce
bugs.  But I hope not.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





  parent reply	other threads:[~2016-02-12  4:19 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-09  4:13 Lars Ingebrigtsen
2016-02-09  8:35 ` Michael Albinus
2016-02-09  8:35 ` Eric S Fraga
2016-02-09 11:42 ` Malcolm Purvis
2016-02-09 15:30 ` Stefan Monnier
2016-02-09 23:08   ` Lars Ingebrigtsen
2016-02-10  0:50 ` Greg Troxel
2016-02-12  4:19 ` Lars Ingebrigtsen [this message]
2016-02-13  0:22   ` Frank Haun
2016-02-13  1:57   ` Eric Abrahamsen
2016-02-13  3:17     ` Eric Abrahamsen
2016-02-13  3:43       ` Lars Ingebrigtsen
2016-02-13  4:26         ` Eric Abrahamsen
2016-02-13  4:59           ` Lars Ingebrigtsen
2016-02-13 12:00   ` Frank Haun
2016-02-13 15:37     ` Richard Stallman
2016-02-14  2:35     ` Lars Ingebrigtsen
2016-02-14  3:13       ` Adam Sjøgren
2016-02-14 13:12       ` Frank Haun
2016-02-17 21:10       ` Frank Haun
2016-02-18 12:29         ` Frank Haun
2016-03-07 13:52   ` Ted Zlatanov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87fuwyy302.fsf@gnus.org \
    --to=larsi@gnus.org \
    --cc=ding@gnus.org \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).