jpegenc: add a fixme for handling other YUV variants
authorStefan Kost <ensonic@users.sf.net>
Tue, 16 Feb 2010 08:10:45 +0000 (10:10 +0200)
committerStefan Kost <ensonic@users.sf.net>
Tue, 16 Feb 2010 08:15:22 +0000 (10:15 +0200)
ext/jpeg/gstjpegenc.c

index 73e674c9ea97cf504060b835ae1a36e4de673682..01f1665a2ceedbe822bfb64a36eaae3aee7fbeb2 100644 (file)
@@ -514,6 +514,11 @@ gst_jpegenc_chain (GstPad * pad, GstBuffer * buf)
   end[1] = base[1] + (height / 2) * I420_U_ROWSTRIDE (width);
   end[2] = base[2] + (height / 2) * I420_V_ROWSTRIDE (width);
 
+  /* FIXME: shouldn't we also set
+   * - jpegenc->cinfo.max_{v,h}_samp_factor
+   * - jpegenc->cinfo.comp_info[0,1,2].{v,h}_samp_factor
+   * accordingly?
+   */
   jpegenc->jdest.next_output_byte = GST_BUFFER_DATA (jpegenc->output_buffer);
   jpegenc->jdest.free_in_buffer = GST_BUFFER_SIZE (jpegenc->output_buffer);