Error on `get-text-property`

Error on `get-text-property`

[guo.yong--- via Users list for the GNU Emacs text editor]

Hello.

I'm using `smartparens-mode` and it causes an erorr when I try to delete word when the kill ring is empty.

From the backtrack, the error occured from `kill-append`. The reason is that `kill-append` calls `get-text-property` in the function. But when the kill ring is empty, the function call is `(get-text-property 0 'yank-handler nil)` which cause the error.

I don't know Elisp well so I'm at lost. Is there any workaround for this problem or do I have to report this to Emacs devs (the functions are from `simple.el`)?

Regards.Yong.

P.S. This is my first time using mailing list. Sorry for the previous mail from me (https://lists.gnu.org/archive/html/help-gnu-emacs/2021-11/msg00415.html)

Re: Error on `get-text-property`

[Emanuel Berg via Users list for the GNU Emacs text editor]

guo.yong--- via Users list for the GNU Emacs text editor wrote:

> From the backtrack, the error occured from `kill-append`.
> The reason is that `kill-append` calls `get-text-property`
> in the function. But when the kill ring is empty, the
> function call is `(get-text-property 0 'yank-handler nil)`
> which cause the error.

What error, this one

  eval: Args out of range: 0,0 

?

-- 
underground experts united
https://dataswamp.org/~incal

Re: Error on `get-text-property`

[guo.yong--- via Users list for the GNU Emacs text editor]

Yes. It is that error.
I think I have a workaround now. Changing the call of get-text-property in kill-append to
(get-text-property (if (null cur)
                                               1
                                             0) 'yank-handler cur)

Seems to fix the error. I'll have to try it out for a while to see how it affects other functions as well.

Regrads.
Yong.
-- 
 Sent with Tutanota, the secure & ad-free mailbox. 



Nov 28, 2021, 22:19 by help-gnu-emacs@gnu.org:

> guo.yong--- via Users list for the GNU Emacs text editor wrote:
>
>> From the backtrack, the error occured from `kill-append`.
>> The reason is that `kill-append` calls `get-text-property`
>> in the function. But when the kill ring is empty, the
>> function call is `(get-text-property 0 'yank-handler nil)`
>> which cause the error.
>>
>
> What error, this one
>
>  eval: Args out of range: 0,0 
>
> ?
>
> -- 
> underground experts united
> https://dataswamp.org/~incal
>

Re: Error on `get-text-property`

[Michael Heerdegen]

guo.yong--- via Users list for the GNU Emacs text editor
<help-gnu-emacs@gnu.org> writes:

> Yes. It is that error.
> I think I have a workaround now. Changing the call of
> get-text-property in kill-append to
> (get-text-property (if (null cur)
>                                                1
>                                              0) 'yank-handler cur)
>
> Seems to fix the error.

This only masks the symptom.  `kill-append' must not be called when the
kill-ring is empty.  smartparens should take care of this.

Michael.

Re: Error on `get-text-property`

[guo.yong--- via Users list for the GNU Emacs text editor]

You are right. I found another error from get-text-property but I cannot reproduce this bug yet :/ It looks like I have to hack around smartparens for a bit. 

Yong. 
-- 
 Sent with Tutanota, the secure & ad-free mailbox. 



Nov 29, 2021, 17:32 by michael_heerdegen@web.de:

> guo.yong--- via Users list for the GNU Emacs text editor
> <help-gnu-emacs@gnu.org> writes:
>
>> Yes. It is that error.
>> I think I have a workaround now. Changing the call of
>> get-text-property in kill-append to
>> (get-text-property (if (null cur)
>>                                                1
>>                                              0) 'yank-handler cur)
>>
>> Seems to fix the error.
>>
>
> This only masks the symptom.  `kill-append' must not be called when the
> kill-ring is empty.  smartparens should take care of this.
>
> Michael.
>

Re: Error on `get-text-property`

[Emanuel Berg via Users list for the GNU Emacs text editor]

guo.yong--- via Users list for the GNU Emacs text editor wrote:

> From the backtrack, the error occured from `kill-append`.

BTW `kill-append' is another one of these functions that end
with something that can be t or nil, from the on-line help it
says

  (kill-append STRING BEFORE-P)

  [...] If BEFORE-P is non-nil, prepend STRING to the kill
  instead

One can instead make the last argument optional, it then
defaults ot nil, instead of having ugly code that looks like
this

  (kill-append "This doesn't look good" nil)

where people have to check what that second argument nil
really is ...

-- 
underground experts united
https://dataswamp.org/~incal