Skip to content

clojure-mode.el (clojure-docstring-fill-prefix): New option #244

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Jul 27, 2014
Merged

Conversation

bzg
Copy link
Contributor

@bzg bzg commented Jul 27, 2014

  • clojure-mode.el (clojure-docstring-fill-prefix): New option.

See #241 for a discussion about this.

* clojure-mode.el (clojure-docstring-fill-prefix): New option.
@@ -278,6 +288,8 @@ ENDP and DELIMITER."
(setq-local parse-sexp-ignore-comments t)
(clojure-mode-font-lock-setup)
(setq-local open-paren-in-column-0-is-defun-start nil)
(setq clojure-docstring-fill-prefix-string
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps we should generate this dynamically to avoid having to restart clojure-mode (meaning that clojure-docstring-fill-prefix-string should be a function).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bozhidar Batsov notifications@github.com writes:

Perhaps we should generate this dynamically to avoid having to
restart clojure-mode.

Done in latest PR.

…fix-width'

* clojure-mode.el (clojure-docstring-fill-prefix-width): Rename
from `clojure-docstring-fill-prefix'.
(clojure-docstring-fill-prefix-string): Remove.
(clojure-mode, clojure-adaptive-fill-function)
(clojure-fill-paragraph, clojure-indent-line): Don't use
`clojure-docstring-fill-prefix-string'.
@bbatsov
Copy link
Member

bbatsov commented Jul 27, 2014

👍

bbatsov added a commit that referenced this pull request Jul 27, 2014
clojure-mode.el (clojure-docstring-fill-prefix): New option
@bbatsov bbatsov merged commit 47ed969 into clojure-emacs:master Jul 27, 2014
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants