bug#15666: 24.3.50; regression: `defadvice' of a special form

[Drew Adams]

(elisp) `Defining Advice' says that you can use `defadvice' to advise a
special form.  It is quite clear about this:

  "Here, FUNCTION is the name of the function (or macro or special form)
  to be advised.  From now on, we will write just "function" when
  describing the entity being advised, but this always includes macros
                                           ^^^^^^^^^^^^^^^^^^^^
  and special forms."
      ^^^^^^^^^^^^^

And indeed it does...in Emacs 20.  But this was apparently broken,
starting with Emacs 22 (I don't have 21 to test there).  In Emacs 20,
this works fine:

(defadvice defconst (after foo last activate)
  (put (ad-get-arg 0) 'foo-prop t))

In Emacs 22 you get this: (error "Invalid lambda variable [docstring]").
In Emacs 24 you get this:

 (error "Advice impossible: defconst is a special form")

Please reverse the regression, allowing use of `defadvice' with special
forms, as the doc claims and as was in fact the case prior to Emacs 22.


In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2013-10-19 on LEG570
Bzr revision: 114715 rgm@gnu.org-20131019023520-s8mwtib7xcx9e05w
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'