Merge pull request #4512 from el-j/perf/canvas-upload-buffer-4504

perf(ui-egui): eliminate zero-filling allocation on canvas texture upload in color_image (#4504)
This commit is contained in:
BFlat
2026-10-29 10:43:58 +07:00
committed by GitHub
+6 -5
View File
@@ -5,12 +5,13 @@ use super::{Affine, CacheKey, VectorcraftApp, Xf};
/// A raster whose bytes don't match its size gives an empty image of that size.
pub(crate) fn color_image(img: &vectorcraft_render::Rendered) -> egui::ColorImage {
let size = [img.width as usize, img.height as usize];
let mut pixels = vec![egui::Color32::TRANSPARENT; size[0] * size[1]];
let bytes: &mut [u8] = bytemuck::cast_slice_mut(&mut pixels);
if bytes.len() == img.pixels.len() {
bytes.copy_from_slice(&img.pixels);
let expected = size[0].saturating_mul(size[1]).saturating_mul(4);
if expected == img.pixels.len() {
let pixels: Vec<egui::Color32> = bytemuck::cast_slice(&img.pixels).to_vec();
egui::ColorImage::new(size, pixels)
} else {
egui::ColorImage::new(size, vec![egui::Color32::TRANSPARENT; size[0].saturating_mul(size[1])])
}
egui::ColorImage::new(size, pixels)
}
/// The pixels of the `w`×`h` frame through `view` to render again for `doc`, when only the document