Re: loading message.el for two small functions

[Bill Wohler]

Richard Stallman <address@hidden> writes:
>       * mail/smtpmail.el: Use with-current-buffer.
>       (message-make-date, message-make-message-id): Autoload when needed.
> 
> message.el is almost 5000 lines.  It is very inefficient to load that
> file just to get these two small functions.  Would you please copy and
> rename the functions instead?

<soapbox>

  I don't know how much time I've spent looking for and fixing bugs in
  code that was fixed in cut-and-pasted code elsewhere. Or how many
  bugs I've seen pop up that were seemingly fixed before. Programmers
  should have a finger cut off every time they cut and paste.

  At the very least, mention the source of the code in the target, and
  vice-versa.

  A far better solution in this case is to break up message.el into
  smaller, more manageable chunks, so one is not loathe to load it to
  share functionality. Indeed, the file should never have gotten
  above, say, 500 lines in the first place.

  This will take a single person a few hours now, but will save days
  in the collective down the road.

</soapbox>

-- 
Bill Wohler <address@hidden>  http://www.newt.com/wohler/  GnuPG ID:610BD9AD
Maintainer of comp.mail.mh FAQ and mh-e. Vote Libertarian!
If you're passed on the right, you're in the wrong lane.