describe-function etc commands should inform about el-patched stuff as advice do
caadar opened this issue · 7 comments
Please consider to add reminders in the Help system about patched items as it so likely will be forgotten otherwise.
Advice commands provide such glue, so it's possible.
Note that while defining a patch, you actually have complete control over changing the docstring. Thus, you can put an exact human-readable description of what you have changed into the new docstring. I might want to make this easier by providing a way to patch a string without swapping it out entirely (undesirable for long docstrings).
Even so I agree it might be nice to have a [Note: this function has been patched by `el-patch' while loading `init.el'.] reminder inserted automatically. Shouldn't be too complicated but I may not get to it soon.
Ok, will use noted docstrings in the meantime.
Also, note that M-x el-patch-validate will list all your patches, so you can check if a function is patched that way.
Thank you, I already use it. It seems have three aspects:
(a) check new patch for some errors (presume that original file not changed yet),
(b) list patches available,
(c) alarm about changed upstream (main function).
But in case of chase and long jumping by `describe-function' it out of help though...
docstring-note applied only if docstring exist, so info about functions w/o docstring missed, but it's important. Someone may add empty docstrind to effect but it so easy to forget...
Why don't you open a new issue to track this enhancement?