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

Previous Next

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.

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Vincenzo Pupillo <v.pupillo <at> gmail.com>
Subject: bug#62752: closed (Re: bug#62752: html-ts-mode: treesit--simple-indent-eval:
 Wrong number of arguments: #<subr point-min>, 3)
Date: Mon, 10 Apr 2023 21:26:02 +0000

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

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

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 62752 <at> debbugs.gnu.org.

-- 
62752: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62752
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]
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.


[Message part 3 (message/rfc822, inline)]
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 4 (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)]
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.