GNU bug report logs - #62752
html-ts-mode: treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3

Package: emacs;

Reported by: Vincenzo Pupillo <v.pupillo <at> gmail.com>

Date: Mon, 10 Apr 2023 14:33:02 UTC

Severity: normal

Done: Dmitry Gutov <dmitry <at> gutov.dev>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 62752 in the body.
You can then email your comments to 62752 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox

Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#62752; Package emacs. (Mon, 10 Apr 2023 14:33:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Vincenzo Pupillo <v.pupillo <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Mon, 10 Apr 2023 14:33:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Vincenzo Pupillo <v.pupillo <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: html-ts-mode: treesit--simple-indent-eval: Wrong number of arguments:
 #<subr point-min>, 3
Date: Mon, 10 Apr 2023 16:32:35 +0200

[Message part 1 (text/plain, inline)]

Hi, 
I found a bug in html-ts-mode.
If you try to indent this simple html snippet (after enabling html-ts mode) 
and try to indent, say, the first or second line, emacs reports the following 
error:
Matched rule: ((parent-is "fragment") point-min 0)
treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3

Using parent-bol instead of point-min solves the problem.

GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.37, cairo 
version 1.17.6) of 2023-04-09

The patch is attached.

The fragment is as follows:
<!DOCTYPE html>
<html>
<head>
<title>test</title>
<style>
body {
    background-color: linen;
}

h1 {
    color: maroon;
    margin-left: 40px;
}
</style>
<script>
function myFunc(theObject) {
    theObject.make = "Isotta fraschini";
}

const mycar = {
    make: "Iso Rivolta",
    model: "Grifo",
    year: 1965,
};

// x gets the value "Iso Rivolta"
const x = mycar.make;

myFunc(mycar);
// y gets the value "Isotta fraschini"
const y = mycar.make;

</script>
</head>
  <body>
    <h1 style="color:blue;text-align:center;">This is a heading</h1>
    <p style="color:red;">This is a paragraph.</p>
</body>
</html>

Thank you.
V.

[0001-Fixed-the-html-ts-mode-indentation-rule.patch (text/x-patch, attachment)]

Reply sent to Dmitry Gutov <dmitry <at> gutov.dev>:
You have taken responsibility. (Mon, 10 Apr 2023 21:26:01 GMT) Full text and rfc822 format available.

Notification sent to Vincenzo Pupillo <v.pupillo <at> gmail.com>:
bug acknowledged by developer. (Mon, 10 Apr 2023 21:26:02 GMT) Full text and rfc822 format available.

Message #10 received at 62752-done <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dmitry <at> gutov.dev>
To: Vincenzo Pupillo <v.pupillo <at> gmail.com>, 62752-done <at> debbugs.gnu.org
Subject: Re: bug#62752: html-ts-mode: treesit--simple-indent-eval: Wrong
 number of arguments: #<subr point-min>, 3
Date: Tue, 11 Apr 2023 00:24:54 +0300

Hi!

Thanks for the report.

On 10/04/2023 17:32, Vincenzo Pupillo wrote:
> Hi,
> I found a bug in html-ts-mode.
> If you try to indent this simple html snippet (after enabling html-ts mode)
> and try to indent, say, the first or second line, emacs reports the following
> error:
> Matched rule: ((parent-is "fragment") point-min 0)
> treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3
> 
> Using parent-bol instead of point-min solves the problem.
> 
> GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.37, cairo
> version 1.17.6) of 2023-04-09

I think the indention here was to use 'column-0', though. The mode was 
just missed in the update in commit 4c16fd3a51286e1b1.

The practical difference is minimal, but still.

I've pushed this change to master; closing. Let us know if something's 
still amiss, though.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#62752; Package emacs. (Tue, 11 Apr 2023 05:54:02 GMT) Full text and rfc822 format available.

Message #13 received at 62752 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Dmitry Gutov <dmitry <at> gutov.dev>
Cc: v.pupillo <at> gmail.com, 62752 <at> debbugs.gnu.org
Subject: Re: bug#62752: html-ts-mode: treesit--simple-indent-eval: Wrong
 number of arguments: #<subr point-min>, 3
Date: Tue, 11 Apr 2023 08:53:58 +0300

> Resent-To: bug-gnu-emacs <at> gnu.org
> Date: Tue, 11 Apr 2023 00:24:54 +0300
> From: Dmitry Gutov <dmitry <at> gutov.dev>
> 
> > I found a bug in html-ts-mode.
> > If you try to indent this simple html snippet (after enabling html-ts mode)
> > and try to indent, say, the first or second line, emacs reports the following
> > error:
> > Matched rule: ((parent-is "fragment") point-min 0)
> > treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3
> > 
> > Using parent-bol instead of point-min solves the problem.
> > 
> > GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.37, cairo
> > version 1.17.6) of 2023-04-09
> 
> I think the indention here was to use 'column-0', though. The mode was 
> just missed in the update in commit 4c16fd3a51286e1b1.
> 
> The practical difference is minimal, but still.
> 
> I've pushed this change to master; closing. Let us know if something's 
> still amiss, though.

