Re: master c916f471caa 2/3: Add tree-sitter performance note

Re: master c916f471caa 2/3: Add tree-sitter performance note

[Po Lu]

Yuan Fu <casouri@gmail.com> writes:

> branch: master
> commit c916f471caada4eb9a4b7f9f53925a71525c694a
> Author: Yuan Fu <casouri@gmail.com>
> Commit: Yuan Fu <casouri@gmail.com>
>
>     Add tree-sitter performance note
> ---
>  admin/notes/tree-sitter/performance | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/admin/notes/tree-sitter/performance b/admin/notes/tree-sitter/performance
> new file mode 100644
> index 00000000000..0e8351403af
> --- /dev/null
> +++ b/admin/notes/tree-sitter/performance
> @@ -0,0 +1,12 @@
> +TREE-SITTER PERFORMANCE NOTES -*- org -*-
> +
> +* Facts
> +
> +Incremental parsing of a few characters worth of edit usually takes
> +less than 0.1ms. If it takes longer than that, something is wrong.  There’s one time where I found tree-sitter-c takes ~30ms to incremental parse. Updating to the latest version of tree-sitter-c solves it, so I didn’t investigate further.

Shouldn't this paragraph be filled to 72 columns, as you have done in
the next one?

Re: master c916f471caa 2/3: Add tree-sitter performance note

[Yuan Fu]

> On Sep 13, 2023, at 6:43 PM, Po Lu <luangruo@yahoo.com> wrote:
> 
> Yuan Fu <casouri@gmail.com> writes:
> 
>> branch: master
>> commit c916f471caada4eb9a4b7f9f53925a71525c694a
>> Author: Yuan Fu <casouri@gmail.com>
>> Commit: Yuan Fu <casouri@gmail.com>
>> 
>>    Add tree-sitter performance note
>> ---
>> admin/notes/tree-sitter/performance | 12 ++++++++++++
>> 1 file changed, 12 insertions(+)
>> 
>> diff --git a/admin/notes/tree-sitter/performance b/admin/notes/tree-sitter/performance
>> new file mode 100644
>> index 00000000000..0e8351403af
>> --- /dev/null
>> +++ b/admin/notes/tree-sitter/performance
>> @@ -0,0 +1,12 @@
>> +TREE-SITTER PERFORMANCE NOTES -*- org -*-
>> +
>> +* Facts
>> +
>> +Incremental parsing of a few characters worth of edit usually takes
>> +less than 0.1ms. If it takes longer than that, something is wrong.  There’s one time where I found tree-sitter-c takes ~30ms to incremental parse. Updating to the latest version of tree-sitter-c solves it, so I didn’t investigate further.
> 
> Shouldn't this paragraph be filled to 72 columns, as you have done in
> the next one?

Indeed it should. Thanks for catching this. I use word-wrap so filled and unfilled paragraph look the same to me, apparently I forgot to fill this paragraph.

Yuan

Re: master c916f471caa 2/3: Add tree-sitter performance note

[Po Lu]

Yuan Fu <casouri@gmail.com> writes:

> Indeed it should. Thanks for catching this. I use word-wrap so filled
> and unfilled paragraph look the same to me, apparently I forgot to
> fill this paragraph.

OK, thanks.  I truncate long lines :-)