From: Yuan Fu <casouri@gmail.com>
To: Po Lu <luangruo@yahoo.com>
Cc: Robert Pluim <rpluim@gmail.com>,
"Po Lu via Emacs development discussions." <emacs-devel@gnu.org>
Subject: Re: Stylistic changes to tree-sitter code
Date: Thu, 27 Oct 2022 21:14:48 -0700 [thread overview]
Message-ID: <8BD3AD59-DE34-4434-9912-BA33515BFC6B@gmail.com> (raw)
In-Reply-To: <87y1t06apv.fsf@yahoo.com>
> On Oct 27, 2022, at 5:41 PM, Po Lu via Emacs development discussions. <emacs-devel@gnu.org> wrote:
>
> Robert Pluim <rpluim@gmail.com> writes:
>
>>>>>>> On Thu, 27 Oct 2022 21:33:10 +0800, Po Lu via "Emacs development discussions." <emacs-devel@gnu.org> said:
>>
>>
>> Po Lu> Unfortunately, I don't have tree-sitter installed, so would someone
>> Po Lu> please do a smoke test on the changes before I install them?
>>
>> I have it installed, but I donʼt use it yet, so all I can offer is a
>> compile test:
>>
>> CC treesit.o
>> treesit.c: In function ‘treesit_find_override_name’:
>> treesit.c:467:41: error: ‘list’ undeclared (first use in this function); did you mean ‘listn’?
>> 467 | *name = Fnth (make_fixnum (1), XCAR (list));
>> | ^~~~
>> | listn
>> treesit.c:467:41: note: each undeclared identifier is reported only once for each function it appears in
>> treesit.c:476:57: error: expected ‘;’ before ‘return’
>> 476 | CHECK_LIST_END (tem, Vtreesit_load_name_override_list)
>> | ^
>> | ;
>> 477 |
>> 478 | return false;
>> | ~~~~~~
>> treesit.c: In function ‘treesit_load_language’:
>> treesit.c:555:15: error: redeclaration of ‘tail’ with no linkage
>> 555 | Lisp_Object tail;
>> | ^~~~
>> treesit.c:539:15: note: previous declaration of ‘tail’ was here
>> 539 | Lisp_Object tail;
>> | ^~~~
>> treesit.c: In function ‘treesit_find_override_name’:
>> treesit.c:479:1: warning: control reaches end of non-void function [-Wreturn-type]
>> 479 | }
>> | ^
>> make[2]: *** [Makefile:424: treesit.o] Error 1
>>
>> Robert
>
> Thanks. What about this patch?
Thanks for your work! I don’t have opinions on stylistic changes :-)
Yuan
next prev parent reply other threads:[~2022-10-28 4:14 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <87h6zp75nt.fsf.ref@yahoo.com>
2022-10-27 13:33 ` Stylistic changes to tree-sitter code Po Lu via Emacs development discussions.
2022-10-27 13:56 ` Robert Pluim
2022-10-28 0:41 ` Po Lu via Emacs development discussions.
2022-10-28 4:14 ` Yuan Fu [this message]
2022-10-28 7:03 ` Robert Pluim
2022-10-28 7:08 ` Po Lu
2022-10-29 2:45 ` Po Lu via Emacs development discussions.
2022-10-29 4:19 ` Yuan Fu
2022-10-29 5:44 ` Po Lu
2022-10-29 7:01 ` Eli Zaretskii
2022-10-29 7:25 ` Po Lu
2022-10-29 8:16 ` Eli Zaretskii
2022-10-29 8:34 ` Po Lu
2022-10-29 6:54 ` Eli Zaretskii
2022-10-29 7:29 ` Stefan Kangas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8BD3AD59-DE34-4434-9912-BA33515BFC6B@gmail.com \
--to=casouri@gmail.com \
--cc=emacs-devel@gnu.org \
--cc=luangruo@yahoo.com \
--cc=rpluim@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.