Thanks.

Any reasons not to backport to emacs-29?  This mode is new in Emacs
29, so no compatibility issues should exist.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#62752; Package emacs. (Tue, 11 Apr 2023 08:26:02 GMT) Full text and rfc822 format available.

Message #16 received at 62752 <at> debbugs.gnu.org (full text, mbox):

From: Vincenzo Pupillo <v.pupillo <at> gmail.com>
To: Dmitry Gutov <dmitry <at> gutov.dev>, Eli Zaretskii <eliz <at> gnu.org>
Cc: 62752 <at> debbugs.gnu.org
Subject: Re: bug#62752: html-ts-mode: treesit--simple-indent-eval: Wrong
 number of arguments: #<subr point-min>, 3
Date: Tue, 11 Apr 2023 10:25:12 +0200

Thank you Dmitry.
V.

In data marted� 11 aprile 2023 07:53:58 CEST, Eli Zaretskii ha scritto:
> > Resent-To: bug-gnu-emacs <at> gnu.org
> > Date: Tue, 11 Apr 2023 00:24:54 +0300
> > From: Dmitry Gutov <dmitry <at> gutov.dev>
> > 
> > > I found a bug in html-ts-mode.
> > > If you try to indent this simple html snippet (after enabling html-ts mode)
> > > and try to indent, say, the first or second line, emacs reports the following
> > > error:
> > > Matched rule: ((parent-is "fragment") point-min 0)
> > > treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3
> > > 
> > > Using parent-bol instead of point-min solves the problem.
> > > 
> > > GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.37, cairo
> > > version 1.17.6) of 2023-04-09
> > 
> > I think the indention here was to use 'column-0', though. The mode was 
> > just missed in the update in commit 4c16fd3a51286e1b1.
> > 
> > The practical difference is minimal, but still.
> > 
> > I've pushed this change to master; closing. Let us know if something's 
> > still amiss, though.
> 
> Thanks.
> 
> Any reasons not to backport to emacs-29?  This mode is new in Emacs
> 29, so no compatibility issues should exist.
>

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#62752; Package emacs. (Tue, 11 Apr 2023 19:32:01 GMT) Full text and rfc822 format available.

Message #19 received at 62752 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dmitry <at> gutov.dev>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: v.pupillo <at> gmail.com, 62752 <at> debbugs.gnu.org
Subject: Re: bug#62752: html-ts-mode: treesit--simple-indent-eval: Wrong
 number of arguments: #<subr point-min>, 3
Date: Tue, 11 Apr 2023 22:31:02 +0300

On 11/04/2023 08:53, Eli Zaretskii wrote:
>> Resent-To:bug-gnu-emacs <at> gnu.org
>> Date: Tue, 11 Apr 2023 00:24:54 +0300
>> From: Dmitry Gutov<dmitry <at> gutov.dev>
>>
>>> I found a bug in html-ts-mode.
>>> If you try to indent this simple html snippet (after enabling html-ts mode)
>>> and try to indent, say, the first or second line, emacs reports the following
>>> error:
>>> Matched rule: ((parent-is "fragment") point-min 0)
>>> treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3
>>>
>>> Using parent-bol instead of point-min solves the problem.
>>>
>>> GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.37, cairo
>>> version 1.17.6) of 2023-04-09
>> I think the indention here was to use 'column-0', though. The mode was
>> just missed in the update in commit 4c16fd3a51286e1b1.
>>
>> The practical difference is minimal, but still.
>>
>> I've pushed this change to master; closing. Let us know if something's
>> still amiss, though.
> Thanks.
> 
> Any reasons not to backport to emacs-29?  This mode is new in Emacs
> 29, so no compatibility issues should exist.

html-ts-mode is not in emacs-29.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#62752; Package emacs. (Tue, 11 Apr 2023 19:39:02 GMT) Full text and rfc822 format available.

Message #22 received at 62752 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Dmitry Gutov <dmitry <at> gutov.dev>
Cc: v.pupillo <at> gmail.com, 62752 <at> debbugs.gnu.org
Subject: Re: bug#62752: html-ts-mode: treesit--simple-indent-eval: Wrong
 number of arguments: #<subr point-min>, 3
Date: Tue, 11 Apr 2023 22:38:40 +0300

> Date: Tue, 11 Apr 2023 22:31:02 +0300
> Cc: 62752 <at> debbugs.gnu.org, v.pupillo <at> gmail.com
> From: Dmitry Gutov <dmitry <at> gutov.dev>
> 
> > Any reasons not to backport to emacs-29?  This mode is new in Emacs
> > 29, so no compatibility issues should exist.
> 
> html-ts-mode is not in emacs-29.

Oops, I confused it with css-ts-mode.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Wed, 10 May 2023 11:24:08 GMT) Full text and rfc822 format available.

This bug report was last modified 2 years and 98 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.

GNU bug report logs - #62752 html-ts-mode: treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3

GNU bug report logs - #62752
html-ts-mode: treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3