[FFmpeg-devel] Subject: [PATCH] libavcodec/qsv.c: Linux-only code part has been moved to separate function in order to avoid the "ISO C90 forbids mixed declarations and code" compiler warning.

Ivan Uskov ivan.uskov at nablet.com
Mon Jul 6 20:37:52 CEST 2015


Hello All,

Unfortunately, previous patch for  libavcodec/qsv.c at the commit
db89f45535aa3e99bceb5f6bf957c90e7ca39841 does initiate the compiler warning
"ISO C90 forbids mixed declarations and code" under linux platform,
because there is declaration block under preprocessor condition. Looks
like it can not be re-arranged by graceful way into one function.
I believe the best way to fix it is moving linux-related code into own
separate function. Suggested patch does this.
Please review. Thank.
  

-- 
Best regards,
 Ivan                          mailto:ivan.uskov at nablet.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-libavcodec-qsv.c-Linux-only-code-part-has-been-moved.patch
Type: application/octet-stream
Size: 4118 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150706/984bf5c8/attachment.obj>


More information about the ffmpeg-devel mailing list