Hi
Just to resume the discussion. I insert below the emails discussing the
two problems I found
I applied the two patches, I attach below and then I run into a problem:
* First problem
Take one of these awful long comment line
ev = 0; % el número
de evaluaciones al inicio es cero, pues no ha habido ninguna iteración
this after running matlab-fill-comment-line to
ev = 0; % el número de evaluaciones al inicio es cero, pues no ha
% habido
% ninguna
% iteración
Which does not look correct to me, shouldn't it be
ev = 0; % el número de evaluaciones al inicio es cero, pues no ha
% habido ninguna iteración
Or do I miss here something
* Second problem.
I tried just to run
matlab-indent-region
on
#+begin_src
ev = 0; % el número de evaluaciones al inicio es cero, pues no ha
% habido
% ninguna
% iteración
#+end_src
and I obtained an error I attach, something is not right here
--- matlab-syntax.el_orig 2022-03-16 21:21:45.648400003 -0400 +++ matlab-syntax.el 2023-05-07 07:54:40.316705480 -0400 @@ -451,7 +451,10 @@ (goto-char (nth 8 pps)) t) - (when all-comments (forward-comment -100000))))) + (when all-comments + (prog1 + (forward-comment -100000) + (skip-chars-forward " \t\n\r")))))) (defun matlab-end-of-string-or-comment (&optional all-comments) "If the cursor is in a string or comment, move to the end.
--- /mathworks/hub/share/sbtools/apps/emacs-add-ons/src/matlab-emacs/matlab-emacs/matlab.el 2023-04-25 14:30:34.743571388 -0400
+++ matlab.el 2023-05-08 08:56:32.582984029 -0400
@@ -1708,7 +1708,7 @@
(defun matlab-beginning-of-command ()
"Go to the beginning of an M command.
Travels across continuations."
- (interactive "P")
+ (interactive)
(matlab-scan-beginning-of-command))
(defun matlab-end-of-command ()
@@ -2400,19 +2400,28 @@
(defun matlab-set-comm-fill-prefix ()
"Set the `fill-prefix' for the current (comment) line."
(interactive)
- (if (matlab-line-comment-p (matlab-compute-line-context 1))
- (setq fill-prefix
- (save-excursion
- (beginning-of-line)
- (let ((e (matlab-point-at-eol))
- (pf nil))
- (while (and (re-search-forward "%+[ \t]*\\($$$ \\|\\* \\)?" e t)
- (matlab-cursor-in-string)))
- (setq pf (match-string 0))
- (when (string-match "%\\s-*\\* " pf)
- (setq pf (concat "%" (make-string (1- (length pf)) ? ))))
- (concat (make-string (- (current-column) (length pf)) ? )
- pf))))))
+ (let ((ctxt (matlab-compute-line-context 1)))
+ (if (matlab-line-comment-p ctxt)
+ (setq fill-prefix
+ (save-excursion
+ (beginning-of-line)
+ (let ((e (matlab-point-at-eol))
+ (pf nil))
+ (while (and (re-search-forward "%+[ \t]*\\($$$ \\|\\* \\)?" e t)
+ (matlab-cursor-in-string)))
+ (setq pf (match-string 0))
+ (when (string-match "%\\s-*\\* " pf)
+ (setq pf (concat "%" (make-string (1- (length pf)) ? ))))
+ (concat (make-string (- (current-column) (length pf)) ? )
+ pf))))
+ ;; Not a comment line, but maybe a comment at end of a line? If
+ ;; so make next comment line up with the end of line comment
+ ;; column the way aoto fill does.
+ (let ((cstart (matlab-line-end-comment-column ctxt)))
+ (if cstart
+ (setq fill-prefix
+ (concat (make-string cstart ? ) "% "))
+ )) )))
(defun matlab-set-comm-fill-prefix-post-code ()
"Set the `fill-prefix' for the current post-code comment line."
@@ -2564,7 +2573,7 @@
((or (matlab-line-comment-p lvl1)
(and (save-excursion (move-to-column fill-column)
(matlab-cursor-in-comment))
- (matlab-line-comment-p lvl1)))
+ (matlab-line-end-comment-column lvl1)))
;; If the whole line is a comment, do this.
(matlab-set-comm-fill-prefix) (do-auto-fill)
(matlab-reset-fill-prefix))
@@ -2672,35 +2681,9 @@
"Fill the current comment line.
With optional argument, JUSTIFY the comment as well."
(interactive)
- (if (not (matlab-comment-on-line))
- (error "No comment to fill"))
- (beginning-of-line)
- ;; First, find the beginning of this comment...
- (while (and (looking-at matlab-cline-start-skip)
- (not (bobp)))
- (forward-line -1)
- (beginning-of-line))
- (if (not (looking-at matlab-cline-start-skip))
- (forward-line 1))
- ;; Now scan to the end of this comment so we have our outer bounds,
- ;; and narrow to that region.
- (save-restriction
- (narrow-to-region (point)
- (save-excursion
- (while (and (looking-at matlab-cline-start-skip)
- (not (save-excursion (end-of-line) (eobp))))
- (forward-line 1)
- (beginning-of-line))
- (if (not (looking-at matlab-cline-start-skip))
- (forward-line -1))
- (end-of-line)
- (point)))
- ;; Find the fill prefix...
- (matlab-comment-on-line)
- (looking-at "%[ \t]*")
- (let ((fill-prefix (concat (make-string (current-column) ? )
- (match-string 0))))
- (fill-region (point-min) (point-max) justify))))
+ ;; Set the fill prefix
+ (matlab-set-comm-fill-prefix)
+ (fill-paragraph justify))
(defun matlab-justify-line ()
"Delete space on end of line and justify."
@@ -2724,7 +2707,7 @@
(let ((paragraph-separate "%%\\|%[a-zA-Z]\\|%[ \t]*$\\|[ \t]*$")
(paragraph-start "%[a-zA-Z]\\|%[ \t]*$\\|[ \t]*$\\|%\\s-*\\*")
(paragraph-ignore-fill-prefix nil)
- (start (save-excursion (matlab-scan-beginning-of-command)
+ (start (save-excursion (matlab-beginning-of-string-or-comment t)
(if (looking-at "%%")
(progn (end-of-line)
(forward-char 1)))
Debugger entered--Lisp error: (wrong-number-of-arguments #f(compiled-function (start end &optional column noprogress) (interactive nil) #<bytecode 0xbf403742a90dff7>) 0) matlab-indent-region() funcall-interactively(matlab-indent-region) call-interactively(matlab-indent-region record nil) command-execute(matlab-indent-region record) execute-extended-command(nil "matlab-indent-region" "matlab-indent-re") funcall-interactively(execute-extended-command nil "matlab-indent-region" "matlab-indent-re") call-interactively(execute-extended-command nil nil) command-execute(execute-extended-command)
-- Warning: Content may be disturbing to some audiences I strongly condemn Putin's war of aggression against the Ukraine. I support to deliver weapons to Ukraine's military. I support the NATO membership of the Ukraine. I support the EU membership of the Ukraine. https://addons.thunderbird.net/en-US/thunderbird/addon/gmail-conversation-view/
smime.p7s
Description: S/MIME cryptographic signature
_______________________________________________ Matlab-emacs-discuss mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/matlab-emacs-discuss
