emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#6556: closed (html-mode does not align to prev non


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#6556: closed (html-mode does not align to prev non-empty line when indenting)
Date: Tue, 07 Feb 2017 03:44:01 +0000

Your message dated Mon, 06 Feb 2017 20:43:00 -0700
with message-id <address@hidden>
and subject line works for me
has caused the debbugs.gnu.org bug report #6556,
regarding html-mode does not align to prev non-empty line when indenting
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
6556: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=6556
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: html-mode does not align to prev non-empty line when indenting Date: Sat, 3 Jul 2010 02:14:23 +0200
In a buffer like this

   <p>
      row 1
         row 2
     row 3

the "row 3" gets indented like above. I think it instead should be
aligned with row2. (I need this for mumamo.)

The comment in sgml-calculate-indent seems to say that this is the
intention, but the code is maybe unfinished, line 1483:

            (context
             ;; If possible, align on the previous non-empty text line.
             ;; Otherwise, do a more serious parsing to find the
             ;; tag(s) relative to which we should be indenting.
             (if (and (not unclosed) (skip-chars-backward " \t")
                      (< (skip-chars-backward " \t\n") 0)
                      (back-to-indentation)
                      (> (point) (cdr lcon)))
                 nil
               (goto-char here)
               (nreverse (sgml-get-context (if unclosed nil 'empty)))))

I do not know how it is supposed to work, but I notice that
(back-to-indentation) is not supposed to return anything special so
the argument to if may fail (and this seems to happen). Writing that
part as


                    (progn
                      (back-to-indentation)
                      (> (point) (cdr lcon))))


seems to work as expected.



--- End Message ---
--- Begin Message --- Subject: works for me Date: Mon, 06 Feb 2017 20:43:00 -0700 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1.91 (gnu/linux)
I tried this and it works for me, so I'm closing this bug.

Tom


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]