all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#33989: 27.0.50; master: Tcl indentation broken
@ 2019-01-05 17:19 Benjamin Riefenstahl
  2019-01-05 18:10 ` Eli Zaretskii
  2020-08-26 11:34 ` Lars Ingebrigtsen
  0 siblings, 2 replies; 9+ messages in thread
From: Benjamin Riefenstahl @ 2019-01-05 17:19 UTC (permalink / raw)
  To: 33989

Commit cd5bb4bf3d "Fix two tcl-mode defun-related bugs" breaks
indentation of Tcl code for me outside of procs.  Consider:

fooBaz {
    |
}

Where the | indicates the expected indentation.  But on current master
pressing TAB instead goes to the beginning of the line.  Given that it
is common in Tcl to have code on the global level of a script, this is
not what I expect and want.

Reproduced with current master and "emacs -Q".





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#33989: 27.0.50; master: Tcl indentation broken
  2019-01-05 17:19 bug#33989: 27.0.50; master: Tcl indentation broken Benjamin Riefenstahl
@ 2019-01-05 18:10 ` Eli Zaretskii
  2019-01-12 11:43   ` Eli Zaretskii
  2019-01-15 21:42   ` Benjamin Riefenstahl
  2020-08-26 11:34 ` Lars Ingebrigtsen
  1 sibling, 2 replies; 9+ messages in thread
From: Eli Zaretskii @ 2019-01-05 18:10 UTC (permalink / raw)
  To: Benjamin Riefenstahl, Tom Tromey; +Cc: 33989

> From: Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>
> Date: Sat, 05 Jan 2019 18:19:11 +0100
> 
> Commit cd5bb4bf3d "Fix two tcl-mode defun-related bugs" breaks
> indentation of Tcl code for me outside of procs.  Consider:
> 
> fooBaz {
>     |
> }
> 
> Where the | indicates the expected indentation.  But on current master
> pressing TAB instead goes to the beginning of the line.  Given that it
> is common in Tcl to have code on the global level of a script, this is
> not what I expect and want.
> 
> Reproduced with current master and "emacs -Q".

Thanks, I'm CC'ing Tom who made those changes.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#33989: 27.0.50; master: Tcl indentation broken
  2019-01-05 18:10 ` Eli Zaretskii
@ 2019-01-12 11:43   ` Eli Zaretskii
  2019-01-19  7:34     ` Eli Zaretskii
  2019-01-15 21:42   ` Benjamin Riefenstahl
  1 sibling, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2019-01-12 11:43 UTC (permalink / raw)
  To: tom; +Cc: 33989, b.riefenstahl

> Date: Sat, 05 Jan 2019 20:10:40 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: 33989@debbugs.gnu.org

Ping!  Tom, could you please look at this issue?

> > From: Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>
> > Date: Sat, 05 Jan 2019 18:19:11 +0100
> > 
> > Commit cd5bb4bf3d "Fix two tcl-mode defun-related bugs" breaks
> > indentation of Tcl code for me outside of procs.  Consider:
> > 
> > fooBaz {
> >     |
> > }
> > 
> > Where the | indicates the expected indentation.  But on current master
> > pressing TAB instead goes to the beginning of the line.  Given that it
> > is common in Tcl to have code on the global level of a script, this is
> > not what I expect and want.
> > 
> > Reproduced with current master and "emacs -Q".
> 
> Thanks, I'm CC'ing Tom who made those changes.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#33989: 27.0.50; master: Tcl indentation broken
  2019-01-05 18:10 ` Eli Zaretskii
  2019-01-12 11:43   ` Eli Zaretskii
@ 2019-01-15 21:42   ` Benjamin Riefenstahl
  1 sibling, 0 replies; 9+ messages in thread
From: Benjamin Riefenstahl @ 2019-01-15 21:42 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 33989, Tom Tromey

[-- Attachment #1: Type: text/plain, Size: 44 bytes --]

Attaching a patch with a simple test-case.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-tcl-mode-Add-a-test-for-top-level-indentation.patch --]
[-- Type: text/x-diff, Size: 1029 bytes --]

From 65e830aa45fc61536c3aec4af70cb0a58538519a Mon Sep 17 00:00:00 2001
From: Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>
Date: Sat, 12 Jan 2019 15:42:35 +0100
Subject: [PATCH] tcl-mode: Add a test for top level indentation

* test/lisp/progmodes/tcl-tests.el (tcl-mode-global-indent): Add.
---
 test/lisp/progmodes/tcl-tests.el | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/test/lisp/progmodes/tcl-tests.el b/test/lisp/progmodes/tcl-tests.el
index 50c3eba75d..b145244a63 100644
--- a/test/lisp/progmodes/tcl-tests.el
+++ b/test/lisp/progmodes/tcl-tests.el
@@ -72,6 +72,15 @@
       (indent-region (point-min) (point-max))
       (should (equal (buffer-string) text)))))
 
+;; Reproduce bug#33989
+(ert-deftest tcl-mode-global-indent ()
+  (with-temp-buffer
+    (tcl-mode)
+    (let ((text "fooBar {\n    inside\n}\n"))
+      (insert text)
+      (indent-region (point-min) (point-max))
+      (should (equal (buffer-string) text)))))
+
 (provide 'tcl-tests)
 
 ;;; tcl-tests.el ends here
-- 
2.11.0


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* bug#33989: 27.0.50; master: Tcl indentation broken
  2019-01-12 11:43   ` Eli Zaretskii
