[Github-comments] Re: [geany/geany-plugins] Spell Check plugin crashes with color-variant-selector emojis (Issue #1272)

[xiota via Github-comments]

@nyamatongwe 

When I tested, I put the conditional return statement before the 
`PLATFORM_ASSERT` line:
```
void SurfaceImpl::DrawRGBAImage(PRectangle rc, int width, int height, const 
unsigned char *pixelsImage) {
        if (width == 0) return;

        PLATFORM_ASSERT(context);
```

I noticed in your commit that you put it after.  What if the `PLATFORM_ASSERT` 
is the origin of the abort?
```
 void SurfaceImpl::DrawRGBAImage(PRectangle rc, int width, int height, const 
unsigned char *pixelsImage) {
        PLATFORM_ASSERT(context);
+       if (width == 0)
+               return;
        if (rc.Width() > width)
```

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/issues/1272#issuecomment-1716834236
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany-plugins/issues/1272/1716834...@github.com>