[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Allow inserting non-BMP characters
From: |
Eli Zaretskii |
Subject: |
Re: [PATCH] Allow inserting non-BMP characters |
Date: |
Thu, 28 Dec 2017 18:29:00 +0200 |
> From: Philipp Stephani <address@hidden>
> Date: Thu, 28 Dec 2017 12:31:39 +0000
> Cc: address@hidden, Eli Zaretskii <address@hidden>, address@hidden
>
> Alan Third <address@hidden> schrieb am Do., 28. Dez. 2017 um 12:38 Uhr:
>
> On Wed, Dec 27, 2017 at 05:41:23AM +0200, Eli Zaretskii wrote:
> > What about the possibility that SAFE_NALLOCA could signal an error and
> > longjmp to top level? Does this code always run in the main thread,
> > and if so, can it allow such longjmp's?
>
> I think it does always run in the main thread, however since it runs
> within the NSApplication run loop I’ve no idea what would happen if we
> did a longjmp.
>
> We should probably avoid longjmps here. This message is invoked by the window
> manager, which most likely
> can't deal with longjmps.
Then maybe process the input in chunks using a fixed-size buffer?
- Re: [PATCH] Allow inserting non-BMP characters, (continued)
- Re: [PATCH] Allow inserting non-BMP characters, Alan Third, 2017/12/25
- Re: [PATCH] Allow inserting non-BMP characters, Eli Zaretskii, 2017/12/25
- Re: [PATCH] Allow inserting non-BMP characters, Philipp Stephani, 2017/12/26
- Re: [PATCH] Allow inserting non-BMP characters, Eli Zaretskii, 2017/12/26
- Re: [PATCH] Allow inserting non-BMP characters, Philipp Stephani, 2017/12/26
- Re: [PATCH] Allow inserting non-BMP characters, Eli Zaretskii, 2017/12/26
- Re: [PATCH] Allow inserting non-BMP characters, Alan Third, 2017/12/26
- Re: [PATCH] Allow inserting non-BMP characters, Eli Zaretskii, 2017/12/26
- Re: [PATCH] Allow inserting non-BMP characters, Alan Third, 2017/12/28
- Re: [PATCH] Allow inserting non-BMP characters, Philipp Stephani, 2017/12/28
- Re: [PATCH] Allow inserting non-BMP characters,
Eli Zaretskii <=
- Re: [PATCH] Allow inserting non-BMP characters, Philipp Stephani, 2017/12/29
- Re: [PATCH] Allow inserting non-BMP characters, Eli Zaretskii, 2017/12/29