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

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

bug#62752: closed (html-ts-mode: treesit--simple-indent-eval: Wrong numb


From: GNU bug Tracking System
Subject: bug#62752: closed (html-ts-mode: treesit--simple-indent-eval: Wrong number of arguments: #<subr point-min>, 3)
Date: Mon, 10 Apr 2023 21:26:01 +0000

Your message dated Tue, 11 Apr 2023 00:24:54 +0300
with message-id <0f1c27e1-4ff5-8909-ffd7-f185065e082e@gutov.dev>
and subject line Re: bug#62752: html-ts-mode: treesit--simple-indent-eval: 
Wrong number of arguments: #<subr point-min>, 3
has caused the debbugs.gnu.org bug report #62752,
regarding html-ts-mode: treesit--simple-indent-eval: Wrong number of arguments: 
#<subr point-min>, 3
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
62752: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62752
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- 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
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.

Attachment: 0001-Fixed-the-html-ts-mode-indentation-rule.patch
Description: Text Data


--- End Message ---
--- Begin Message --- 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 User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0
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.


--- End Message ---

reply via email to

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