editing plain source code blocks as well

[Luis-Henriquez-Perez]

It would be nice if separedit could also edit code blocks--regular code blocks I
mean that are not in a string. org-edit-src-code is not as simple and easily
customizable as edit-indirect buffers. Also, it would be more consistent if
every "separate" edit were done using the same backend--edit-indirect.

Before I knew of this project I had been working on using edit-indirect
instead of org-edit-src-code so I have some code on this I could offer. Please
let me know what you think of this idea.

[twlz0ne]

Idea/code are Welcome.

Although it is not directly supported yet, but we can simply wrap a function to do it:

(defun separedit/edit-org-src-block ()
  "Edit org src code block."
  (interactive)
  (when-let* ((datum (when (org-in-src-block-p)
                       (org-element-context)))
              (content-info (org-src--contents-area datum)))
    (separedit
     (separedit-mark-region (nth 0 content-info)
                            (nth 1 content-info)
                            (org-src-get-lang-mode
                             (plist-get (cadr datum) :language))))))

[Luis-Henriquez-Perez]

This looks good!

I have other ideas as well but I'll create separate issues for them.

[Luis-Henriquez-Perez]

There is a bug with separedit/edit-src-block. The #+end_src ends up off. I think this is because the source block had no contents.

[twlz0ne]

(defun separedit/edit-org-src-block ()
  "Edit org src code block."
  (interactive)
  (when-let* ((datum (when (org-in-src-block-p) (org-element-context)))
              (content-info (org-src--contents-area datum)))
    (lexical-let*
        ((ov (when (string-empty-p (nth 2 content-info))
               (make-overlay (nth 1 content-info)
                             (nth 1 content-info) nil nil t)))
         (ad (when ov
               (lambda ()
                 (with-current-buffer (overlay-buffer ov)
                   (save-excursion
                     (goto-char (overlay-end ov))
                     (delete-overlay ov)
                     (unless (= (point) (point-at-bol))
                       (insert "\n")))))))
         (buf (separedit
               (separedit-mark-region (nth 0 content-info)
                                      (nth 1 content-info)
                                      (org-src-get-lang-mode
                                       (plist-get (cadr datum) :language))))))
      (when ad
        (with-current-buffer buf
          (setq-local kill-buffer-hook (append (list ad) kill-buffer-hook)))))))

[Luis-Henriquez-Perez]

I still get the same problem with the new version. When I have time I'll try my
hand at this. I've already tried to do this so I have some code.

Question: would you accept a PR to bring this into separedit?