Re: Integrate my package into Emacs

[Stefan Monnier]

> See patch below which includes the copyright change along with various
> FIXME comments (meant to be read rather than applied.  If you have any
> questions/answers about those FIXMEs, feel free to send them right here).

One more comment: it would be good to expand the docstring of
`cl-defgeneric inspect-object` to clarify what is expected from methods
of this generic function.  E.g. specify that they're supposed to write
into the current-buffer, clarify if that buffer can be presumed to be
empty or not and what kind of formatting it can/should obey
(e.g. mention that they're expected to use
`inspector--insert-inspect-button` when inserting a sub-object).

Also, I'm curious why you used a default method for that generic
function where you have a big `cond` dispatching on the type of the
argument: why not use separate methods instead?


        Stefan