bug#14734: 24.3.50; REGRESSION: defadvice broken wrt doc strings (C-h f)

[Drew Adams]

> > The change introduced is horrible for users.  Now they see only
> > the original doc string, plus a link that will be incomprehensible
> > to many (so skipped over by them):
> 
> You're overdramatizing. It's less convenient, but far from "horrible".
> 
> > Not only can Emacs do better, it always HAS.  This is a real step
> > backward for users.  Intentional or not.  I'm reopening the bug.
> > I hope you will seriously consider reverting the misguided changes
> > that introduced this regression.
> 
> It's obviously the result of advice.el being re-implemented using
> nadvice.el, for backward compatibility. Having a minor regression in
> functionality in this kind of situation is fairly normal.

If your point of view is only that of an implementor, you see only
"less convenient", "minor regression", and "fairly normal".  You see
the regression as just "a result of ... being reimplemented", as if
design and the user experience do not matter.  Implementation leads.

If your point of view is that of a user, the result is a real step
backward and, yes, pretty horrible.  The design should lead, for users.

One opinion, of course.

And just why did something already implemented (and stable for years)
need to be REimplemented "for backward compatibility"?  Why did adding
something new and different and presumably better require ALSO
reimplementing something that was already, by definition, backward
compatible?