all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Michael Heerdegen <michael_heerdegen@web.de>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: 47813@debbugs.gnu.org
Subject: bug#47813: 28.0.50; Confusing new calling convention for define-minor-mode
Date: Fri, 16 Apr 2021 06:26:53 +0200	[thread overview]
Message-ID: <87lf9isvle.fsf@web.de> (raw)
In-Reply-To: <jwvtuo6gbkf.fsf-monnier+emacs@gnu.org> (Stefan Monnier's message of "Thu, 15 Apr 2021 23:23:07 -0400")

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> That's why I introduced the warning and changed the arglist: I realized
> that even though I think the old convention is inconvenient and
> confusing, it seems that a lot of code (even brand new code) follows the
> old rather than the new convention, so we need to be more proactive to
> get people to abandon the old convention so we can at some point in the
> future accept code like the one above.

Ok.


My problem was: When compiling my init file (I have a lot of personal,
quite trivial, minor modes) I got 20 warnings, a whole bulk, then I did
what the warnings said and deleted all those nil's, and then everything
was broken.

It's now hard to find out how to get your code correct and warning free
at all.  Can we do something better to help people find the correct
solution?

Regards,

Michael.





  parent reply	other threads:[~2021-04-16  4:26 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-16  1:51 bug#47813: 28.0.50; Confusing new calling convention for define-minor-mode Michael Heerdegen
2021-04-16  3:23 ` Stefan Monnier
2021-04-16  3:27   ` Stefan Monnier
2021-04-16  4:26   ` Michael Heerdegen [this message]
2021-04-16 12:43     ` Stefan Monnier
2021-04-16 23:53       ` Michael Heerdegen
2021-04-17  0:44         ` Stefan Monnier
2021-04-17  2:28           ` Michael Heerdegen
2021-04-17  3:42             ` Stefan Monnier
2021-04-17 11:49               ` Lars Ingebrigtsen
2021-04-17 13:57                 ` Stefan Monnier
2021-04-17 23:21                   ` Michael Heerdegen
2021-04-18  3:56                     ` Stefan Monnier

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=87lf9isvle.fsf@web.de \
    --to=michael_heerdegen@web.de \
    --cc=47813@debbugs.gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.