* bug#954: indentation in latex-mode in emacs-23 @ 2008-09-10 7:46 ` Chirok Han 2008-09-10 16:41 ` Stefan Monnier 2008-09-30 9:10 ` bug#954: marked as done (indentation in latex-mode in emacs-23) Emacs bug Tracking System 0 siblings, 2 replies; 13+ messages in thread From: Chirok Han @ 2008-09-10 7:46 UTC (permalink / raw To: bug-gnu-emacs Hi I've been using gnu emacs 22 so far (happily) and recently I compiled and installed cvs emacs-23 (updated today: 2008-09-10). I am very happy with this new version. Thank you! I do lots of latex work (and I don't use auctex). Up to emacs 22, the line following \[ is not indented but now in emacs 23, the line after \[ is indent. For example, in emacs 22, `indent-region' gives \[ abc \] but now in emacs 23 it looks like \[ abc \] This may be a new feature, but I would like the old style. Is there any way I can get back the old behavior? I tried to read tex-mode.el but it was too difficult for me. Thank you. ch ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-10 7:46 ` bug#954: indentation in latex-mode in emacs-23 Chirok Han @ 2008-09-10 16:41 ` Stefan Monnier 2008-09-30 9:10 ` bug#954: marked as done (indentation in latex-mode in emacs-23) Emacs bug Tracking System 1 sibling, 0 replies; 13+ messages in thread From: Stefan Monnier @ 2008-09-10 16:41 UTC (permalink / raw To: Chirok Han; +Cc: 954, bug-gnu-emacs > but now in emacs 23 it looks like > \[ > abc > \] This lookds like a bug in my indentation code which fails to take the \ into account. I'll take a look at it, Stefan ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: marked as done (indentation in latex-mode in emacs-23) 2008-09-10 7:46 ` bug#954: indentation in latex-mode in emacs-23 Chirok Han 2008-09-10 16:41 ` Stefan Monnier @ 2008-09-30 9:10 ` Emacs bug Tracking System 1 sibling, 0 replies; 13+ messages in thread From: Emacs bug Tracking System @ 2008-09-30 9:10 UTC (permalink / raw To: martin rudalics [-- Attachment #1: Type: text/plain, Size: 835 bytes --] Your message dated Tue, 30 Sep 2008 10:58:07 +0200 with message-id <48E1EA1F.101@gmx.at> and subject line Re: bug#954: indentation in latex-mode in emacs-23 has caused the Emacs bug report #954, regarding indentation in latex-mode in emacs-23 to be marked as done. This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact don@donarmstrong.com immediately.) -- 954: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=954 Emacs Bug Tracking System Contact don@donarmstrong.com with problems [-- Attachment #2: Type: message/rfc822, Size: 3873 bytes --] From: "Chirok Han" <chirok.han@gmail.com> To: bug-gnu-emacs@gnu.org Subject: indentation in latex-mode in emacs-23 Date: Wed, 10 Sep 2008 16:46:35 +0900 Message-ID: <221cda020809100046i2c319f8apb9fb572831be0080@mail.gmail.com> Hi I've been using gnu emacs 22 so far (happily) and recently I compiled and installed cvs emacs-23 (updated today: 2008-09-10). I am very happy with this new version. Thank you! I do lots of latex work (and I don't use auctex). Up to emacs 22, the line following \[ is not indented but now in emacs 23, the line after \[ is indent. For example, in emacs 22, `indent-region' gives \[ abc \] but now in emacs 23 it looks like \[ abc \] This may be a new feature, but I would like the old style. Is there any way I can get back the old behavior? I tried to read tex-mode.el but it was too difficult for me. Thank you. ch [-- Attachment #3: Type: message/rfc822, Size: 2129 bytes --] From: martin rudalics <rudalics@gmx.at> To: 954-done@emacsbugs.donarmstrong.com Cc: Chirok Han <chirok.han@gmail.com> Subject: Re: bug#954: indentation in latex-mode in emacs-23 Date: Tue, 30 Sep 2008 10:58:07 +0200 Message-ID: <48E1EA1F.101@gmx.at> Fixed as 2008-09-23 Martin Rudalics <rudalics@gmx.at> * textmodes/tex-mode.el (latex-find-indent): Try to handle escaped close parens correctly. (Bug#954) for indenting the closing paren, and 2008-09-25 Martin Rudalics <rudalics@gmx.at> * textmodes/tex-mode.el (latex-handle-escaped-parens): New variable. (latex-backward-sexp-1, latex-forward-sexp-1): Treat escaped parens specially only if latex-handle-escaped-parens is non-nil. (latex-indent-within-escaped-parens): New option. (latex-find-indent): Bind latex-handle-escaped-parens to latex-indent-within-escaped-parens. Do not treat escaped parens specially when this is nil. (Bug#954) for indenting the text within the parens. Thanks again for reporting, martin. ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 @ 2008-09-23 6:33 martin rudalics 2008-09-23 11:13 ` Chirok Han 0 siblings, 1 reply; 13+ messages in thread From: martin rudalics @ 2008-09-23 6:33 UTC (permalink / raw To: 954; +Cc: Stefan Monnier > but now in emacs 23 it looks like > > \[ > abc > \] I've checked in a fix for this. Please retry. Thanks for reporting, martin. ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-23 6:33 bug#954: indentation in latex-mode in emacs-23 martin rudalics @ 2008-09-23 11:13 ` Chirok Han 2008-09-23 12:28 ` martin rudalics 0 siblings, 1 reply; 13+ messages in thread From: Chirok Han @ 2008-09-23 11:13 UTC (permalink / raw To: martin rudalics; +Cc: 954 Thanks. It is now \[ abc \] I would like \[ abc \] but if that's what you intend. Thanks again. Chirok On Tue, Sep 23, 2008 at 3:33 PM, martin rudalics <rudalics@gmx.at> wrote: >> but now in emacs 23 it looks like >> >> \[ >> abc >> \] > > I've checked in a fix for this. Please retry. > > Thanks for reporting, martin. > ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-23 11:13 ` Chirok Han @ 2008-09-23 12:28 ` martin rudalics 2008-09-23 12:35 ` Leslie Lamport 2008-09-23 12:45 ` Chirok Han 0 siblings, 2 replies; 13+ messages in thread From: martin rudalics @ 2008-09-23 12:28 UTC (permalink / raw To: Chirok Han; +Cc: 954, Leslie Lamport > It is now > \[ > abc > \] > > I would like > \[ > abc > \] > but if that's what you intend. This part of the Emacs 23 behavior is due to that change: 2007-09-21 Glenn Morris <rgm@gnu.org> (latex-forward-sexp-1, latex-backward-sexp-1): Doc fix. Handle escaped parens. If you convinced us that \[ abc \] is more correct LaTeX style than \[ abc \] we might reconsider this. Leslie, what's the preferred way to indent this? martin ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-23 12:28 ` martin rudalics @ 2008-09-23 12:35 ` Leslie Lamport 2008-09-23 12:45 ` Chirok Han 1 sibling, 0 replies; 13+ messages in thread From: Leslie Lamport @ 2008-09-23 12:35 UTC (permalink / raw To: martin rudalics, Chirok Han; +Cc: 954@emacsbugs.donarmstrong.com, Stefan Martin, I have no opinion on how a LaTeX input file should be formatted. Leslie -----Original Message----- From: martin rudalics [mailto:rudalics@gmx.at] Sent: Tuesday, September 23, 2008 5:28 AM To: Chirok Han Cc: 954@emacsbugs.donarmstrong.com; Stefan Monnier; Leslie Lamport Subject: Re: bug#954: indentation in latex-mode in emacs-23 > It is now > \[ > abc > \] > > I would like > \[ > abc > \] > but if that's what you intend. This part of the Emacs 23 behavior is due to that change: 2007-09-21 Glenn Morris <rgm@gnu.org> (latex-forward-sexp-1, latex-backward-sexp-1): Doc fix. Handle escaped parens. If you convinced us that \[ abc \] is more correct LaTeX style than \[ abc \] we might reconsider this. Leslie, what's the preferred way to indent this? martin ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-23 12:28 ` martin rudalics 2008-09-23 12:35 ` Leslie Lamport @ 2008-09-23 12:45 ` Chirok Han 2008-09-24 8:18 ` martin rudalics 1 sibling, 1 reply; 13+ messages in thread From: Chirok Han @ 2008-09-23 12:45 UTC (permalink / raw To: martin rudalics; +Cc: 954, Leslie Lamport Hi Martin, It's not something I can convince you about. And I don't think there is any correct or incorrect formatting style. It's just from my 10 years experience. As I get older, it gets harder to accommodate new things no matter how small they are.... Many thanks for your efforts. I like latex on emacs very much. Chirok On Tue, Sep 23, 2008 at 9:28 PM, martin rudalics <rudalics@gmx.at> wrote: >> It is now >> \[ >> abc >> \] >> >> I would like >> \[ >> abc >> \] >> but if that's what you intend. > > This part of the Emacs 23 behavior is due to that change: > > 2007-09-21 Glenn Morris <rgm@gnu.org> > > (latex-forward-sexp-1, latex-backward-sexp-1): Doc fix. > Handle escaped parens. > > If you convinced us that > > \[ > abc > \] > > is more correct LaTeX style than > > \[ > abc > \] > > we might reconsider this. > > Leslie, what's the preferred way to indent this? > > martin > ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-23 12:45 ` Chirok Han @ 2008-09-24 8:18 ` martin rudalics 2008-09-24 16:48 ` Glenn Morris 0 siblings, 1 reply; 13+ messages in thread From: martin rudalics @ 2008-09-24 8:18 UTC (permalink / raw To: Chirok Han; +Cc: 954 [-- Attachment #1: Type: text/plain, Size: 325 bytes --] > It's not something I can convince you about. And I don't think there > is any correct or incorrect formatting style. It's just from my 10 > years experience. As I get older, it gets harder to accommodate new > things no matter how small they are.... ... sounds convincing. What about the attached patch? martin [-- Attachment #2: 954.diff --] [-- Type: text/plain, Size: 11418 bytes --] *** textmodes/tex-mode.el.~1.218.~ 2008-09-23 10:17:14.531250000 +0200 --- textmodes/tex-mode.el 2008-09-24 09:57:17.203125000 +0200 *************** *** 1482,1487 **** --- 1482,1489 ---- (push-mark) (goto-char spot))) + (defvar latex-handle-escaped-parens t) + ;; Don't think this one actually _needs_ (for the purposes of ;; tex-mode) to handle escaped parens. (defun latex-backward-sexp-1 () *************** *** 1489,1499 **** (let ((pos (point)) (forward-sexp-function)) (backward-sexp 1) ! (cond ((looking-at "\\\\\\(begin\\>\\|[[({]\\)") (signal 'scan-error (list "Containing expression ends prematurely" (point) (prog1 (point) (goto-char pos))))) ! ((looking-at "\\\\\\([])}]\\)") (tex-last-unended-eparen (match-string 1))) ((eq (char-after) ?{) (let ((newpos (point))) --- 1491,1505 ---- (let ((pos (point)) (forward-sexp-function)) (backward-sexp 1) ! (cond ((looking-at ! (if latex-handle-escaped-parens ! "\\\\\\(begin\\>\\|[[({]\\)" ! "\\\\begin\\>")) (signal 'scan-error (list "Containing expression ends prematurely" (point) (prog1 (point) (goto-char pos))))) ! ((and latex-handle-escaped-parens ! (looking-at "\\\\\\([])}]\\)")) (tex-last-unended-eparen (match-string 1))) ((eq (char-after) ?{) (let ((newpos (point))) *************** *** 1528,1539 **** (tex-next-unmatched-end)) ;; A better way to handle this, \( .. \) etc, is probably to ;; temporarily change the syntax of the \ in \( to punctuation. ! ((looking-back "\\\\[])}]") (signal 'scan-error (list "Containing expression ends prematurely" (- (point) 2) (prog1 (point) (goto-char pos))))) ! ((looking-back "\\\\\\([({[]\\)") (tex-next-unmatched-eparen (match-string 1))) (t (goto-char newpos)))))) --- 1534,1547 ---- (tex-next-unmatched-end)) ;; A better way to handle this, \( .. \) etc, is probably to ;; temporarily change the syntax of the \ in \( to punctuation. ! ((and latex-handle-escaped-parens ! (looking-back "\\\\[])}]")) (signal 'scan-error (list "Containing expression ends prematurely" (- (point) 2) (prog1 (point) (goto-char pos))))) ! ((and latex-handle-escaped-parens ! (looking-back "\\\\\\([({[]\\)")) (tex-next-unmatched-eparen (match-string 1))) (t (goto-char newpos)))))) *************** *** 2568,2670 **** (indent-line-to indent) (save-excursion (indent-line-to indent))))))) (defun latex-find-indent (&optional virtual) "Find the proper indentation of text after point. VIRTUAL if non-nil indicates that we're only trying to find the indentation in order to determine the indentation of something else. There might be text before point." ! (save-excursion ! (skip-chars-forward " \t") ! (or ! ;; Stick the first line at column 0. ! (and (= (point-min) (line-beginning-position)) 0) ! ;; Trust the current indentation, if such info is applicable. ! (and virtual (save-excursion (skip-chars-backward " \t&") (bolp)) ! (current-column)) ! ;; Stick verbatim environments to the left margin. ! (and (looking-at "\\\\\\(begin\\|end\\) *{\\([^\n}]+\\)") ! (member (match-string 2) tex-verbatim-environments) ! 0) ! ;; Put leading close-paren where the matching open paren would be. ! (let (escaped) ! (and (or (eq (latex-syntax-after) ?\)) ! ;; Try to handle escaped close parens but keep original ! ;; position if it doesn't work out. ! (setq escaped (looking-at "\\\\\\([])}]\\)"))) ! (ignore-errors ! (save-excursion ! (when escaped ! (goto-char (match-beginning 1))) ! (latex-skip-close-parens) (latex-backward-sexp-1) ! (latex-find-indent 'virtual))))) ! ;; Default (maybe an argument) ! (let ((pos (point)) ! ;; Outdent \item if necessary. ! (indent (if (looking-at tex-indent-item-re) (- tex-indent-item) 0)) ! up-list-pos) ! ;; Find the previous point which determines our current indentation. ! (condition-case err ! (progn ! (latex-backward-sexp-1) ! (while (> (current-column) (current-indentation)) ! (latex-backward-sexp-1))) ! (scan-error ! (setq up-list-pos (nth 2 err)))) ! (cond ! ((= (point-min) pos) 0) ; We're really just indenting the first line. ! ((integerp up-list-pos) ! ;; Have to indent relative to the open-paren. ! (goto-char up-list-pos) ! (if (and (not tex-indent-allhanging) ! (save-excursion ! ;; Make sure we're an argument to a macro and ! ;; that the macro is at the beginning of a line. ! (condition-case nil ! (progn ! (while (eq (char-syntax (char-after)) ?\() ! (forward-sexp -1)) ! (and (eq (char-syntax (char-after)) ?/) ! (progn (skip-chars-backward " \t&") ! (bolp)))) ! (scan-error nil))) ! (> pos (progn (latex-down-list) ! (forward-comment (point-max)) ! (point)))) ! ;; Align with the first element after the open-paren. ! (current-column) ! ;; We're the first element after a hanging brace. (goto-char up-list-pos) ! (+ (if (and (looking-at "\\\\begin *{\\([^\n}]+\\)") ! (member (match-string 1) ! latex-noindent-environments)) ! 0 tex-indent-basic) ! indent (latex-find-indent 'virtual)))) ! ;; We're now at the "beginning" of a line. ! ((not (and (not virtual) (eq (char-after) ?\\))) ! ;; Nothing particular here: just keep the same indentation. ! (+ indent (current-column))) ! ;; We're now looking at a macro call. ! ((looking-at tex-indent-item-re) ! ;; Indenting relative to an item, have to re-add the outdenting. ! (+ indent (current-column) tex-indent-item)) ! (t ! (let ((col (current-column))) ! (if (or (not (eq (char-syntax (or (char-after pos) ?\s)) ?\()) ! ;; Can't be an arg if there's an empty line inbetween. ! (save-excursion (re-search-forward "^[ \t]*$" pos t))) ! ;; If the first char was not an open-paren, there's ! ;; a risk that this is really not an argument to the ! ;; macro at all. ! (+ indent col) ! (forward-sexp 1) ! (if (< (line-end-position) ! (save-excursion (forward-comment (point-max)) ! (point))) ! ;; we're indenting the first argument. ! (min (current-column) (+ tex-indent-arg col)) ! (skip-syntax-forward " ") ! (current-column)))))))))) ;;; DocTeX support (defun doctex-font-lock-^^A () --- 2576,2690 ---- (indent-line-to indent) (save-excursion (indent-line-to indent))))))) + (defcustom latex-indent-within-escaped-parens nil + "Non-nil means add extra indent to text within escaped parens. + When this is non-nil, text within matching pairs of escaped + parens is indented at the column following the open paren. The + default value does not add any extra indent thus providing the + behavior of Emacs 22 and earlier." + :type 'boolean + :group 'tex + :version "23.1") + (defun latex-find-indent (&optional virtual) "Find the proper indentation of text after point. VIRTUAL if non-nil indicates that we're only trying to find the indentation in order to determine the indentation of something else. There might be text before point." ! (let ((latex-handle-escaped-parens latex-indent-within-escaped-parens)) ! (save-excursion ! (skip-chars-forward " \t") ! (or ! ;; Stick the first line at column 0. ! (and (= (point-min) (line-beginning-position)) 0) ! ;; Trust the current indentation, if such info is applicable. ! (and virtual (save-excursion (skip-chars-backward " \t&") (bolp)) ! (current-column)) ! ;; Stick verbatim environments to the left margin. ! (and (looking-at "\\\\\\(begin\\|end\\) *{\\([^\n}]+\\)") ! (member (match-string 2) tex-verbatim-environments) ! 0) ! ;; Put leading close-paren where the matching open paren would be. ! (let (escaped) ! (and (or (eq (latex-syntax-after) ?\)) ! ;; Try to handle escaped close parens but keep ! ;; original position if it doesn't work out. ! (and latex-handle-escaped-parens ! (setq escaped (looking-at "\\\\\\([])}]\\)")))) ! (ignore-errors ! (save-excursion ! (when escaped ! (goto-char (match-beginning 1))) ! (latex-skip-close-parens) ! (latex-backward-sexp-1) ! (latex-find-indent 'virtual))))) ! ;; Default (maybe an argument) ! (let ((pos (point)) ! ;; Outdent \item if necessary. ! (indent (if (looking-at tex-indent-item-re) (- tex-indent-item) 0)) ! up-list-pos) ! ;; Find the previous point which determines our current indentation. ! (condition-case err ! (progn (latex-backward-sexp-1) ! (while (> (current-column) (current-indentation)) ! (latex-backward-sexp-1))) ! (scan-error ! (setq up-list-pos (nth 2 err)))) ! (cond ! ((= (point-min) pos) 0) ; We're really just indenting the first line. ! ((integerp up-list-pos) ! ;; Have to indent relative to the open-paren. (goto-char up-list-pos) ! (if (and (not tex-indent-allhanging) ! (save-excursion ! ;; Make sure we're an argument to a macro and ! ;; that the macro is at the beginning of a line. ! (condition-case nil ! (progn ! (while (eq (char-syntax (char-after)) ?\() ! (forward-sexp -1)) ! (and (eq (char-syntax (char-after)) ?/) ! (progn (skip-chars-backward " \t&") ! (bolp)))) ! (scan-error nil))) ! (> pos (progn (latex-down-list) ! (forward-comment (point-max)) ! (point)))) ! ;; Align with the first element after the open-paren. ! (current-column) ! ;; We're the first element after a hanging brace. ! (goto-char up-list-pos) ! (+ (if (and (looking-at "\\\\begin *{\\([^\n}]+\\)") ! (member (match-string 1) ! latex-noindent-environments)) ! 0 tex-indent-basic) ! indent (latex-find-indent 'virtual)))) ! ;; We're now at the "beginning" of a line. ! ((not (and (not virtual) (eq (char-after) ?\\))) ! ;; Nothing particular here: just keep the same indentation. ! (+ indent (current-column))) ! ;; We're now looking at a macro call. ! ((looking-at tex-indent-item-re) ! ;; Indenting relative to an item, have to re-add the outdenting. ! (+ indent (current-column) tex-indent-item)) ! (t ! (let ((col (current-column))) ! (if (or (not (eq (char-syntax (or (char-after pos) ?\s)) ?\()) ! ;; Can't be an arg if there's an empty line inbetween. ! (save-excursion (re-search-forward "^[ \t]*$" pos t))) ! ;; If the first char was not an open-paren, there's ! ;; a risk that this is really not an argument to the ! ;; macro at all. ! (+ indent col) ! (forward-sexp 1) ! (if (< (line-end-position) ! (save-excursion (forward-comment (point-max)) ! (point))) ! ;; we're indenting the first argument. ! (min (current-column) (+ tex-indent-arg col)) ! (skip-syntax-forward " ") ! (current-column))))))))))) ;;; DocTeX support (defun doctex-font-lock-^^A () ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-24 8:18 ` martin rudalics @ 2008-09-24 16:48 ` Glenn Morris 2008-09-24 17:01 ` martin rudalics ` (2 more replies) 0 siblings, 3 replies; 13+ messages in thread From: Glenn Morris @ 2008-09-24 16:48 UTC (permalink / raw To: martin rudalics; +Cc: 954 martin rudalics wrote: > > It's not something I can convince you about. And I don't think there > > is any correct or incorrect formatting style. It's just from my 10 > > years experience. As I get older, it gets harder to accommodate new > > things no matter how small they are.... > > ... sounds convincing. What about the attached patch? Sorry to disagree, but "change bad" is not a convincing argument to me. I prefer the current form because I think that \begin{math} a = b \end{math} and \( a = b \) should be formatted the same. A comment in the original bug that motivated this change seems more convincing to me: http://lists.gnu.org/archive/html/bug-gnu-emacs/2007-09/msg00045.html The correct solution would require that \(, \), \[, and \] are treated the same as \begin{math}, \end{math}, \begin{displaymath}, and \end{displaymath}. (The latter are defined in terms of the former in latex.ltx.) But I have to admit that: i) I use AUCTeX; and ii) I don't use \( \), so take all that with a large grain of salt. :) So please just check that whatever change is installed does not bring back the original tex-validate-region problem: http://lists.gnu.org/archive/html/bug-gnu-emacs/2007-09/msg00038.html ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-24 16:48 ` Glenn Morris @ 2008-09-24 17:01 ` martin rudalics 2008-09-24 20:31 ` Stefan Monnier 2008-09-25 1:07 ` Chirok Han 2 siblings, 0 replies; 13+ messages in thread From: martin rudalics @ 2008-09-24 17:01 UTC (permalink / raw To: Glenn Morris; +Cc: 954 > I prefer the current form because I think that > > \begin{math} > a = b > \end{math} > > and > > \( > a = b > \) > > should be formatted the same. I think so too. On the other hand, indenting habits are persistent. And, according to Leslie, there doesn't seem to exist a convention for indentation of LaTeX source. > A comment in the original bug that > motivated this change seems more convincing to me: > > http://lists.gnu.org/archive/html/bug-gnu-emacs/2007-09/msg00045.html > > The correct solution would require that \(, \), \[, and \] are > treated the same as \begin{math}, \end{math}, \begin{displaymath}, > and \end{displaymath}. (The latter are defined in terms of the > former in latex.ltx.) > > But I have to admit that: i) I use AUCTeX; and ii) I don't use \( \), > so take all that with a large grain of salt. :) I don't know - how does AUCTeX indent this? > So please just check that whatever change is installed does not bring > back the original tex-validate-region problem: > > http://lists.gnu.org/archive/html/bug-gnu-emacs/2007-09/msg00038.html Sure. I strongly hope that `tex-validate-region' doesn't care about indentation. martin ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-24 16:48 ` Glenn Morris 2008-09-24 17:01 ` martin rudalics @ 2008-09-24 20:31 ` Stefan Monnier 2008-09-25 1:07 ` Chirok Han 2 siblings, 0 replies; 13+ messages in thread From: Stefan Monnier @ 2008-09-24 20:31 UTC (permalink / raw To: Glenn Morris; +Cc: 954 >> > It's not something I can convince you about. And I don't think there >> > is any correct or incorrect formatting style. It's just from my 10 >> > years experience. As I get older, it gets harder to accommodate new >> > things no matter how small they are.... >> >> ... sounds convincing. What about the attached patch? > Sorry to disagree, but "change bad" is not a convincing argument to me. > I prefer the current form because I think that > \begin{math} > a = b > \end{math} > and > \( > a = b > \) > should be formatted the same. A comment in the original bug that I don't think there's a right answer. But at least if the above two alternative forms indent differently, that makes it possible for the user to choose the indentation (indirectly) by choosing which form to use. So I'd argue that the two should *not* for indented identically. Stefan PS: The indentation in tex-mode (for which I largely plead guilty) usually indents a good bit more than what I've seen used in most LaTeX files. I tend to think of LaTeX as a programming language, and most of what I write is LaTeX is about programming languages, so I like to indent it more than people who think of it as "text with markup". ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#954: indentation in latex-mode in emacs-23 2008-09-24 16:48 ` Glenn Morris 2008-09-24 17:01 ` martin rudalics 2008-09-24 20:31 ` Stefan Monnier @ 2008-09-25 1:07 ` Chirok Han 2 siblings, 0 replies; 13+ messages in thread From: Chirok Han @ 2008-09-25 1:07 UTC (permalink / raw To: Glenn Morris; +Cc: 954 Hi Glenn and Martin, I have no objections to Glenn's argument. And also many thanks to Martin; I can now figure out which part to modify for customization.... :) Chirok PS. Just one correction. I think Glenn meant \[ abc \] instead of \( abc \) On Thu, Sep 25, 2008 at 1:48 AM, Glenn Morris <rgm@gnu.org> wrote: > martin rudalics wrote: > >> > It's not something I can convince you about. And I don't think there >> > is any correct or incorrect formatting style. It's just from my 10 >> > years experience. As I get older, it gets harder to accommodate new >> > things no matter how small they are.... >> >> ... sounds convincing. What about the attached patch? > > Sorry to disagree, but "change bad" is not a convincing argument to me. > > I prefer the current form because I think that > > \begin{math} > a = b > \end{math} > > and > > \( > a = b > \) > > should be formatted the same. A comment in the original bug that > motivated this change seems more convincing to me: > > http://lists.gnu.org/archive/html/bug-gnu-emacs/2007-09/msg00045.html > > The correct solution would require that \(, \), \[, and \] are > treated the same as \begin{math}, \end{math}, \begin{displaymath}, > and \end{displaymath}. (The latter are defined in terms of the > former in latex.ltx.) > > But I have to admit that: i) I use AUCTeX; and ii) I don't use \( \), > so take all that with a large grain of salt. :) > > So please just check that whatever change is installed does not bring > back the original tex-validate-region problem: > > http://lists.gnu.org/archive/html/bug-gnu-emacs/2007-09/msg00038.html > ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2008-09-30 9:10 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <48E1EA1F.101@gmx.at> 2008-09-10 7:46 ` bug#954: indentation in latex-mode in emacs-23 Chirok Han 2008-09-10 16:41 ` Stefan Monnier 2008-09-30 9:10 ` bug#954: marked as done (indentation in latex-mode in emacs-23) Emacs bug Tracking System 2008-09-23 6:33 bug#954: indentation in latex-mode in emacs-23 martin rudalics 2008-09-23 11:13 ` Chirok Han 2008-09-23 12:28 ` martin rudalics 2008-09-23 12:35 ` Leslie Lamport 2008-09-23 12:45 ` Chirok Han 2008-09-24 8:18 ` martin rudalics 2008-09-24 16:48 ` Glenn Morris 2008-09-24 17:01 ` martin rudalics 2008-09-24 20:31 ` Stefan Monnier 2008-09-25 1:07 ` Chirok Han
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/emacs.git https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.