From: Ethan A M. <me...@uw...> - 2023-06-12 05:56:51
|
On Sunday, 11 June 2023 20:44:31 PDT Dima Kogan wrote: > Sorry for yet another email. I was dogfooding the patch, and found a > minor bug. Sometimes term->image is not available, and the new colorbox > implementation crashes in that case. I added a check to the top of the > function, and it's good now: > > static void > draw_inside_colorbox_bitmap_smooth() > { > if(term->image == NULL) > return; > ..... > > I hit this with the "dumb" terminal. I think not rendering the colorbox > at all in that case is fine. Are there other terminals where term->image > is unavailable, but drawing discrete slices would still have produced > good output? It will take a while for me to work my way through your proposed changes. But this much I can answer quickly - Yes there are terminals that support smooth palette color but not term->image(). At a minimum they include block cgm emf fig sixeltek Ethan |