@ 2019-01-19  7:34     ` Eli Zaretskii
  2019-01-25  8:10       ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2019-01-19  7:34 UTC (permalink / raw)
  To: tom; +Cc: 33989, b.riefenstahl

Ping! Ping!  Thom, could you please look at this?

> Date: Sat, 12 Jan 2019 13:43:57 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: 33989@debbugs.gnu.org, b.riefenstahl@turtle-trading.net
> 
> > Date: Sat, 05 Jan 2019 20:10:40 +0200
> > From: Eli Zaretskii <eliz@gnu.org>
> > Cc: 33989@debbugs.gnu.org
> 
> Ping!  Tom, could you please look at this issue?
> 
> > > From: Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>
> > > Date: Sat, 05 Jan 2019 18:19:11 +0100
> > > 
> > > Commit cd5bb4bf3d "Fix two tcl-mode defun-related bugs" breaks
> > > indentation of Tcl code for me outside of procs.  Consider:
> > > 
> > > fooBaz {
> > >     |
> > > }
> > > 
> > > Where the | indicates the expected indentation.  But on current master
> > > pressing TAB instead goes to the beginning of the line.  Given that it
> > > is common in Tcl to have code on the global level of a script, this is
> > > not what I expect and want.
> > > 
> > > Reproduced with current master and "emacs -Q".
> > 
> > Thanks, I'm CC'ing Tom who made those changes.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#33989: 27.0.50; master: Tcl indentation broken
  2019-01-19  7:34     ` Eli Zaretskii
@ 2019-01-25  8:10       ` Eli Zaretskii
  2019-02-01  9:30         ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2019-01-25  8:10 UTC (permalink / raw)
  To: tom; +Cc: 33989, b.riefenstahl

Ping! Ping! Ping!  Tom, please look into this problem.

> Date: Sat, 19 Jan 2019 09:34:17 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: 33989@debbugs.gnu.org, b.riefenstahl@turtle-trading.net
> 
> Ping! Ping!  Thom, could you please look at this?
> 
> > Date: Sat, 12 Jan 2019 13:43:57 +0200
> > From: Eli Zaretskii <eliz@gnu.org>
> > Cc: 33989@debbugs.gnu.org, b.riefenstahl@turtle-trading.net
> > 
> > > Date: Sat, 05 Jan 2019 20:10:40 +0200
> > > From: Eli Zaretskii <eliz@gnu.org>
> > > Cc: 33989@debbugs.gnu.org
> > 
> > Ping!  Tom, could you please look at this issue?
> > 
> > > > From: Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>
> > > > Date: Sat, 05 Jan 2019 18:19:11 +0100
> > > > 
> > > > Commit cd5bb4bf3d "Fix two tcl-mode defun-related bugs" breaks
> > > > indentation of Tcl code for me outside of procs.  Consider:
> > > > 
> > > > fooBaz {
> > > >     |
> > > > }
> > > > 
> > > > Where the | indicates the expected indentation.  But on current master
> > > > pressing TAB instead goes to the beginning of the line.  Given that it
> > > > is common in Tcl to have code on the global level of a script, this is
> > > > not what I expect and want.
> > > > 
> > > > Reproduced with current master and "emacs -Q".
> > > 
> > > Thanks, I'm CC'ing Tom who made those changes.
> 
> 
> 
> 





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#33989: 27.0.50; master: Tcl indentation broken
  2019-01-25  8:10       ` Eli Zaretskii
