Unexpected panic due to unwrap None in JpegEncoder::encode

[thdusdl1219]

This happens in

L458, 463, 468, 473

image/src/codecs/jpeg/encoder.rs

Lines 443 to 475 in f1e34f3

	pub fn encode(
	&mut self,
	image: &[u8],
	width: u32,
	height: u32,
	color_type: ColorType,
	) -> ImageResult<()> {
	assert_eq!(
	(width as u64 * height as u64).saturating_mul(color_type.bytes_per_pixel() as u64),
	image.len() as u64
	);
	match color_type {
	ColorType::L8 => {
	let image: ImageBuffer<Luma<_>, _> =
	ImageBuffer::from_raw(width, height, image).unwrap();
	self.encode_image(&image)
	}
	ColorType::La8 => {
	let image: ImageBuffer<LumaA<_>, _> =
	ImageBuffer::from_raw(width, height, image).unwrap();
	self.encode_image(&image)
	}
	ColorType::Rgb8 => {
	let image: ImageBuffer<Rgb<_>, _> =
	ImageBuffer::from_raw(width, height, image).unwrap();
	self.encode_image(&image)
	}
	ColorType::Rgba8 => {
	let image: ImageBuffer<Rgba<_>, _> =
	ImageBuffer::from_raw(width, height, image).unwrap();
	self.encode_image(&image)
	}

Expected

return Result instead of panic?

Actual behaviour

thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', .cargo/registry/src/index.crates.io-6f17d22bba15001f/image-0.24.7/./src/codecs/jpeg/encoder.rs:449

Reproduction steps

    let mut buffer = Vec::new();
    image::ImageEncoder::write_image(image::codecs::jpeg::JpegEncoder::new(&mut
    buffer), &[0], 1, 1024, image::ColorType::L8 /* , La8, Rgb8, or Rgba8 */ );

[fintelia]

Could you double check that this still happens on master? I think the assertion on line 450 (added in #2008) should now catch this case

[thdusdl1219]

Right, it does not happen on master after the fix. Please feel free to close this issue, Thanks!