all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Reto Zimmermann <reto@gnu.org>
To: Glenn Morris <rgm@gnu.org>
Cc: Stefan Monnier <monnier@iro.umontreal.ca>, emacs-devel@gnu.org
Subject: Re: [Emacs-diffs] trunk r116878: Sync with upstream vhdl mode v3.35.1
Date: Fri, 28 Mar 2014 13:50:37 +0100	[thread overview]
Message-ID: <20140328135037.Horde.Hkmu2nCKs7FDnFJSl377KA1@webmail.retoweb.net> (raw)
In-Reply-To: <nf8urxrm1r.fsf@fencepost.gnu.org>


Quoting Glenn Morris <rgm@gnu.org>:

> Stefan Monnier wrote:
>
>>>  ;; function for expanding abbrevs and dabbrevs
>>> -(defalias 'vhdl-expand-abbrev (make-hippie-expand-function
>>> -			       '(try-expand-dabbrev
>>> -				 try-expand-dabbrev-all-buffers
>>> -				 vhdl-try-expand-abbrev)))
>>> +(defun vhdl-expand-abbrev (arg))
>>> +(fset 'vhdl-expand-abbrev (make-hippie-expand-function
>>> +			   '(try-expand-dabbrev
>>> +			     try-expand-dabbrev-all-buffers
>>> +			     vhdl-try-expand-abbrev)))
>>
>> Why??!?!
>
> Yes, that made no sense to me either.
> I meant to think about it before committing, then forgot.
> My random guess would be compat with old Emacs versions.
>
> This is how the code used to be prior to Emacs trunk r108403.
> There's no ChangeLog information associated with that commit.
> Knowing why it was changed that time would have helped.

In 2007 I changed my own copy of vhdl-mode to use defalias.  In 2012 I  
synced some differences from the GNU distribution to my own copy, so  
it went back to defun/fset (because that's what it was in the  
repository).  I didn't think much about it, so I don't think there was  
any other reason than just to be in sync with the GNU version.

There's other things that are probably still different in my own copy  
compared to the GNU version, that I did not sync because of  
compatibility reasons.  I meanwhile dropped compatibility with XEmacs,  
but I still want to prevent any changes that are not compatible with  
older Emacs versions in the copy that I distribute on my site.

Reto




  reply	other threads:[~2014-03-28 12:50 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1WSez3-0007bu-Ro@vcs.savannah.gnu.org>
2014-03-26 12:49 ` [Emacs-diffs] trunk r116878: Sync with upstream vhdl mode v3.35.1 Stefan Monnier
2014-03-26 15:56   ` Glenn Morris
2014-03-28 12:50     ` Reto Zimmermann [this message]
2014-03-28 14:54       ` Stefan Monnier
2014-03-28 19:40         ` Reto Zimmermann
2014-03-28 19:47           ` Glenn Morris
2014-03-28 19:49             ` Reto Zimmermann
2014-03-28 22:53           ` Stefan Monnier
2014-03-29  0:39             ` Reto Zimmermann
2014-03-28 20:39         ` Reto Zimmermann

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=20140328135037.Horde.Hkmu2nCKs7FDnFJSl377KA1@webmail.retoweb.net \
    --to=reto@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=rgm@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.
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.