Re: [libav-devel] [PATCH 3/3] mpegvideo: remove VLAs

2012-06-19 Thread Janne Grunau
On 2012-06-14 16:36:44 +0200, Christophe Gisquet wrote: Hi, 2012/6/14 Mans Rullgard m...@mansr.com:  static void guess_mv(MpegEncContext *s)  { -    uint8_t fixed[s-mb_stride * s-mb_height]; +    uint8_t *fixed = s-fixed; What about setting s-fixed to NULL when initializing, and

Re: [libav-devel] [PATCH 3/3] mpegvideo: remove VLAs

2012-06-19 Thread Måns Rullgård
Janne Grunau janne-li...@jannau.net writes: On 2012-06-14 16:36:44 +0200, Christophe Gisquet wrote: Hi, 2012/6/14 Mans Rullgard m...@mansr.com:  static void guess_mv(MpegEncContext *s)  { -    uint8_t fixed[s-mb_stride * s-mb_height]; +    uint8_t *fixed = s-fixed; What about

Re: [libav-devel] [PATCH 3/3] mpegvideo: remove VLAs

2012-06-14 Thread Christophe Gisquet
Hi, 2012/6/14 Mans Rullgard m...@mansr.com:  static void guess_mv(MpegEncContext *s)  { -    uint8_t fixed[s-mb_stride * s-mb_height]; +    uint8_t *fixed = s-fixed; What about setting s-fixed to NULL when initializing, and only setting s-fixed to something here, like (just an example):