-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show docstring-note even if docstring is missed #31
Conversation
el-patch.el
Outdated
(setq definition (cl-copy-list definition)) | ||
(setf (nth docstring-idx definition) | ||
new-docstring))))) | ||
(setq definition (cl-copy-list definition)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please turn off indent-tabs-mode
. In fact, I recommend doing this globally, since almost everyone agrees it should never be used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are my changes correct?
el-patch.el
Outdated
(setq definition (cl-copy-list definition)) | ||
(setf (nth docstring-idx definition) | ||
new-docstring))))) | ||
(setq definition (cl-copy-list definition)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this only needs to be in the first branch of the if
clause, since the second branch doesn't actually modify definition
.
el-patch.el
Outdated
@@ -1010,6 +1015,7 @@ This mode is enabled or disabled automatically when the | |||
(provide 'el-patch) | |||
|
|||
;; Local Variables: | |||
;; indent-tabs-mode: nil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did this (and untabified the above code section) in separate commits on develop
.
Sorry for noise. |
Looks good, thanks. |
No description provided.