@ 2019-02-01  9:30         ` Eli Zaretskii
  0 siblings, 0 replies; 9+ messages in thread
From: Eli Zaretskii @ 2019-02-01  9:30 UTC (permalink / raw)
  To: tom; +Cc: 33989, b.riefenstahl

Ping! Ping! Ping! Ping!  Tom, could you please look at this issue?

> Date: Fri, 25 Jan 2019 10:10:09 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: 33989@debbugs.gnu.org, b.riefenstahl@turtle-trading.net
> 
> Ping! Ping! Ping!  Tom, please look into this problem.
> 
> > Date: Sat, 19 Jan 2019 09:34:17 +0200
> > From: Eli Zaretskii <eliz@gnu.org>
> > Cc: 33989@debbugs.gnu.org, b.riefenstahl@turtle-trading.net
> > 
> > Ping! Ping!  Thom, could you please look at this?
> > 
> > > Date: Sat, 12 Jan 2019 13:43:57 +0200
> > > From: Eli Zaretskii <eliz@gnu.org>
> > > Cc: 33989@debbugs.gnu.org, b.riefenstahl@turtle-trading.net
> > > 
> > > > Date: Sat, 05 Jan 2019 20:10:40 +0200
> > > > From: Eli Zaretskii <eliz@gnu.org>
> > > > Cc: 33989@debbugs.gnu.org
> > > 
> > > Ping!  Tom, could you please look at this issue?
> > > 
> > > > > From: Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>
> > > > > Date: Sat, 05 Jan 2019 18:19:11 +0100
> > > > > 
> > > > > Commit cd5bb4bf3d "Fix two tcl-mode defun-related bugs" breaks
> > > > > indentation of Tcl code for me outside of procs.  Consider:
> > > > > 
> > > > > fooBaz {
> > > > >     |
> > > > > }
> > > > > 
> > > > > Where the | indicates the expected indentation.  But on current master
> > > > > pressing TAB instead goes to the beginning of the line.  Given that it
> > > > > is common in Tcl to have code on the global level of a script, this is
> > > > > not what I expect and want.
> > > > > 
> > > > > Reproduced with current master and "emacs -Q".
> > > > 
> > > > Thanks, I'm CC'ing Tom who made those changes.





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#33989: 27.0.50; master: Tcl indentation broken
  2019-01-05 17:19 bug#33989: 27.0.50; master: Tcl indentation broken Benjamin Riefenstahl
  2019-01-05 18:10 ` Eli Zaretskii
@ 2020-08-26 11:34 ` Lars Ingebrigtsen
  2020-08-30 12:20   ` Benjamin Riefenstahl
  1 sibling, 1 reply; 9+ messages in thread
From: Lars Ingebrigtsen @ 2020-08-26 11:34 UTC (permalink / raw)
  To: Benjamin Riefenstahl; +Cc: 33989

Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net> writes:

> Commit cd5bb4bf3d "Fix two tcl-mode defun-related bugs" breaks
> indentation of Tcl code for me outside of procs.  Consider:
>
> fooBaz {
>     |
> }

This was apparently fixed by:

commit 518ff50f7a245f437576c5f7e716be9cba336287
Author:     Rolf Ade <rolf@pointsman.de>
AuthorDate: Sun Oct 13 05:18:36 2019 +0200
Commit:     Lars Ingebrigtsen <larsi@gnus.org>
CommitDate: Sun Oct 13 05:18:36 2019 +0200

    TCL indentation fix outside functions
    
    * lisp/progmodes/tcl.el (tcl-calculate-indent): Indent code
    correctly outside of function definitions (bug#23565).

I'm unable to reproduce the bug in the included test case, at least, so
I'm closing this bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 9+ messages in thread

* bug#33989: 27.0.50; master: Tcl indentation broken
  2020-08-26 11:34 ` Lars Ingebrigtsen
@ 2020-08-30 12:20   ` Benjamin Riefenstahl
  0 siblings, 0 replies; 9+ messages in thread
From: Benjamin Riefenstahl @ 2020-08-30 12:20 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 33989

Lars Ingebrigtsen writes:
> This was apparently fixed by:

Thanks, looks good.

benny





^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-08-30 12:20 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-01-05 17:19 bug#33989: 27.0.50; master: Tcl indentation broken Benjamin Riefenstahl
2019-01-05 18:10 ` Eli Zaretskii
2019-01-12 11:43   ` Eli Zaretskii
2019-01-19  7:34     ` Eli Zaretskii
2019-01-25  8:10       ` Eli Zaretskii
2019-02-01  9:30         ` Eli Zaretskii
2019-01-15 21:42   ` Benjamin Riefenstahl
2020-08-26 11:34 ` Lars Ingebrigtsen
2020-08-30 12:20   ` Benjamin Riefenstahl

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.