]> git.sesse.net Git - ffmpeg/blobdiff - libswscale/swscale.c
Const correctness for src pointer. Remove all constness related warnings in
[ffmpeg] / libswscale / swscale.c
index 4a69ab8ea15b83161dd069b75b2c92d09bf0ddfb..f525420f910ae645c3f1eaf483ad6aef344627c1 100644 (file)
@@ -74,14 +74,28 @@ untested special converters
 #include "swscale.h"
 #include "swscale_internal.h"
 #include "rgb2rgb.h"
+#include "libavutil/intreadwrite.h"
 #include "libavutil/x86_cpu.h"
+#include "libavutil/avutil.h"
 #include "libavutil/bswap.h"
+#include "libavutil/pixdesc.h"
 
 unsigned swscale_version(void)
 {
     return LIBSWSCALE_VERSION_INT;
 }
 
+const char *swscale_configuration(void)
+{
+    return FFMPEG_CONFIGURATION;
+}
+
+const char *swscale_license(void)
+{
+#define LICENSE_PREFIX "libswscale license: "
+    return LICENSE_PREFIX FFMPEG_LICENSE + sizeof(LICENSE_PREFIX) - 1;
+}
+
 #undef MOVNTQ
 #undef PAVGB
 
@@ -121,6 +135,8 @@ unsigned swscale_version(void)
         || (x)==PIX_FMT_GRAY8       \
         || (x)==PIX_FMT_YUV410P     \
         || (x)==PIX_FMT_YUV440P     \
+        || (x)==PIX_FMT_NV12        \
+        || (x)==PIX_FMT_NV21        \
         || (x)==PIX_FMT_GRAY16BE    \
         || (x)==PIX_FMT_GRAY16LE    \
         || (x)==PIX_FMT_YUV444P     \
@@ -134,13 +150,19 @@ unsigned swscale_version(void)
         || (x)==PIX_FMT_YUV440P     \
         || (x)==PIX_FMT_MONOWHITE   \
         || (x)==PIX_FMT_MONOBLACK   \
-        || (x)==PIX_FMT_YUV420PLE   \
-        || (x)==PIX_FMT_YUV422PLE   \
-        || (x)==PIX_FMT_YUV444PLE   \
-        || (x)==PIX_FMT_YUV420PBE   \
-        || (x)==PIX_FMT_YUV422PBE   \
-        || (x)==PIX_FMT_YUV444PBE   \
+        || (x)==PIX_FMT_YUV420P16LE   \
+        || (x)==PIX_FMT_YUV422P16LE   \
+        || (x)==PIX_FMT_YUV444P16LE   \
+        || (x)==PIX_FMT_YUV420P16BE   \
+        || (x)==PIX_FMT_YUV422P16BE   \
+        || (x)==PIX_FMT_YUV444P16BE   \
     )
+
+int sws_isSupportedInput(enum PixelFormat pix_fmt)
+{
+    return isSupportedIn(pix_fmt);
+}
+
 #define isSupportedOut(x)   (       \
            (x)==PIX_FMT_YUV420P     \
         || (x)==PIX_FMT_YUVA420P    \
@@ -158,13 +180,19 @@ unsigned swscale_version(void)
         || (x)==PIX_FMT_GRAY8       \
         || (x)==PIX_FMT_YUV410P     \
         || (x)==PIX_FMT_YUV440P     \
-        || (x)==PIX_FMT_YUV420PLE   \
-        || (x)==PIX_FMT_YUV422PLE   \
-        || (x)==PIX_FMT_YUV444PLE   \
-        || (x)==PIX_FMT_YUV420PBE   \
-        || (x)==PIX_FMT_YUV422PBE   \
-        || (x)==PIX_FMT_YUV444PBE   \
+        || (x)==PIX_FMT_YUV420P16LE   \
+        || (x)==PIX_FMT_YUV422P16LE   \
+        || (x)==PIX_FMT_YUV444P16LE   \
+        || (x)==PIX_FMT_YUV420P16BE   \
+        || (x)==PIX_FMT_YUV422P16BE   \
+        || (x)==PIX_FMT_YUV444P16BE   \
     )
+
+int sws_isSupportedOutput(enum PixelFormat pix_fmt)
+{
+    return isSupportedOut(pix_fmt);
+}
+
 #define isPacked(x)         (       \
            (x)==PIX_FMT_PAL8        \
         || (x)==PIX_FMT_YUYV422     \
@@ -172,13 +200,7 @@ unsigned swscale_version(void)
         || isRGB(x)                 \
         || isBGR(x)                 \
     )
-#define usePal(x)           (       \
-           (x)==PIX_FMT_PAL8        \
-        || (x)==PIX_FMT_BGR4_BYTE   \
-        || (x)==PIX_FMT_RGB4_BYTE   \
-        || (x)==PIX_FMT_BGR8        \
-        || (x)==PIX_FMT_RGB8        \
-    )
+#define usePal(x) (av_pix_fmt_descriptors[x].flags & PIX_FMT_PAL)
 
 #define RGB2YUV_SHIFT 15
 #define BY ( (int)(0.114*219/255*(1<<RGB2YUV_SHIFT)+0.5))
@@ -276,9 +298,6 @@ DECLARE_ASM_CONST(8, uint64_t, ff_bgr24toUVOffset)= 0x0040400000404000ULL;
 
 #endif /* ARCH_X86 && CONFIG_GPL */
 
-// clipping helper table for C implementations:
-static unsigned char clip_table[768];
-
 static SwsVector *sws_getConvVec(SwsVector *a, SwsVector *b);
 
 DECLARE_ALIGNED(8, static const uint8_t, dither_2x2_4[2][8])={
@@ -364,113 +383,89 @@ DECLARE_ALIGNED(8, const uint8_t, dither_8x8_220[8][8])={
 
 const char *sws_format_name(enum PixelFormat format)
 {
-    switch (format) {
-        case PIX_FMT_YUV420P:
-            return "yuv420p";
-        case PIX_FMT_YUVA420P:
-            return "yuva420p";
-        case PIX_FMT_YUYV422:
-            return "yuyv422";
-        case PIX_FMT_RGB24:
-            return "rgb24";
-        case PIX_FMT_BGR24:
-            return "bgr24";
-        case PIX_FMT_YUV422P:
-            return "yuv422p";
-        case PIX_FMT_YUV444P:
-            return "yuv444p";
-        case PIX_FMT_RGB32:
-            return "rgb32";
-        case PIX_FMT_YUV410P:
-            return "yuv410p";
-        case PIX_FMT_YUV411P:
-            return "yuv411p";
-        case PIX_FMT_RGB565:
-            return "rgb565";
-        case PIX_FMT_RGB555:
-            return "rgb555";
-        case PIX_FMT_GRAY16BE:
-            return "gray16be";
-        case PIX_FMT_GRAY16LE:
-            return "gray16le";
-        case PIX_FMT_GRAY8:
-            return "gray8";
-        case PIX_FMT_MONOWHITE:
-            return "mono white";
-        case PIX_FMT_MONOBLACK:
-            return "mono black";
-        case PIX_FMT_PAL8:
-            return "Palette";
-        case PIX_FMT_YUVJ420P:
-            return "yuvj420p";
-        case PIX_FMT_YUVJ422P:
-            return "yuvj422p";
-        case PIX_FMT_YUVJ444P:
-            return "yuvj444p";
-        case PIX_FMT_XVMC_MPEG2_MC:
-            return "xvmc_mpeg2_mc";
-        case PIX_FMT_XVMC_MPEG2_IDCT:
-            return "xvmc_mpeg2_idct";
-        case PIX_FMT_UYVY422:
-            return "uyvy422";
-        case PIX_FMT_UYYVYY411:
-            return "uyyvyy411";
-        case PIX_FMT_RGB32_1:
-            return "rgb32x";
-        case PIX_FMT_BGR32_1:
-            return "bgr32x";
-        case PIX_FMT_BGR32:
-            return "bgr32";
-        case PIX_FMT_BGR565:
-            return "bgr565";
-        case PIX_FMT_BGR555:
-            return "bgr555";
-        case PIX_FMT_BGR8:
-            return "bgr8";
-        case PIX_FMT_BGR4:
-            return "bgr4";
-        case PIX_FMT_BGR4_BYTE:
-            return "bgr4 byte";
-        case PIX_FMT_RGB8:
-            return "rgb8";
-        case PIX_FMT_RGB4:
-            return "rgb4";
-        case PIX_FMT_RGB4_BYTE:
-            return "rgb4 byte";
-        case PIX_FMT_RGB48BE:
-            return "rgb48be";
-        case PIX_FMT_RGB48LE:
-            return "rgb48le";
-        case PIX_FMT_NV12:
-            return "nv12";
-        case PIX_FMT_NV21:
-            return "nv21";
-        case PIX_FMT_YUV440P:
-            return "yuv440p";
-        case PIX_FMT_VDPAU_H264:
-            return "vdpau_h264";
-        case PIX_FMT_VDPAU_MPEG1:
-            return "vdpau_mpeg1";
-        case PIX_FMT_VDPAU_MPEG2:
-            return "vdpau_mpeg2";
-        case PIX_FMT_VDPAU_WMV3:
-            return "vdpau_wmv3";
-        case PIX_FMT_VDPAU_VC1:
-            return "vdpau_vc1";
-        case PIX_FMT_YUV420PLE:
-            return "yuv420ple";
-        case PIX_FMT_YUV422PLE:
-            return "yuv422ple";
-        case PIX_FMT_YUV444PLE:
-            return "yuv444ple";
-        case PIX_FMT_YUV420PBE:
-            return "yuv420pbe";
-        case PIX_FMT_YUV422PBE:
-            return "yuv422pbe";
-        case PIX_FMT_YUV444PBE:
-            return "yuv444pbe";
-        default:
-            return "Unknown format";
+    if ((unsigned)format < PIX_FMT_NB && av_pix_fmt_descriptors[format].name)
+        return av_pix_fmt_descriptors[format].name;
+    else
+        return "Unknown format";
+}
+
+static av_always_inline void yuv2yuvX16inC_template(const int16_t *lumFilter, const int16_t **lumSrc, int lumFilterSize,
+                                                    const int16_t *chrFilter, const int16_t **chrSrc, int chrFilterSize,
+                                                    const int16_t **alpSrc, uint16_t *dest, uint16_t *uDest, uint16_t *vDest, uint16_t *aDest,
+                                                    int dstW, int chrDstW, int big_endian)
+{
+    //FIXME Optimize (just quickly written not optimized..)
+    int i;
+
+    for (i = 0; i < dstW; i++) {
+        int val = 1 << 10;
+        int j;
+
+        for (j = 0; j < lumFilterSize; j++)
+            val += lumSrc[j][i] * lumFilter[j];
+
+        if (big_endian) {
+            AV_WB16(&dest[i], av_clip_uint16(val >> 11));
+        } else {
+            AV_WL16(&dest[i], av_clip_uint16(val >> 11));
+        }
+    }
+
+    if (uDest) {
+        for (i = 0; i < chrDstW; i++) {
+            int u = 1 << 10;
+            int v = 1 << 10;
+            int j;
+
+            for (j = 0; j < chrFilterSize; j++) {
+                u += chrSrc[j][i       ] * chrFilter[j];
+                v += chrSrc[j][i + VOFW] * chrFilter[j];
+            }
+
+            if (big_endian) {
+                AV_WB16(&uDest[i], av_clip_uint16(u >> 11));
+                AV_WB16(&vDest[i], av_clip_uint16(v >> 11));
+            } else {
+                AV_WL16(&uDest[i], av_clip_uint16(u >> 11));
+                AV_WL16(&vDest[i], av_clip_uint16(v >> 11));
+            }
+        }
+    }
+
+    if (CONFIG_SWSCALE_ALPHA && aDest) {
+        for (i = 0; i < dstW; i++) {
+            int val = 1 << 10;
+            int j;
+
+            for (j = 0; j < lumFilterSize; j++)
+                val += alpSrc[j][i] * lumFilter[j];
+
+            if (big_endian) {
+                AV_WB16(&aDest[i], av_clip_uint16(val >> 11));
+            } else {
+                AV_WL16(&aDest[i], av_clip_uint16(val >> 11));
+            }
+        }
+    }
+}
+
+static inline void yuv2yuvX16inC(const int16_t *lumFilter, const int16_t **lumSrc, int lumFilterSize,
+                                 const int16_t *chrFilter, const int16_t **chrSrc, int chrFilterSize,
+                                 const int16_t **alpSrc, uint16_t *dest, uint16_t *uDest, uint16_t *vDest, uint16_t *aDest, int dstW, int chrDstW,
+                                 enum PixelFormat dstFormat)
+{
+    if (isBE(dstFormat)) {
+        yuv2yuvX16inC_template(lumFilter, lumSrc, lumFilterSize,
+                               chrFilter, chrSrc, chrFilterSize,
+                               alpSrc,
+                               dest, uDest, vDest, aDest,
+                               dstW, chrDstW, 1);
+    } else {
+        yuv2yuvX16inC_template(lumFilter, lumSrc, lumFilterSize,
+                               chrFilter, chrSrc, chrFilterSize,
+                               alpSrc,
+                               dest, uDest, vDest, aDest,
+                               dstW, chrDstW, 0);
     }
 }
 
@@ -480,8 +475,7 @@ static inline void yuv2yuvXinC(const int16_t *lumFilter, const int16_t **lumSrc,
 {
     //FIXME Optimize (just quickly written not optimized..)
     int i;
-    for (i=0; i<dstW; i++)
-    {
+    for (i=0; i<dstW; i++) {
         int val=1<<18;
         int j;
         for (j=0; j<lumFilterSize; j++)
@@ -491,13 +485,11 @@ static inline void yuv2yuvXinC(const int16_t *lumFilter, const int16_t **lumSrc,
     }
 
     if (uDest)
-        for (i=0; i<chrDstW; i++)
-        {
+        for (i=0; i<chrDstW; i++) {
             int u=1<<18;
             int v=1<<18;
             int j;
-            for (j=0; j<chrFilterSize; j++)
-            {
+            for (j=0; j<chrFilterSize; j++) {
                 u += chrSrc[j][i] * chrFilter[j];
                 v += chrSrc[j][i + VOFW] * chrFilter[j];
             }
@@ -507,7 +499,7 @@ static inline void yuv2yuvXinC(const int16_t *lumFilter, const int16_t **lumSrc,
         }
 
     if (CONFIG_SWSCALE_ALPHA && aDest)
-        for (i=0; i<dstW; i++){
+        for (i=0; i<dstW; i++) {
             int val=1<<18;
             int j;
             for (j=0; j<lumFilterSize; j++)
@@ -524,8 +516,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
 {
     //FIXME Optimize (just quickly written not optimized..)
     int i;
-    for (i=0; i<dstW; i++)
-    {
+    for (i=0; i<dstW; i++) {
         int val=1<<18;
         int j;
         for (j=0; j<lumFilterSize; j++)
@@ -538,13 +529,11 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         return;
 
     if (dstFormat == PIX_FMT_NV12)
-        for (i=0; i<chrDstW; i++)
-        {
+        for (i=0; i<chrDstW; i++) {
             int u=1<<18;
             int v=1<<18;
             int j;
-            for (j=0; j<chrFilterSize; j++)
-            {
+            for (j=0; j<chrFilterSize; j++) {
                 u += chrSrc[j][i] * chrFilter[j];
                 v += chrSrc[j][i + VOFW] * chrFilter[j];
             }
@@ -553,13 +542,11 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
             uDest[2*i+1]= av_clip_uint8(v>>19);
         }
     else
-        for (i=0; i<chrDstW; i++)
-        {
+        for (i=0; i<chrDstW; i++) {
             int u=1<<18;
             int v=1<<18;
             int j;
-            for (j=0; j<chrFilterSize; j++)
-            {
+            for (j=0; j<chrFilterSize; j++) {
                 u += chrSrc[j][i] * chrFilter[j];
                 v += chrSrc[j][i + VOFW] * chrFilter[j];
             }
@@ -570,7 +557,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
 }
 
 #define YSCALE_YUV_2_PACKEDX_NOCLIP_C(type,alpha) \
-    for (i=0; i<(dstW>>1); i++){\
+    for (i=0; i<(dstW>>1); i++) {\
         int j;\
         int Y1 = 1<<18;\
         int Y2 = 1<<18;\
@@ -580,13 +567,11 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         type av_unused *r, *b, *g;\
         const int i2= 2*i;\
         \
-        for (j=0; j<lumFilterSize; j++)\
-        {\
+        for (j=0; j<lumFilterSize; j++) {\
             Y1 += lumSrc[j][i2] * lumFilter[j];\
             Y2 += lumSrc[j][i2+1] * lumFilter[j];\
         }\
-        for (j=0; j<chrFilterSize; j++)\
-        {\
+        for (j=0; j<chrFilterSize; j++) {\
             U += chrSrc[j][i] * chrFilter[j];\
             V += chrSrc[j][i+VOFW] * chrFilter[j];\
         }\
@@ -594,10 +579,10 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         Y2>>=19;\
         U >>=19;\
         V >>=19;\
-        if (alpha){\
+        if (alpha) {\
             A1 = 1<<18;\
             A2 = 1<<18;\
-            for (j=0; j<lumFilterSize; j++){\
+            for (j=0; j<lumFilterSize; j++) {\
                 A1 += alpSrc[j][i2  ] * lumFilter[j];\
                 A2 += alpSrc[j][i2+1] * lumFilter[j];\
             }\
@@ -607,8 +592,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
 
 #define YSCALE_YUV_2_PACKEDX_C(type,alpha) \
         YSCALE_YUV_2_PACKEDX_NOCLIP_C(type,alpha)\
-        if ((Y1|Y2|U|V)&256)\
-        {\
+        if ((Y1|Y2|U|V)&256) {\
             if (Y1>255)   Y1=255; \
             else if (Y1<0)Y1=0;   \
             if (Y2>255)   Y2=255; \
@@ -618,13 +602,13 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
             if (V>255)    V=255;  \
             else if (V<0) V=0;    \
         }\
-        if (alpha && ((A1|A2)&256)){\
+        if (alpha && ((A1|A2)&256)) {\
             A1=av_clip_uint8(A1);\
             A2=av_clip_uint8(A2);\
         }
 
 #define YSCALE_YUV_2_PACKEDX_FULL_C(rnd,alpha) \
-    for (i=0; i<dstW; i++){\
+    for (i=0; i<dstW; i++) {\
         int j;\
         int Y = 0;\
         int U = -128<<19;\
@@ -632,17 +616,17 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         int av_unused A;\
         int R,G,B;\
         \
-        for (j=0; j<lumFilterSize; j++){\
+        for (j=0; j<lumFilterSize; j++) {\
             Y += lumSrc[j][i     ] * lumFilter[j];\
         }\
-        for (j=0; j<chrFilterSize; j++){\
+        for (j=0; j<chrFilterSize; j++) {\
             U += chrSrc[j][i     ] * chrFilter[j];\
             V += chrSrc[j][i+VOFW] * chrFilter[j];\
         }\
         Y >>=10;\
         U >>=10;\
         V >>=10;\
-        if (alpha){\
+        if (alpha) {\
             A = rnd;\
             for (j=0; j<lumFilterSize; j++)\
                 A += alpSrc[j][i     ] * lumFilter[j];\
@@ -659,7 +643,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         R= Y + V*c->yuv2rgb_v2r_coeff;\
         G= Y + V*c->yuv2rgb_v2g_coeff + U*c->yuv2rgb_u2g_coeff;\
         B= Y +                          U*c->yuv2rgb_u2b_coeff;\
-        if ((R|G|B)&(0xC0000000)){\
+        if ((R|G|B)&(0xC0000000)) {\
             if (R>=(256<<22))   R=(256<<22)-1; \
             else if (R<0)R=0;   \
             if (G>=(256<<22))   G=(256<<22)-1; \
@@ -670,7 +654,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
 
 
 #define YSCALE_YUV_2_GRAY16_C \
-    for (i=0; i<(dstW>>1); i++){\
+    for (i=0; i<(dstW>>1); i++) {\
         int j;\
         int Y1 = 1<<18;\
         int Y2 = 1<<18;\
@@ -679,15 +663,13 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         \
         const int i2= 2*i;\
         \
-        for (j=0; j<lumFilterSize; j++)\
-        {\
+        for (j=0; j<lumFilterSize; j++) {\
             Y1 += lumSrc[j][i2] * lumFilter[j];\
             Y2 += lumSrc[j][i2+1] * lumFilter[j];\
         }\
         Y1>>=11;\
         Y2>>=11;\
-        if ((Y1|Y2|U|V)&65536)\
-        {\
+        if ((Y1|Y2|U|V)&65536) {\
             if (Y1>65535)   Y1=65535; \
             else if (Y1<0)Y1=0;   \
             if (Y2>65535)   Y2=65535; \
@@ -701,7 +683,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
     b = (type *)c->table_bU[U];   \
 
 #define YSCALE_YUV_2_PACKED2_C(type,alpha)   \
-    for (i=0; i<(dstW>>1); i++){ \
+    for (i=0; i<(dstW>>1); i++) { \
         const int i2= 2*i;       \
         int Y1= (buf0[i2  ]*yalpha1+buf1[i2  ]*yalpha)>>19;           \
         int Y2= (buf0[i2+1]*yalpha1+buf1[i2+1]*yalpha)>>19;           \
@@ -709,13 +691,13 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         int V= (uvbuf0[i+VOFW]*uvalpha1+uvbuf1[i+VOFW]*uvalpha)>>19;  \
         type av_unused *r, *b, *g;                                    \
         int av_unused A1, A2;                                         \
-        if (alpha){\
+        if (alpha) {\
             A1= (abuf0[i2  ]*yalpha1+abuf1[i2  ]*yalpha)>>19;         \
             A2= (abuf0[i2+1]*yalpha1+abuf1[i2+1]*yalpha)>>19;         \
         }\
 
 #define YSCALE_YUV_2_GRAY16_2_C   \
-    for (i=0; i<(dstW>>1); i++){ \
+    for (i=0; i<(dstW>>1); i++) { \
         const int i2= 2*i;       \
         int Y1= (buf0[i2  ]*yalpha1+buf1[i2  ]*yalpha)>>11;           \
         int Y2= (buf0[i2+1]*yalpha1+buf1[i2+1]*yalpha)>>11;           \
@@ -727,7 +709,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
     b = (type *)c->table_bU[U];\
 
 #define YSCALE_YUV_2_PACKED1_C(type,alpha) \
-    for (i=0; i<(dstW>>1); i++){\
+    for (i=0; i<(dstW>>1); i++) {\
         const int i2= 2*i;\
         int Y1= buf0[i2  ]>>7;\
         int Y2= buf0[i2+1]>>7;\
@@ -735,13 +717,13 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         int V= (uvbuf1[i+VOFW])>>7;\
         type av_unused *r, *b, *g;\
         int av_unused A1, A2;\
-        if (alpha){\
+        if (alpha) {\
             A1= abuf0[i2  ]>>7;\
             A2= abuf0[i2+1]>>7;\
         }\
 
 #define YSCALE_YUV_2_GRAY16_1_C \
-    for (i=0; i<(dstW>>1); i++){\
+    for (i=0; i<(dstW>>1); i++) {\
         const int i2= 2*i;\
         int Y1= buf0[i2  ]<<1;\
         int Y2= buf0[i2+1]<<1;\
@@ -753,7 +735,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
     b = (type *)c->table_bU[U];\
 
 #define YSCALE_YUV_2_PACKED1B_C(type,alpha) \
-    for (i=0; i<(dstW>>1); i++){\
+    for (i=0; i<(dstW>>1); i++) {\
         const int i2= 2*i;\
         int Y1= buf0[i2  ]>>7;\
         int Y2= buf0[i2+1]>>7;\
@@ -761,7 +743,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         int V= (uvbuf0[i+VOFW] + uvbuf1[i+VOFW])>>8;\
         type av_unused *r, *b, *g;\
         int av_unused A1, A2;\
-        if (alpha){\
+        if (alpha) {\
             A1= abuf0[i2  ]>>7;\
             A2= abuf0[i2+1]>>7;\
         }\
@@ -775,7 +757,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
 #define YSCALE_YUV_2_MONO2_C \
     const uint8_t * const d128=dither_8x8_220[y&7];\
     uint8_t *g= c->table_gU[128] + c->table_gV[128];\
-    for (i=0; i<dstW-7; i+=8){\
+    for (i=0; i<dstW-7; i+=8) {\
         int acc;\
         acc =       g[((buf0[i  ]*yalpha1+buf1[i  ]*yalpha)>>19) + d128[0]];\
         acc+= acc + g[((buf0[i+1]*yalpha1+buf1[i+1]*yalpha)>>19) + d128[1]];\
@@ -794,20 +776,18 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
     const uint8_t * const d128=dither_8x8_220[y&7];\
     uint8_t *g= c->table_gU[128] + c->table_gV[128];\
     int acc=0;\
-    for (i=0; i<dstW-1; i+=2){\
+    for (i=0; i<dstW-1; i+=2) {\
         int j;\
         int Y1=1<<18;\
         int Y2=1<<18;\
 \
-        for (j=0; j<lumFilterSize; j++)\
-        {\
+        for (j=0; j<lumFilterSize; j++) {\
             Y1 += lumSrc[j][i] * lumFilter[j];\
             Y2 += lumSrc[j][i+1] * lumFilter[j];\
         }\
         Y1>>=19;\
         Y2>>=19;\
-        if ((Y1|Y2)&256)\
-        {\
+        if ((Y1|Y2)&256) {\
             if (Y1>255)   Y1=255;\
             else if (Y1<0)Y1=0;\
             if (Y2>255)   Y2=255;\
@@ -815,7 +795,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         }\
         acc+= acc + g[Y1+d128[(i+0)&7]];\
         acc+= acc + g[Y2+d128[(i+1)&7]];\
-        if ((i&7)==6){\
+        if ((i&7)==6) {\
             ((uint8_t*)dest)[0]= c->dstFormat == PIX_FMT_MONOBLACK ? acc : ~acc;\
             dest++;\
         }\
@@ -823,8 +803,7 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
 
 
 #define YSCALE_YUV_2_ANYRGB_C(func, func2, func_g16, func_monoblack)\
-    switch(c->dstFormat)\
-    {\
+    switch(c->dstFormat) {\
     case PIX_FMT_RGB48BE:\
     case PIX_FMT_RGB48LE:\
         func(uint8_t,0)\
@@ -845,19 +824,19 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         break;\
     case PIX_FMT_RGBA:\
     case PIX_FMT_BGRA:\
-        if (CONFIG_SMALL){\
+        if (CONFIG_SMALL) {\
             int needAlpha = CONFIG_SWSCALE_ALPHA && c->alpPixBuf;\
             func(uint32_t,needAlpha)\
                 ((uint32_t*)dest)[i2+0]= r[Y1] + g[Y1] + b[Y1] + (needAlpha ? (A1<<24) : 0);\
                 ((uint32_t*)dest)[i2+1]= r[Y2] + g[Y2] + b[Y2] + (needAlpha ? (A2<<24) : 0);\
             }\
-        }else{\
-            if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf){\
+        } else {\
+            if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf) {\
                 func(uint32_t,1)\
                     ((uint32_t*)dest)[i2+0]= r[Y1] + g[Y1] + b[Y1] + (A1<<24);\
                     ((uint32_t*)dest)[i2+1]= r[Y2] + g[Y2] + b[Y2] + (A2<<24);\
                 }\
-            }else{\
+            } else {\
                 func(uint32_t,0)\
                     ((uint32_t*)dest)[i2+0]= r[Y1] + g[Y1] + b[Y1];\
                     ((uint32_t*)dest)[i2+1]= r[Y2] + g[Y2] + b[Y2];\
@@ -867,19 +846,19 @@ static inline void yuv2nv12XinC(const int16_t *lumFilter, const int16_t **lumSrc
         break;\
     case PIX_FMT_ARGB:\
     case PIX_FMT_ABGR:\
-        if (CONFIG_SMALL){\
+        if (CONFIG_SMALL) {\
             int needAlpha = CONFIG_SWSCALE_ALPHA && c->alpPixBuf;\
             func(uint32_t,needAlpha)\
                 ((uint32_t*)dest)[i2+0]= r[Y1] + g[Y1] + b[Y1] + (needAlpha ? A1 : 0);\
                 ((uint32_t*)dest)[i2+1]= r[Y2] + g[Y2] + b[Y2] + (needAlpha ? A2 : 0);\
             }\
-        }else{\
-            if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf){\
+        } else {\
+            if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf) {\
                 func(uint32_t,1)\
                     ((uint32_t*)dest)[i2+0]= r[Y1] + g[Y1] + b[Y1] + A1;\
                     ((uint32_t*)dest)[i2+1]= r[Y2] + g[Y2] + b[Y2] + A2;\
                 }\
-            }else{\
+            } else {\
                 func(uint32_t,0)\
                     ((uint32_t*)dest)[i2+0]= r[Y1] + g[Y1] + b[Y1];\
                     ((uint32_t*)dest)[i2+1]= r[Y2] + g[Y2] + b[Y2];\
@@ -1029,14 +1008,14 @@ static inline void yuv2rgbXinC_full(SwsContext *c, const int16_t *lumFilter, con
     int step= fmt_depth(c->dstFormat)/8;
     int aidx= 3;
 
-    switch(c->dstFormat){
+    switch(c->dstFormat) {
     case PIX_FMT_ARGB:
         dest++;
         aidx= 0;
     case PIX_FMT_RGB24:
         aidx--;
     case PIX_FMT_RGBA:
-        if (CONFIG_SMALL){
+        if (CONFIG_SMALL) {
             int needAlpha = CONFIG_SWSCALE_ALPHA && c->alpPixBuf;
             YSCALE_YUV_2_RGBX_FULL_C(1<<21, needAlpha)
                 dest[aidx]= needAlpha ? A : 255;
@@ -1045,8 +1024,8 @@ static inline void yuv2rgbXinC_full(SwsContext *c, const int16_t *lumFilter, con
                 dest[2]= B>>22;
                 dest+= step;
             }
-        }else{
-            if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf){
+        } else {
+            if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf) {
                 YSCALE_YUV_2_RGBX_FULL_C(1<<21, 1)
                     dest[aidx]= A;
                     dest[0]= R>>22;
@@ -1054,7 +1033,7 @@ static inline void yuv2rgbXinC_full(SwsContext *c, const int16_t *lumFilter, con
                     dest[2]= B>>22;
                     dest+= step;
                 }
-            }else{
+            } else {
                 YSCALE_YUV_2_RGBX_FULL_C(1<<21, 0)
                     dest[aidx]= 255;
                     dest[0]= R>>22;
@@ -1071,7 +1050,7 @@ static inline void yuv2rgbXinC_full(SwsContext *c, const int16_t *lumFilter, con
     case PIX_FMT_BGR24:
         aidx--;
     case PIX_FMT_BGRA:
-        if (CONFIG_SMALL){
+        if (CONFIG_SMALL) {
             int needAlpha = CONFIG_SWSCALE_ALPHA && c->alpPixBuf;
             YSCALE_YUV_2_RGBX_FULL_C(1<<21, needAlpha)
                 dest[aidx]= needAlpha ? A : 255;
@@ -1080,8 +1059,8 @@ static inline void yuv2rgbXinC_full(SwsContext *c, const int16_t *lumFilter, con
                 dest[2]= R>>22;
                 dest+= step;
             }
-        }else{
-            if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf){
+        } else {
+            if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf) {
                 YSCALE_YUV_2_RGBX_FULL_C(1<<21, 1)
                     dest[aidx]= A;
                     dest[0]= B>>22;
@@ -1089,7 +1068,7 @@ static inline void yuv2rgbXinC_full(SwsContext *c, const int16_t *lumFilter, con
                     dest[2]= R>>22;
                     dest+= step;
                 }
-            }else{
+            } else {
                 YSCALE_YUV_2_RGBX_FULL_C(1<<21, 0)
                     dest[aidx]= 255;
                     dest[0]= B>>22;
@@ -1105,16 +1084,18 @@ static inline void yuv2rgbXinC_full(SwsContext *c, const int16_t *lumFilter, con
     }
 }
 
-static void fillPlane(uint8_t* plane, int stride, int width, int height, int y, uint8_t val){
+static void fillPlane(uint8_t* plane, int stride, int width, int height, int y, uint8_t val)
+{
     int i;
     uint8_t *ptr = plane + stride*y;
-    for (i=0; i<height; i++){
+    for (i=0; i<height; i++) {
         memset(ptr, val, width);
         ptr += stride;
     }
 }
 
-static inline void rgb48ToY(uint8_t *dst, const uint8_t *src, int width)
+static inline void rgb48ToY(uint8_t *dst, const uint8_t *src, int width,
+                            uint32_t *unused)
 {
     int i;
     for (i = 0; i < width; i++) {
@@ -1127,7 +1108,8 @@ static inline void rgb48ToY(uint8_t *dst, const uint8_t *src, int width)
 }
 
 static inline void rgb48ToUV(uint8_t *dstU, uint8_t *dstV,
-                             uint8_t *src1, uint8_t *src2, int width)
+                             const uint8_t *src1, const uint8_t *src2,
+                             int width, uint32_t *unused)
 {
     int i;
     assert(src1==src2);
@@ -1142,7 +1124,8 @@ static inline void rgb48ToUV(uint8_t *dstU, uint8_t *dstV,
 }
 
 static inline void rgb48ToUV_half(uint8_t *dstU, uint8_t *dstV,
-                                  uint8_t *src1, uint8_t *src2, int width)
+                                  const uint8_t *src1, const uint8_t *src2,
+                                  int width, uint32_t *unused)
 {
     int i;
     assert(src1==src2);
@@ -1160,8 +1143,7 @@ static inline void rgb48ToUV_half(uint8_t *dstU, uint8_t *dstV,
 static inline void name(uint8_t *dst, const uint8_t *src, long width, uint32_t *unused)\
 {\
     int i;\
-    for (i=0; i<width; i++)\
-    {\
+    for (i=0; i<width; i++) {\
         int b= (((const type*)src)[i]>>shb)&maskb;\
         int g= (((const type*)src)[i]>>shg)&maskg;\
         int r= (((const type*)src)[i]>>shr)&maskr;\
@@ -1177,9 +1159,10 @@ BGR2Y(uint16_t, bgr15ToY, 0, 0, 0, 0x001F, 0x03E0, 0x7C00, RY<<10, GY<<5, BY
 BGR2Y(uint16_t, rgb16ToY, 0, 0, 0, 0xF800, 0x07E0, 0x001F, RY    , GY<<5, BY<<11, RGB2YUV_SHIFT+8)
 BGR2Y(uint16_t, rgb15ToY, 0, 0, 0, 0x7C00, 0x03E0, 0x001F, RY    , GY<<5, BY<<10, RGB2YUV_SHIFT+7)
 
-static inline void abgrToA(uint8_t *dst, const uint8_t *src, long width, uint32_t *unused){
+static inline void abgrToA(uint8_t *dst, const uint8_t *src, long width, uint32_t *unused)
+{
     int i;
-    for (i=0; i<width; i++){
+    for (i=0; i<width; i++) {
         dst[i]= src[4*i];
     }
 }
@@ -1188,8 +1171,7 @@ static inline void abgrToA(uint8_t *dst, const uint8_t *src, long width, uint32_
 static inline void name(uint8_t *dstU, uint8_t *dstV, const uint8_t *src, const uint8_t *dummy, long width, uint32_t *unused)\
 {\
     int i;\
-    for (i=0; i<width; i++)\
-    {\
+    for (i=0; i<width; i++) {\
         int b= (((const type*)src)[i]&maskb)>>shb;\
         int g= (((const type*)src)[i]&maskg)>>shg;\
         int r= (((const type*)src)[i]&maskr)>>shr;\
@@ -1201,8 +1183,7 @@ static inline void name(uint8_t *dstU, uint8_t *dstV, const uint8_t *src, const
 static inline void name ## _half(uint8_t *dstU, uint8_t *dstV, const uint8_t *src, const uint8_t *dummy, long width, uint32_t *unused)\
 {\
     int i;\
-    for (i=0; i<width; i++)\
-    {\
+    for (i=0; i<width; i++) {\
         int pix0= ((const type*)src)[2*i+0];\
         int pix1= ((const type*)src)[2*i+1];\
         int g= (pix0&~(maskr|maskb))+(pix1&~(maskr|maskb));\
@@ -1227,8 +1208,7 @@ BGR2UV(uint16_t, rgb15ToUV, 0, 0, 0,          0,   0x7C00, 0x03E0,   0x001F, RU
 static inline void palToY(uint8_t *dst, const uint8_t *src, long width, uint32_t *pal)
 {
     int i;
-    for (i=0; i<width; i++)
-    {
+    for (i=0; i<width; i++) {
         int d= src[i];
 
         dst[i]= pal[d] & 0xFF;
@@ -1241,8 +1221,7 @@ static inline void palToUV(uint8_t *dstU, uint8_t *dstV,
 {
     int i;
     assert(src1 == src2);
-    for (i=0; i<width; i++)
-    {
+    for (i=0; i<width; i++) {
         int p= pal[src1[i]];
 
         dstU[i]= p>>8;
@@ -1253,7 +1232,7 @@ static inline void palToUV(uint8_t *dstU, uint8_t *dstV,
 static inline void monowhite2Y(uint8_t *dst, const uint8_t *src, long width, uint32_t *unused)
 {
     int i, j;
-    for (i=0; i<width/8; i++){
+    for (i=0; i<width/8; i++) {
         int d= ~src[i];
         for(j=0; j<8; j++)
             dst[8*i+j]= ((d>>(7-j))&1)*255;
@@ -1263,7 +1242,7 @@ static inline void monowhite2Y(uint8_t *dst, const uint8_t *src, long width, uin
 static inline void monoblack2Y(uint8_t *dst, const uint8_t *src, long width, uint32_t *unused)
 {
     int i, j;
-    for (i=0; i<width/8; i++){
+    for (i=0; i<width/8; i++) {
         int d= src[i];
         for(j=0; j<8; j++)
             dst[8*i+j]= ((d>>(7-j))&1)*255;
@@ -1362,12 +1341,12 @@ static inline void monoblack2Y(uint8_t *dst, const uint8_t *src, long width, uin
 static double getSplineCoeff(double a, double b, double c, double d, double dist)
 {
 //    printf("%f %f %f %f %f\n", a,b,c,d,dist);
-    if (dist<=1.0)      return ((d*dist + c)*dist + b)*dist +a;
-    else                return getSplineCoeff(        0.0,
-                                             b+ 2.0*c + 3.0*d,
-                                                    c + 3.0*d,
-                                            -b- 3.0*c - 6.0*d,
-                                            dist-1.0);
+    if (dist<=1.0) return ((d*dist + c)*dist + b)*dist +a;
+    else           return getSplineCoeff(        0.0,
+                                          b+ 2.0*c + 3.0*d,
+                                                 c + 3.0*d,
+                                         -b- 3.0*c - 6.0*d,
+                                         dist-1.0);
 }
 
 static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outFilterSize, int xInc,
@@ -1388,65 +1367,54 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
 #endif
 
     // NOTE: the +1 is for the MMX scaler which reads over the end
-    *filterPos = av_malloc((dstW+1)*sizeof(int16_t));
+    FF_ALLOC_OR_GOTO(NULL, *filterPos, (dstW+1)*sizeof(int16_t), fail);
 
-    if (FFABS(xInc - 0x10000) <10) // unscaled
-    {
+    if (FFABS(xInc - 0x10000) <10) { // unscaled
         int i;
         filterSize= 1;
-        filter= av_mallocz(dstW*sizeof(*filter)*filterSize);
+        FF_ALLOCZ_OR_GOTO(NULL, filter, dstW*sizeof(*filter)*filterSize, fail);
 
-        for (i=0; i<dstW; i++)
-        {
+        for (i=0; i<dstW; i++) {
             filter[i*filterSize]= fone;
             (*filterPos)[i]=i;
         }
 
-    }
-    else if (flags&SWS_POINT) // lame looking point sampling mode
-    {
+    } else if (flags&SWS_POINT) { // lame looking point sampling mode
         int i;
         int xDstInSrc;
         filterSize= 1;
-        filter= av_malloc(dstW*sizeof(*filter)*filterSize);
+        FF_ALLOC_OR_GOTO(NULL, filter, dstW*sizeof(*filter)*filterSize, fail);
 
         xDstInSrc= xInc/2 - 0x8000;
-        for (i=0; i<dstW; i++)
-        {
+        for (i=0; i<dstW; i++) {
             int xx= (xDstInSrc - ((filterSize-1)<<15) + (1<<15))>>16;
 
             (*filterPos)[i]= xx;
             filter[i]= fone;
             xDstInSrc+= xInc;
         }
-    }
-    else if ((xInc <= (1<<16) && (flags&SWS_AREA)) || (flags&SWS_FAST_BILINEAR)) // bilinear upscale
-    {
+    } else if ((xInc <= (1<<16) && (flags&SWS_AREA)) || (flags&SWS_FAST_BILINEAR)) { // bilinear upscale
         int i;
         int xDstInSrc;
         filterSize= 2;
-        filter= av_malloc(dstW*sizeof(*filter)*filterSize);
+        FF_ALLOC_OR_GOTO(NULL, filter, dstW*sizeof(*filter)*filterSize, fail);
 
         xDstInSrc= xInc/2 - 0x8000;
-        for (i=0; i<dstW; i++)
-        {
+        for (i=0; i<dstW; i++) {
             int xx= (xDstInSrc - ((filterSize-1)<<15) + (1<<15))>>16;
             int j;
 
             (*filterPos)[i]= xx;
-                //bilinear upscale / linear interpolate / area averaging
-                for (j=0; j<filterSize; j++)
-                {
-                    int64_t coeff= fone - FFABS((xx<<16) - xDstInSrc)*(fone>>16);
-                    if (coeff<0) coeff=0;
-                    filter[i*filterSize + j]= coeff;
-                    xx++;
-                }
+            //bilinear upscale / linear interpolate / area averaging
+            for (j=0; j<filterSize; j++) {
+                int64_t coeff= fone - FFABS((xx<<16) - xDstInSrc)*(fone>>16);
+                if (coeff<0) coeff=0;
+                filter[i*filterSize + j]= coeff;
+                xx++;
+            }
             xDstInSrc+= xInc;
         }
-    }
-    else
-    {
+    } else {
         int xDstInSrc;
         int sizeFactor;
 
@@ -1468,16 +1436,14 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
 
         if (filterSize > srcW-2) filterSize=srcW-2;
 
-        filter= av_malloc(dstW*sizeof(*filter)*filterSize);
+        FF_ALLOC_OR_GOTO(NULL, filter, dstW*sizeof(*filter)*filterSize, fail);
 
         xDstInSrc= xInc - 0x10000;
-        for (i=0; i<dstW; i++)
-        {
+        for (i=0; i<dstW; i++) {
             int xx= (xDstInSrc - ((filterSize-2)<<16)) / (1<<17);
             int j;
             (*filterPos)[i]= xx;
-            for (j=0; j<filterSize; j++)
-            {
+            for (j=0; j<filterSize; j++) {
                 int64_t d= ((int64_t)FFABS((xx<<17) - xDstInSrc))<<13;
                 double floatd;
                 int64_t coeff;
@@ -1486,8 +1452,7 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
                     d= d*dstW/srcW;
                 floatd= d * (1.0/(1<<30));
 
-                if (flags & SWS_BICUBIC)
-                {
+                if (flags & SWS_BICUBIC) {
                     int64_t B= (param[0] != SWS_PARAM_DEFAULT ? param[0] :   0) * (1<<24);
                     int64_t C= (param[1] != SWS_PARAM_DEFAULT ? param[1] : 0.6) * (1<<24);
                     int64_t dd = ( d*d)>>30;
@@ -1501,14 +1466,12 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
                         coeff=0.0;
                     coeff *= fone>>(30+24);
                 }
-/*                else if (flags & SWS_X)
-                {
+/*                else if (flags & SWS_X) {
                     double p= param ? param*0.01 : 0.3;
                     coeff = d ? sin(d*PI)/(d*PI) : 1.0;
                     coeff*= pow(2.0, - p*d*d);
                 }*/
-                else if (flags & SWS_X)
-                {
+                else if (flags & SWS_X) {
                     double A= param[0] != SWS_PARAM_DEFAULT ? param[0] : 1.0;
                     double c;
 
@@ -1519,42 +1482,29 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
                     if (c<0.0)      c= -pow(-c, A);
                     else            c=  pow( c, A);
                     coeff= (c*0.5 + 0.5)*fone;
-                }
-                else if (flags & SWS_AREA)
-                {
+                } else if (flags & SWS_AREA) {
                     int64_t d2= d - (1<<29);
                     if      (d2*xInc < -(1LL<<(29+16))) coeff= 1.0 * (1LL<<(30+16));
                     else if (d2*xInc <  (1LL<<(29+16))) coeff= -d2*xInc + (1LL<<(29+16));
                     else coeff=0.0;
                     coeff *= fone>>(30+16);
-                }
-                else if (flags & SWS_GAUSS)
-                {
+                } else if (flags & SWS_GAUSS) {
                     double p= param[0] != SWS_PARAM_DEFAULT ? param[0] : 3.0;
                     coeff = (pow(2.0, - p*floatd*floatd))*fone;
-                }
-                else if (flags & SWS_SINC)
-                {
+                } else if (flags & SWS_SINC) {
                     coeff = (d ? sin(floatd*PI)/(floatd*PI) : 1.0)*fone;
-                }
-                else if (flags & SWS_LANCZOS)
-                {
+                } else if (flags & SWS_LANCZOS) {
                     double p= param[0] != SWS_PARAM_DEFAULT ? param[0] : 3.0;
                     coeff = (d ? sin(floatd*PI)*sin(floatd*PI/p)/(floatd*floatd*PI*PI/p) : 1.0)*fone;
                     if (floatd>p) coeff=0;
-                }
-                else if (flags & SWS_BILINEAR)
-                {
+                } else if (flags & SWS_BILINEAR) {
                     coeff= (1<<30) - d;
                     if (coeff<0) coeff=0;
                     coeff *= fone >> 30;
-                }
-                else if (flags & SWS_SPLINE)
-                {
+                } else if (flags & SWS_SPLINE) {
                     double p=-2.196152422706632;
                     coeff = getSplineCoeff(1.0, 0.0, p, -p-1.0, floatd) * fone;
-                }
-                else {
+                } else {
                     coeff= 0.0; //GCC warning killer
                     assert(0);
                 }
@@ -1574,18 +1524,17 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
     if (srcFilter) filter2Size+= srcFilter->length - 1;
     if (dstFilter) filter2Size+= dstFilter->length - 1;
     assert(filter2Size>0);
-    filter2= av_mallocz(filter2Size*dstW*sizeof(*filter2));
+    FF_ALLOCZ_OR_GOTO(NULL, filter2, filter2Size*dstW*sizeof(*filter2), fail);
 
-    for (i=0; i<dstW; i++)
-    {
+    for (i=0; i<dstW; i++) {
         int j, k;
 
-        if(srcFilter){
-            for (k=0; k<srcFilter->length; k++){
+        if(srcFilter) {
+            for (k=0; k<srcFilter->length; k++) {
                 for (j=0; j<filterSize; j++)
                     filter2[i*filter2Size + k + j] += srcFilter->coeff[k]*filter[i*filterSize + j];
             }
-        }else{
+        } else {
             for (j=0; j<filterSize; j++)
                 filter2[i*filter2Size + j]= filter[i*filterSize + j];
         }
@@ -1598,15 +1547,13 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
     /* try to reduce the filter-size (step1 find size and shift left) */
     // Assume it is near normalized (*0.5 or *2.0 is OK but * 0.001 is not).
     minFilterSize= 0;
-    for (i=dstW-1; i>=0; i--)
-    {
+    for (i=dstW-1; i>=0; i--) {
         int min= filter2Size;
         int j;
         int64_t cutOff=0.0;
 
         /* get rid off near zero elements on the left by shifting left */
-        for (j=0; j<filter2Size; j++)
-        {
+        for (j=0; j<filter2Size; j++) {
             int k;
             cutOff += FFABS(filter2[i*filter2Size]);
 
@@ -1624,8 +1571,7 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
 
         cutOff=0;
         /* count near zeros on the right */
-        for (j=filter2Size-1; j>0; j--)
-        {
+        for (j=filter2Size-1; j>0; j--) {
             cutOff += FFABS(filter2[i*filter2Size + j]);
 
             if (cutOff > SWS_MAX_REDUCE_CUTOFF*fone) break;
@@ -1661,18 +1607,16 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
     assert(filterSize > 0);
     filter= av_malloc(filterSize*dstW*sizeof(*filter));
     if (filterSize >= MAX_FILTER_SIZE*16/((flags&SWS_ACCURATE_RND) ? APCK_SIZE : 16) || !filter)
-        goto error;
+        goto fail;
     *outFilterSize= filterSize;
 
     if (flags&SWS_PRINT_INFO)
         av_log(NULL, AV_LOG_VERBOSE, "SwScaler: reducing / aligning filtersize %d -> %d\n", filter2Size, filterSize);
     /* try to reduce the filter-size (step2 reduce it) */
-    for (i=0; i<dstW; i++)
-    {
+    for (i=0; i<dstW; i++) {
         int j;
 
-        for (j=0; j<filterSize; j++)
-        {
+        for (j=0; j<filterSize; j++) {
             if (j>=filter2Size) filter[i*filterSize + j]= 0;
             else               filter[i*filterSize + j]= filter2[i*filter2Size + j];
             if((flags & SWS_BITEXACT) && j>=minFilterSize)
@@ -1684,14 +1628,11 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
     //FIXME try to align filterPos if possible
 
     //fix borders
-    for (i=0; i<dstW; i++)
-    {
+    for (i=0; i<dstW; i++) {
         int j;
-        if ((*filterPos)[i] < 0)
-        {
+        if ((*filterPos)[i] < 0) {
             // move filter coefficients left to compensate for filterPos
-            for (j=1; j<filterSize; j++)
-            {
+            for (j=1; j<filterSize; j++) {
                 int left= FFMAX(j + (*filterPos)[i], 0);
                 filter[i*filterSize + left] += filter[i*filterSize + j];
                 filter[i*filterSize + j]=0;
@@ -1699,12 +1640,10 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
             (*filterPos)[i]= 0;
         }
 
-        if ((*filterPos)[i] + filterSize > srcW)
-        {
+        if ((*filterPos)[i] + filterSize > srcW) {
             int shift= (*filterPos)[i] + filterSize - srcW;
             // move filter coefficients right to compensate for filterPos
-            for (j=filterSize-2; j>=0; j--)
-            {
+            for (j=filterSize-2; j>=0; j--) {
                 int right= FFMIN(j + shift, filterSize-1);
                 filter[i*filterSize +right] += filter[i*filterSize +j];
                 filter[i*filterSize +j]=0;
@@ -1715,22 +1654,19 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
 
     // Note the +1 is for the MMX scaler which reads over the end
     /* align at 16 for AltiVec (needed by hScale_altivec_real) */
-    *outFilter= av_mallocz(*outFilterSize*(dstW+1)*sizeof(int16_t));
+    FF_ALLOCZ_OR_GOTO(NULL, *outFilter, *outFilterSize*(dstW+1)*sizeof(int16_t), fail);
 
     /* normalize & store in outFilter */
-    for (i=0; i<dstW; i++)
-    {
+    for (i=0; i<dstW; i++) {
         int j;
         int64_t error=0;
         int64_t sum=0;
 
-        for (j=0; j<filterSize; j++)
-        {
+        for (j=0; j<filterSize; j++) {
             sum+= filter[i*filterSize + j];
         }
         sum= (sum + one/2)/ one;
-        for (j=0; j<*outFilterSize; j++)
-        {
+        for (j=0; j<*outFilterSize; j++) {
             int64_t v= filter[i*filterSize + j] + error;
             int intV= ROUNDED_DIV(v, sum);
             (*outFilter)[i*(*outFilterSize) + j]= intV;
@@ -1739,21 +1675,20 @@ static inline int initFilter(int16_t **outFilter, int16_t **filterPos, int *outF
     }
 
     (*filterPos)[dstW]= (*filterPos)[dstW-1]; // the MMX scaler will read over the end
-    for (i=0; i<*outFilterSize; i++)
-    {
+    for (i=0; i<*outFilterSize; i++) {
         int j= dstW*(*outFilterSize);
         (*outFilter)[j + i]= (*outFilter)[j + i - (*outFilterSize)];
     }
 
     ret=0;
-error:
+fail:
     av_free(filter);
     av_free(filter2);
     return ret;
 }
 
 #ifdef COMPILE_MMX2
-static void initMMX2HScaler(int dstW, int xInc, uint8_t *filterCode, int16_t *filter, int32_t *filterPos, int numSplits)
+static int initMMX2HScaler(int dstW, int xInc, uint8_t *filterCode, int16_t *filter, int32_t *filterPos, int numSplits)
 {
     uint8_t *fragmentA;
     x86_reg imm8OfPShufW1A;
@@ -1768,6 +1703,12 @@ static void initMMX2HScaler(int dstW, int xInc, uint8_t *filterCode, int16_t *fi
     int xpos, i;
 
     // create an optimized horizontal scaling routine
+    /* This scaler is made of runtime-generated MMX2 code using specially
+     * tuned pshufw instructions. For every four output pixels, if four
+     * input pixels are enough for the fast bilinear scaling, then a chunk
+     * of fragmentB is used. If five input pixels are needed, then a chunk
+     * of fragmentA is used.
+     */
 
     //code fragment
 
@@ -1852,12 +1793,10 @@ static void initMMX2HScaler(int dstW, int xInc, uint8_t *filterCode, int16_t *fi
     xpos= 0; //lumXInc/2 - 0x8000; // difference between pixel centers
     fragmentPos=0;
 
-    for (i=0; i<dstW/numSplits; i++)
-    {
+    for (i=0; i<dstW/numSplits; i++) {
         int xx=xpos>>16;
 
-        if ((i&3) == 0)
-        {
+        if ((i&3) == 0) {
             int a=0;
             int b=((xpos+xInc)>>16) - xx;
             int c=((xpos+xInc*2)>>16) - xx;
@@ -1870,47 +1809,43 @@ static void initMMX2HScaler(int dstW, int xInc, uint8_t *filterCode, int16_t *fi
             int maxShift= 3-(d+inc);
             int shift=0;
 
-            filter[i  ] = (( xpos         & 0xFFFF) ^ 0xFFFF)>>9;
-            filter[i+1] = (((xpos+xInc  ) & 0xFFFF) ^ 0xFFFF)>>9;
-            filter[i+2] = (((xpos+xInc*2) & 0xFFFF) ^ 0xFFFF)>>9;
-            filter[i+3] = (((xpos+xInc*3) & 0xFFFF) ^ 0xFFFF)>>9;
-            filterPos[i/2]= xx;
+            if (filterCode) {
+                filter[i  ] = (( xpos         & 0xFFFF) ^ 0xFFFF)>>9;
+                filter[i+1] = (((xpos+xInc  ) & 0xFFFF) ^ 0xFFFF)>>9;
+                filter[i+2] = (((xpos+xInc*2) & 0xFFFF) ^ 0xFFFF)>>9;
+                filter[i+3] = (((xpos+xInc*3) & 0xFFFF) ^ 0xFFFF)>>9;
+                filterPos[i/2]= xx;
 
-            memcpy(filterCode + fragmentPos, fragment, fragmentLength);
+                memcpy(filterCode + fragmentPos, fragment, fragmentLength);
 
-            filterCode[fragmentPos + imm8OfPShufW1]=
-                (a+inc) | ((b+inc)<<2) | ((c+inc)<<4) | ((d+inc)<<6);
-            filterCode[fragmentPos + imm8OfPShufW2]=
-                a | (b<<2) | (c<<4) | (d<<6);
+                filterCode[fragmentPos + imm8OfPShufW1]=
+                    (a+inc) | ((b+inc)<<2) | ((c+inc)<<4) | ((d+inc)<<6);
+                filterCode[fragmentPos + imm8OfPShufW2]=
+                    a | (b<<2) | (c<<4) | (d<<6);
 
-            if (i+4-inc>=dstW) shift=maxShift; //avoid overread
-            else if ((filterPos[i/2]&3) <= maxShift) shift=filterPos[i/2]&3; //Align
+                if (i+4-inc>=dstW) shift=maxShift; //avoid overread
+                else if ((filterPos[i/2]&3) <= maxShift) shift=filterPos[i/2]&3; //Align
 
-            if (shift && i>=shift)
-            {
-                filterCode[fragmentPos + imm8OfPShufW1]+= 0x55*shift;
-                filterCode[fragmentPos + imm8OfPShufW2]+= 0x55*shift;
-                filterPos[i/2]-=shift;
+                if (shift && i>=shift) {
+                    filterCode[fragmentPos + imm8OfPShufW1]+= 0x55*shift;
+                    filterCode[fragmentPos + imm8OfPShufW2]+= 0x55*shift;
+                    filterPos[i/2]-=shift;
+                }
             }
 
             fragmentPos+= fragmentLength;
 
-            filterCode[fragmentPos]= RET;
+            if (filterCode)
+                filterCode[fragmentPos]= RET;
         }
         xpos+=xInc;
     }
-    filterPos[((i/2)+1)&(~1)]= xpos>>16; // needed to jump to the next part
-}
-#endif /* COMPILE_MMX2 */
+    if (filterCode)
+        filterPos[((i/2)+1)&(~1)]= xpos>>16; // needed to jump to the next part
 
-static void globalInit(void){
-    // generating tables:
-    int i;
-    for (i=0; i<768; i++){
-        int c= av_clip_uint8(i-256);
-        clip_table[i]=c;
-    }
+    return fragmentPos + 1;
 }
+#endif /* COMPILE_MMX2 */
 
 static SwsFunc getSwsFunc(SwsContext *c)
 {
@@ -1966,19 +1901,18 @@ static SwsFunc getSwsFunc(SwsContext *c)
 #endif //!CONFIG_RUNTIME_CPUDETECT
 }
 
-static int PlanarToNV12Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                               int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int PlanarToNV12Wrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                               int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *dst=dstParam[0] + dstStride[0]*srcSliceY;
     /* Copy Y plane */
     if (dstStride[0]==srcStride[0] && srcStride[0] > 0)
         memcpy(dst, src[0], srcSliceH*dstStride[0]);
-    else
-    {
+    else {
         int i;
         const uint8_t *srcPtr= src[0];
         uint8_t *dstPtr= dst;
-        for (i=0; i<srcSliceH; i++)
-        {
+        for (i=0; i<srcSliceH; i++) {
             memcpy(dstPtr, srcPtr, c->srcW);
             srcPtr+= srcStride[0];
             dstPtr+= dstStride[0];
@@ -1993,8 +1927,9 @@ static int PlanarToNV12Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], i
     return srcSliceH;
 }
 
-static int PlanarToYuy2Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                               int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int PlanarToYuy2Wrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                               int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *dst=dstParam[0] + dstStride[0]*srcSliceY;
 
     yv12toyuy2(src[0], src[1], src[2], dst, c->srcW, srcSliceH, srcStride[0], srcStride[1], dstStride[0]);
@@ -2002,8 +1937,9 @@ static int PlanarToYuy2Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], i
     return srcSliceH;
 }
 
-static int PlanarToUyvyWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                               int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int PlanarToUyvyWrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                               int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *dst=dstParam[0] + dstStride[0]*srcSliceY;
 
     yv12touyvy(src[0], src[1], src[2], dst, c->srcW, srcSliceH, srcStride[0], srcStride[1], dstStride[0]);
@@ -2011,8 +1947,9 @@ static int PlanarToUyvyWrapper(SwsContext *c, uint8_t* src[], int srcStride[], i
     return srcSliceH;
 }
 
-static int YUV422PToYuy2Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                                int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int YUV422PToYuy2Wrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                                int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *dst=dstParam[0] + dstStride[0]*srcSliceY;
 
     yuv422ptoyuy2(src[0],src[1],src[2],dst,c->srcW,srcSliceH,srcStride[0],srcStride[1],dstStride[0]);
@@ -2020,8 +1957,9 @@ static int YUV422PToYuy2Wrapper(SwsContext *c, uint8_t* src[], int srcStride[],
     return srcSliceH;
 }
 
-static int YUV422PToUyvyWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                                int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int YUV422PToUyvyWrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                                int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *dst=dstParam[0] + dstStride[0]*srcSliceY;
 
     yuv422ptouyvy(src[0],src[1],src[2],dst,c->srcW,srcSliceH,srcStride[0],srcStride[1],dstStride[0]);
@@ -2029,8 +1967,9 @@ static int YUV422PToUyvyWrapper(SwsContext *c, uint8_t* src[], int srcStride[],
     return srcSliceH;
 }
 
-static int YUYV2YUV420Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                               int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int YUYV2YUV420Wrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                               int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *ydst=dstParam[0] + dstStride[0]*srcSliceY;
     uint8_t *udst=dstParam[1] + dstStride[1]*srcSliceY/2;
     uint8_t *vdst=dstParam[2] + dstStride[2]*srcSliceY/2;
@@ -2043,8 +1982,9 @@ static int YUYV2YUV420Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], in
     return srcSliceH;
 }
 
-static int YUYV2YUV422Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                               int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int YUYV2YUV422Wrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                               int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *ydst=dstParam[0] + dstStride[0]*srcSliceY;
     uint8_t *udst=dstParam[1] + dstStride[1]*srcSliceY;
     uint8_t *vdst=dstParam[2] + dstStride[2]*srcSliceY;
@@ -2054,8 +1994,9 @@ static int YUYV2YUV422Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], in
     return srcSliceH;
 }
 
-static int UYVY2YUV420Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                               int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int UYVY2YUV420Wrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                               int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *ydst=dstParam[0] + dstStride[0]*srcSliceY;
     uint8_t *udst=dstParam[1] + dstStride[1]*srcSliceY/2;
     uint8_t *vdst=dstParam[2] + dstStride[2]*srcSliceY/2;
@@ -2068,8 +2009,9 @@ static int UYVY2YUV420Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], in
     return srcSliceH;
 }
 
-static int UYVY2YUV422Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                               int srcSliceH, uint8_t* dstParam[], int dstStride[]){
+static int UYVY2YUV422Wrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                               int srcSliceH, uint8_t* dstParam[], int dstStride[])
+{
     uint8_t *ydst=dstParam[0] + dstStride[0]*srcSliceY;
     uint8_t *udst=dstParam[1] + dstStride[1]*srcSliceY;
     uint8_t *vdst=dstParam[2] + dstStride[2]*srcSliceY;
@@ -2079,21 +2021,22 @@ static int UYVY2YUV422Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], in
     return srcSliceH;
 }
 
-static int pal2rgbWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                          int srcSliceH, uint8_t* dst[], int dstStride[]){
+static int pal2rgbWrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                          int srcSliceH, uint8_t* dst[], int dstStride[])
+{
     const enum PixelFormat srcFormat= c->srcFormat;
     const enum PixelFormat dstFormat= c->dstFormat;
     void (*conv)(const uint8_t *src, uint8_t *dst, long num_pixels,
                  const uint8_t *palette)=NULL;
     int i;
     uint8_t *dstPtr= dst[0] + dstStride[0]*srcSliceY;
-    uint8_t *srcPtr= src[0];
+    const uint8_t *srcPtr= src[0];
 
     if (!usePal(srcFormat))
         av_log(c, AV_LOG_ERROR, "internal error %s -> %s converter\n",
                sws_format_name(srcFormat), sws_format_name(dstFormat));
 
-    switch(dstFormat){
+    switch(dstFormat) {
     case PIX_FMT_RGB32  : conv = palette8topacked32; break;
     case PIX_FMT_BGR32  : conv = palette8topacked32; break;
     case PIX_FMT_BGR32_1: conv = palette8topacked32; break;
@@ -2115,8 +2058,9 @@ static int pal2rgbWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int sr
 }
 
 /* {RGB,BGR}{15,16,24,32,32_1} -> {RGB,BGR}{15,16,24,32} */
-static int rgb2rgbWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                          int srcSliceH, uint8_t* dst[], int dstStride[]){
+static int rgb2rgbWrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                          int srcSliceH, uint8_t* dst[], int dstStride[])
+{
     const enum PixelFormat srcFormat= c->srcFormat;
     const enum PixelFormat dstFormat= c->dstFormat;
     const int srcBpp= (fmt_depth(srcFormat) + 7) >> 3;
@@ -2127,8 +2071,8 @@ static int rgb2rgbWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int sr
 
     /* BGR -> BGR */
     if (  (isBGR(srcFormat) && isBGR(dstFormat))
-       || (isRGB(srcFormat) && isRGB(dstFormat))){
-        switch(srcId | (dstId<<4)){
+       || (isRGB(srcFormat) && isRGB(dstFormat))) {
+        switch(srcId | (dstId<<4)) {
         case 0x34: conv= rgb16to15; break;
         case 0x36: conv= rgb24to15; break;
         case 0x38: conv= rgb32to15; break;
@@ -2144,9 +2088,9 @@ static int rgb2rgbWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int sr
         default: av_log(c, AV_LOG_ERROR, "internal error %s -> %s converter\n",
                         sws_format_name(srcFormat), sws_format_name(dstFormat)); break;
         }
-    }else if (  (isBGR(srcFormat) && isRGB(dstFormat))
-             || (isRGB(srcFormat) && isBGR(dstFormat))){
-        switch(srcId | (dstId<<4)){
+    } else if (  (isBGR(srcFormat) && isRGB(dstFormat))
+             || (isRGB(srcFormat) && isBGR(dstFormat))) {
+        switch(srcId | (dstId<<4)) {
         case 0x33: conv= rgb15tobgr15; break;
         case 0x34: conv= rgb16tobgr15; break;
         case 0x36: conv= rgb24tobgr15; break;
@@ -2166,26 +2110,23 @@ static int rgb2rgbWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int sr
         default: av_log(c, AV_LOG_ERROR, "internal error %s -> %s converter\n",
                         sws_format_name(srcFormat), sws_format_name(dstFormat)); break;
         }
-    }else{
+    } else {
         av_log(c, AV_LOG_ERROR, "internal error %s -> %s converter\n",
                sws_format_name(srcFormat), sws_format_name(dstFormat));
     }
 
-    if(conv)
-    {
-        uint8_t *srcPtr= src[0];
+    if(conv) {
+        const uint8_t *srcPtr= src[0];
         if(srcFormat == PIX_FMT_RGB32_1 || srcFormat == PIX_FMT_BGR32_1)
             srcPtr += ALT32_CORR;
 
         if (dstStride[0]*srcBpp == srcStride[0]*dstBpp && srcStride[0] > 0)
             conv(srcPtr, dst[0] + dstStride[0]*srcSliceY, srcSliceH*srcStride[0]);
-        else
-        {
+        else {
             int i;
             uint8_t *dstPtr= dst[0] + dstStride[0]*srcSliceY;
 
-            for (i=0; i<srcSliceH; i++)
-            {
+            for (i=0; i<srcSliceH; i++) {
                 conv(srcPtr, dstPtr, c->srcW*srcBpp);
                 srcPtr+= srcStride[0];
                 dstPtr+= dstStride[0];
@@ -2195,8 +2136,9 @@ static int rgb2rgbWrapper(SwsContext *c, uint8_t* src[], int srcStride[], int sr
     return srcSliceH;
 }
 
-static int bgr24toyv12Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                              int srcSliceH, uint8_t* dst[], int dstStride[]){
+static int bgr24toyv12Wrapper(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                              int srcSliceH, uint8_t* dst[], int dstStride[])
+{
 
     rgb24toyv12(
         src[0],
@@ -2210,31 +2152,31 @@ static int bgr24toyv12Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], in
     return srcSliceH;
 }
 
-static int yvu9toyv12Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                             int srcSliceH, uint8_t* dst[], int dstStride[]){
+static int yvu9toyv12Wrapper(SwsContext *c, const const uint8_t* src[], int srcStride[], int srcSliceY,
+                             int srcSliceH, uint8_t* dst[], int dstStride[])
+{
     int i;
 
     /* copy Y */
     if (srcStride[0]==dstStride[0] && srcStride[0] > 0)
         memcpy(dst[0]+ srcSliceY*dstStride[0], src[0], srcStride[0]*srcSliceH);
-    else{
-        uint8_t *srcPtr= src[0];
+    else {
+        const uint8_t *srcPtr= src[0];
         uint8_t *dstPtr= dst[0] + dstStride[0]*srcSliceY;
 
-        for (i=0; i<srcSliceH; i++)
-        {
+        for (i=0; i<srcSliceH; i++) {
             memcpy(dstPtr, srcPtr, c->srcW);
             srcPtr+= srcStride[0];
             dstPtr+= dstStride[0];
         }
     }
 
-    if (c->dstFormat==PIX_FMT_YUV420P || c->dstFormat==PIX_FMT_YUVA420P){
+    if (c->dstFormat==PIX_FMT_YUV420P || c->dstFormat==PIX_FMT_YUVA420P) {
         planar2x(src[1], dst[1] + dstStride[1]*(srcSliceY >> 1), c->chrSrcW,
                  srcSliceH >> 2, srcStride[1], dstStride[1]);
         planar2x(src[2], dst[2] + dstStride[2]*(srcSliceY >> 1), c->chrSrcW,
                  srcSliceH >> 2, srcStride[2], dstStride[2]);
-    }else{
+    } else {
         planar2x(src[1], dst[2] + dstStride[2]*(srcSliceY >> 1), c->chrSrcW,
                  srcSliceH >> 2, srcStride[1], dstStride[2]);
         planar2x(src[2], dst[1] + dstStride[1]*(srcSliceY >> 1), c->chrSrcW,
@@ -2246,15 +2188,14 @@ static int yvu9toyv12Wrapper(SwsContext *c, uint8_t* src[], int srcStride[], int
 }
 
 /* unscaled copy like stuff (assumes nearly identical formats) */
-static int packedCopy(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
+static int packedCopy(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
                       int srcSliceH, uint8_t* dst[], int dstStride[])
 {
     if (dstStride[0]==srcStride[0] && srcStride[0] > 0)
         memcpy(dst[0] + dstStride[0]*srcSliceY, src[0], srcSliceH*dstStride[0]);
-    else
-    {
+    else {
         int i;
-        uint8_t *srcPtr= src[0];
+        const uint8_t *srcPtr= src[0];
         uint8_t *dstPtr= dst[0] + dstStride[0]*srcSliceY;
         int length=0;
 
@@ -2263,8 +2204,7 @@ static int packedCopy(SwsContext *c, uint8_t* src[], int srcStride[], int srcSli
            && length+c->srcW <= FFABS(srcStride[0])) length+= c->srcW;
         assert(length!=0);
 
-        for (i=0; i<srcSliceH; i++)
-        {
+        for (i=0; i<srcSliceH; i++) {
             memcpy(dstPtr, srcPtr, length);
             srcPtr+= srcStride[0];
             dstPtr+= dstStride[0];
@@ -2273,47 +2213,45 @@ static int packedCopy(SwsContext *c, uint8_t* src[], int srcStride[], int srcSli
     return srcSliceH;
 }
 
-static int planarCopy(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
+static int planarCopy(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
                       int srcSliceH, uint8_t* dst[], int dstStride[])
 {
     int plane, i, j;
-    for (plane=0; plane<4; plane++)
-    {
+    for (plane=0; plane<4; plane++) {
         int length= (plane==0 || plane==3) ? c->srcW  : -((-c->srcW  )>>c->chrDstHSubSample);
         int y=      (plane==0 || plane==3) ? srcSliceY: -((-srcSliceY)>>c->chrDstVSubSample);
         int height= (plane==0 || plane==3) ? srcSliceH: -((-srcSliceH)>>c->chrDstVSubSample);
-        uint8_t *srcPtr= src[plane];
+        const uint8_t *srcPtr= src[plane];
         uint8_t *dstPtr= dst[plane] + dstStride[plane]*y;
 
         if (!dst[plane]) continue;
         // ignore palette for GRAY8
         if (plane == 1 && !dst[2]) continue;
-        if (!src[plane] || (plane == 1 && !src[2])){
+        if (!src[plane] || (plane == 1 && !src[2])) {
             if(is16BPS(c->dstFormat))
                 length*=2;
             fillPlane(dst[plane], dstStride[plane], length, height, y, (plane==3) ? 255 : 128);
-        }else
-        {
-            if(is16BPS(c->srcFormat) && !is16BPS(c->dstFormat)){
+        } else {
+            if(is16BPS(c->srcFormat) && !is16BPS(c->dstFormat)) {
                 if (!isBE(c->srcFormat)) srcPtr++;
-                for (i=0; i<height; i++){
+                for (i=0; i<height; i++) {
                     for (j=0; j<length; j++) dstPtr[j] = srcPtr[j<<1];
                     srcPtr+= srcStride[plane];
                     dstPtr+= dstStride[plane];
                 }
-            }else if(!is16BPS(c->srcFormat) && is16BPS(c->dstFormat)){
-                for (i=0; i<height; i++){
-                    for (j=0; j<length; j++){
+            } else if(!is16BPS(c->srcFormat) && is16BPS(c->dstFormat)) {
+                for (i=0; i<height; i++) {
+                    for (j=0; j<length; j++) {
                         dstPtr[ j<<1   ] = srcPtr[j];
                         dstPtr[(j<<1)+1] = srcPtr[j];
                     }
                     srcPtr+= srcStride[plane];
                     dstPtr+= dstStride[plane];
                 }
-            }else if(is16BPS(c->srcFormat) && is16BPS(c->dstFormat)
-                  && isBE(c->srcFormat) != isBE(c->dstFormat)){
+            } else if(is16BPS(c->srcFormat) && is16BPS(c->dstFormat)
+                  && isBE(c->srcFormat) != isBE(c->dstFormat)) {
 
-                for (i=0; i<height; i++){
+                for (i=0; i<height; i++) {
                     for (j=0; j<length; j++)
                         ((uint16_t*)dstPtr)[j] = bswap_16(((uint16_t*)srcPtr)[j]);
                     srcPtr+= srcStride[plane];
@@ -2321,12 +2259,10 @@ static int planarCopy(SwsContext *c, uint8_t* src[], int srcStride[], int srcSli
                 }
             } else if (dstStride[plane]==srcStride[plane] && srcStride[plane] > 0)
                 memcpy(dst[plane] + dstStride[plane]*y, src[plane], height*dstStride[plane]);
-            else
-            {
+            else {
                 if(is16BPS(c->srcFormat) && is16BPS(c->dstFormat))
                     length*=2;
-                for (i=0; i<height; i++)
-                {
+                for (i=0; i<height; i++) {
                     memcpy(dstPtr, srcPtr, length);
                     srcPtr+= srcStride[plane];
                     dstPtr+= dstStride[plane];
@@ -2338,64 +2274,22 @@ static int planarCopy(SwsContext *c, uint8_t* src[], int srcStride[], int srcSli
 }
 
 
-static void getSubSampleFactors(int *h, int *v, int format){
-    switch(format){
-    case PIX_FMT_UYVY422:
-    case PIX_FMT_YUYV422:
-        *h=1;
-        *v=0;
-        break;
-    case PIX_FMT_YUV420P:
-    case PIX_FMT_YUV420PLE:
-    case PIX_FMT_YUV420PBE:
-    case PIX_FMT_YUVA420P:
-    case PIX_FMT_GRAY16BE:
-    case PIX_FMT_GRAY16LE:
-    case PIX_FMT_GRAY8: //FIXME remove after different subsamplings are fully implemented
-    case PIX_FMT_NV12:
-    case PIX_FMT_NV21:
-        *h=1;
-        *v=1;
-        break;
-    case PIX_FMT_YUV440P:
-        *h=0;
-        *v=1;
-        break;
-    case PIX_FMT_YUV410P:
-        *h=2;
-        *v=2;
-        break;
-    case PIX_FMT_YUV444P:
-    case PIX_FMT_YUV444PLE:
-    case PIX_FMT_YUV444PBE:
-        *h=0;
-        *v=0;
-        break;
-    case PIX_FMT_YUV422P:
-    case PIX_FMT_YUV422PLE:
-    case PIX_FMT_YUV422PBE:
-        *h=1;
-        *v=0;
-        break;
-    case PIX_FMT_YUV411P:
-        *h=2;
-        *v=0;
-        break;
-    default:
-        *h=0;
-        *v=0;
-        break;
-    }
+static void getSubSampleFactors(int *h, int *v, enum PixelFormat format)
+{
+    *h = av_pix_fmt_descriptors[format].log2_chroma_w;
+    *v = av_pix_fmt_descriptors[format].log2_chroma_h;
 }
 
-static uint16_t roundToInt16(int64_t f){
+static uint16_t roundToInt16(int64_t f)
+{
     int r= (f + (1<<15))>>16;
          if (r<-0x7FFF) return 0x8000;
     else if (r> 0x7FFF) return 0x7FFF;
     else                return r;
 }
 
-int sws_setColorspaceDetails(SwsContext *c, const int inv_table[4], int srcRange, const int table[4], int dstRange, int brightness, int contrast, int saturation){
+int sws_setColorspaceDetails(SwsContext *c, const int inv_table[4], int srcRange, const int table[4], int dstRange, int brightness, int contrast, int saturation)
+{
     int64_t crv =  inv_table[0];
     int64_t cbu =  inv_table[1];
     int64_t cgu = -inv_table[2];
@@ -2416,10 +2310,10 @@ int sws_setColorspaceDetails(SwsContext *c, const int inv_table[4], int srcRange
     c->uOffset=   0x0400040004000400LL;
     c->vOffset=   0x0400040004000400LL;
 
-    if (!srcRange){
+    if (!srcRange) {
         cy= (cy*255) / 219;
         oy= 16<<16;
-    }else{
+    } else {
         crv= (crv*224) / 255;
         cbu= (cbu*224) / 255;
         cgu= (cgu*224) / 255;
@@ -2458,7 +2352,8 @@ int sws_setColorspaceDetails(SwsContext *c, const int inv_table[4], int srcRange
     return 0;
 }
 
-int sws_getColorspaceDetails(SwsContext *c, int **inv_table, int *srcRange, int **table, int *dstRange, int *brightness, int *contrast, int *saturation){
+int sws_getColorspaceDetails(SwsContext *c, int **inv_table, int *srcRange, int **table, int *dstRange, int *brightness, int *contrast, int *saturation)
+{
     if (isYUV(c->dstFormat) || isGray(c->dstFormat)) return -1;
 
     *inv_table = c->srcColorspaceTable;
@@ -2475,20 +2370,20 @@ int sws_getColorspaceDetails(SwsContext *c, int **inv_table, int *srcRange, int
 static int handle_jpeg(enum PixelFormat *format)
 {
     switch (*format) {
-        case PIX_FMT_YUVJ420P:
-            *format = PIX_FMT_YUV420P;
-            return 1;
-        case PIX_FMT_YUVJ422P:
-            *format = PIX_FMT_YUV422P;
-            return 1;
-        case PIX_FMT_YUVJ444P:
-            *format = PIX_FMT_YUV444P;
-            return 1;
-        case PIX_FMT_YUVJ440P:
-            *format = PIX_FMT_YUV440P;
-            return 1;
-        default:
-            return 0;
+    case PIX_FMT_YUVJ420P:
+        *format = PIX_FMT_YUV420P;
+        return 1;
+    case PIX_FMT_YUVJ422P:
+        *format = PIX_FMT_YUV422P;
+        return 1;
+    case PIX_FMT_YUVJ444P:
+        *format = PIX_FMT_YUV444P;
+        return 1;
+    case PIX_FMT_YUVJ440P:
+        *format = PIX_FMT_YUV440P;
+        return 1;
+    default:
+        return 0;
     }
 }
 
@@ -2521,7 +2416,6 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
     flags |= SWS_CPU_CAPS_BFIN;
 #endif
 #endif /* CONFIG_RUNTIME_CPUDETECT */
-    if (clip_table[512] != 255) globalInit();
     if (!rgb15to16) sws_rgb2rgb_init(flags);
 
     unscaled = (srcW == dstW && srcH == dstH);
@@ -2532,13 +2426,11 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
     srcRange = handle_jpeg(&srcFormat);
     dstRange = handle_jpeg(&dstFormat);
 
-    if (!isSupportedIn(srcFormat))
-    {
+    if (!isSupportedIn(srcFormat)) {
         av_log(NULL, AV_LOG_ERROR, "swScaler: %s is not supported as input pixel format\n", sws_format_name(srcFormat));
         return NULL;
     }
-    if (!isSupportedOut(dstFormat))
-    {
+    if (!isSupportedOut(dstFormat)) {
         av_log(NULL, AV_LOG_ERROR, "swScaler: %s is not supported as output pixel format\n", sws_format_name(dstFormat));
         return NULL;
     }
@@ -2554,20 +2446,18 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
                 |SWS_SINC
                 |SWS_SPLINE
                 |SWS_BICUBLIN);
-    if(!i || (i & (i-1)))
-    {
+    if(!i || (i & (i-1))) {
         av_log(NULL, AV_LOG_ERROR, "swScaler: Exactly one scaler algorithm must be chosen\n");
         return NULL;
     }
 
     /* sanity check */
-    if (srcW<4 || srcH<1 || dstW<8 || dstH<1) //FIXME check if these are enough and try to lowwer them after fixing the relevant parts of the code
-    {
+    if (srcW<4 || srcH<1 || dstW<8 || dstH<1) { //FIXME check if these are enough and try to lowwer them after fixing the relevant parts of the code
         av_log(NULL, AV_LOG_ERROR, "swScaler: %dx%d -> %dx%d is invalid scaling dimension\n",
                srcW, srcH, dstW, dstH);
         return NULL;
     }
-    if(srcW > VOFW || dstW > VOFW){
+    if(srcW > VOFW || dstW > VOFW) {
         av_log(NULL, AV_LOG_ERROR, "swScaler: Compile-time maximum width is "AV_STRINGIFY(VOFW)" change VOF/VOFW and recompile\n");
         return NULL;
     }
@@ -2575,7 +2465,7 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
     if (!dstFilter) dstFilter= &dummyFilter;
     if (!srcFilter) srcFilter= &dummyFilter;
 
-    c= av_mallocz(sizeof(SwsContext));
+    FF_ALLOCZ_OR_GOTO(NULL, c, sizeof(SwsContext), fail);
 
     c->av_class = &sws_context_class;
     c->srcW= srcW;
@@ -2617,10 +2507,10 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
       && ((dstW>>c->chrDstHSubSample) <= (srcW>>1) || (flags&(SWS_FAST_BILINEAR|SWS_POINT))))
         c->chrSrcHSubSample=1;
 
-    if (param){
+    if (param) {
         c->param[0] = param[0];
         c->param[1] = param[1];
-    }else{
+    } else {
         c->param[0] =
         c->param[1] = SWS_PARAM_DEFAULT;
     }
@@ -2634,22 +2524,18 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
     sws_setColorspaceDetails(c, ff_yuv2rgb_coeffs[SWS_CS_DEFAULT], srcRange, ff_yuv2rgb_coeffs[SWS_CS_DEFAULT] /* FIXME*/, dstRange, 0, 1<<16, 1<<16);
 
     /* unscaled special cases */
-    if (unscaled && !usesHFilter && !usesVFilter && (srcRange == dstRange || isBGR(dstFormat) || isRGB(dstFormat)))
-    {
+    if (unscaled && !usesHFilter && !usesVFilter && (srcRange == dstRange || isBGR(dstFormat) || isRGB(dstFormat))) {
         /* yv12_to_nv12 */
-        if ((srcFormat == PIX_FMT_YUV420P || srcFormat == PIX_FMT_YUVA420P) && (dstFormat == PIX_FMT_NV12 || dstFormat == PIX_FMT_NV21))
-        {
+        if ((srcFormat == PIX_FMT_YUV420P || srcFormat == PIX_FMT_YUVA420P) && (dstFormat == PIX_FMT_NV12 || dstFormat == PIX_FMT_NV21)) {
             c->swScale= PlanarToNV12Wrapper;
         }
         /* yuv2bgr */
         if ((srcFormat==PIX_FMT_YUV420P || srcFormat==PIX_FMT_YUV422P || srcFormat==PIX_FMT_YUVA420P) && (isBGR(dstFormat) || isRGB(dstFormat))
-            && !(flags & SWS_ACCURATE_RND) && !(dstH&1))
-        {
+            && !(flags & SWS_ACCURATE_RND) && !(dstH&1)) {
             c->swScale= ff_yuv2rgb_get_func_ptr(c);
         }
 
-        if (srcFormat==PIX_FMT_YUV410P && (dstFormat==PIX_FMT_YUV420P || dstFormat==PIX_FMT_YUVA420P) && !(flags & SWS_BITEXACT))
-        {
+        if (srcFormat==PIX_FMT_YUV410P && (dstFormat==PIX_FMT_YUV420P || dstFormat==PIX_FMT_YUVA420P) && !(flags & SWS_BITEXACT)) {
             c->swScale= yvu9toyv12Wrapper;
         }
 
@@ -2684,8 +2570,7 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
                  dstFormat == PIX_FMT_BGR24)))
              c->swScale= pal2rgbWrapper;
 
-        if (srcFormat == PIX_FMT_YUV422P)
-        {
+        if (srcFormat == PIX_FMT_YUV422P) {
             if (dstFormat == PIX_FMT_YUYV422)
                 c->swScale= YUV422PToYuy2Wrapper;
             else if (dstFormat == PIX_FMT_UYVY422)
@@ -2693,10 +2578,9 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
         }
 
         /* LQ converters if -sws 0 or -sws 4*/
-        if (c->flags&(SWS_FAST_BILINEAR|SWS_POINT)){
+        if (c->flags&(SWS_FAST_BILINEAR|SWS_POINT)) {
             /* yv12_to_yuy2 */
-            if (srcFormat == PIX_FMT_YUV420P || srcFormat == PIX_FMT_YUVA420P)
-            {
+            if (srcFormat == PIX_FMT_YUV420P || srcFormat == PIX_FMT_YUVA420P) {
                 if (dstFormat == PIX_FMT_YUYV422)
                     c->swScale= PlanarToYuy2Wrapper;
                 else if (dstFormat == PIX_FMT_UYVY422)
@@ -2747,19 +2631,17 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
             ff_bfin_get_unscaled_swscale (c);
 #endif
 
-        if (c->swScale){
+        if (c->swScale) {
             if (flags&SWS_PRINT_INFO)
                 av_log(c, AV_LOG_INFO, "using unscaled %s -> %s special converter\n",
-                                sws_format_name(srcFormat), sws_format_name(dstFormat));
+                       sws_format_name(srcFormat), sws_format_name(dstFormat));
             return c;
         }
     }
 
-    if (flags & SWS_CPU_CAPS_MMX2)
-    {
+    if (flags & SWS_CPU_CAPS_MMX2) {
         c->canMMX2BeUsed= (dstW >=srcW && (dstW&31)==0 && (srcW&15)==0) ? 1 : 0;
-        if (!c->canMMX2BeUsed && dstW >=srcW && (srcW&15)==0 && (flags&SWS_FAST_BILINEAR))
-        {
+        if (!c->canMMX2BeUsed && dstW >=srcW && (srcW&15)==0 && (flags&SWS_FAST_BILINEAR)) {
             if (flags&SWS_PRINT_INFO)
                 av_log(c, AV_LOG_INFO, "output width is not a multiple of 32 -> no MMX2 scaler\n");
         }
@@ -2777,16 +2659,13 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
     // this is not perfect, but no one should notice the difference, the more correct variant
     // would be like the vertical one, but that would require some special code for the
     // first and last pixel
-    if (flags&SWS_FAST_BILINEAR)
-    {
-        if (c->canMMX2BeUsed)
-        {
+    if (flags&SWS_FAST_BILINEAR) {
+        if (c->canMMX2BeUsed) {
             c->lumXInc+= 20;
             c->chrXInc+= 20;
         }
         //we don't use the x86 asm scaler if MMX is available
-        else if (flags & SWS_CPU_CAPS_MMX)
-        {
+        else if (flags & SWS_CPU_CAPS_MMX) {
             c->lumXInc = ((srcW-2)<<16)/(dstW-2) - 20;
             c->chrXInc = ((c->chrSrcW-2)<<16)/(c->chrDstW-2) - 20;
         }
@@ -2799,42 +2678,45 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
             (flags & SWS_CPU_CAPS_ALTIVEC) ? 8 :
             1;
 
-        initFilter(&c->hLumFilter, &c->hLumFilterPos, &c->hLumFilterSize, c->lumXInc,
-                   srcW      ,       dstW, filterAlign, 1<<14,
-                   (flags&SWS_BICUBLIN) ? (flags|SWS_BICUBIC)  : flags,
-                   srcFilter->lumH, dstFilter->lumH, c->param);
-        initFilter(&c->hChrFilter, &c->hChrFilterPos, &c->hChrFilterSize, c->chrXInc,
-                   c->chrSrcW, c->chrDstW, filterAlign, 1<<14,
-                   (flags&SWS_BICUBLIN) ? (flags|SWS_BILINEAR) : flags,
-                   srcFilter->chrH, dstFilter->chrH, c->param);
+        if (initFilter(&c->hLumFilter, &c->hLumFilterPos, &c->hLumFilterSize, c->lumXInc,
+                       srcW      ,       dstW, filterAlign, 1<<14,
+                       (flags&SWS_BICUBLIN) ? (flags|SWS_BICUBIC)  : flags,
+                       srcFilter->lumH, dstFilter->lumH, c->param) < 0)
+            goto fail;
+        if (initFilter(&c->hChrFilter, &c->hChrFilterPos, &c->hChrFilterSize, c->chrXInc,
+                       c->chrSrcW, c->chrDstW, filterAlign, 1<<14,
+                       (flags&SWS_BICUBLIN) ? (flags|SWS_BILINEAR) : flags,
+                       srcFilter->chrH, dstFilter->chrH, c->param) < 0)
+            goto fail;
 
-#define MAX_MMX2_FILTER_CODE_SIZE 10000
 #if defined(COMPILE_MMX2)
 // can't downscale !!!
-        if (c->canMMX2BeUsed && (flags & SWS_FAST_BILINEAR))
-        {
+        if (c->canMMX2BeUsed && (flags & SWS_FAST_BILINEAR)) {
+            c->lumMmx2FilterCodeSize = initMMX2HScaler(      dstW, c->lumXInc, NULL, NULL, NULL, 8);
+            c->chrMmx2FilterCodeSize = initMMX2HScaler(c->chrDstW, c->chrXInc, NULL, NULL, NULL, 4);
+
 #ifdef MAP_ANONYMOUS
-            c->lumMmx2FilterCode = mmap(NULL, MAX_MMX2_FILTER_CODE_SIZE, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, 0, 0);
-            c->chrMmx2FilterCode = mmap(NULL, MAX_MMX2_FILTER_CODE_SIZE, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, 0, 0);
+            c->lumMmx2FilterCode = mmap(NULL, c->lumMmx2FilterCodeSize, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, 0, 0);
+            c->chrMmx2FilterCode = mmap(NULL, c->chrMmx2FilterCodeSize, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, 0, 0);
 #elif HAVE_VIRTUALALLOC
-            c->lumMmx2FilterCode = VirtualAlloc(NULL, MAX_MMX2_FILTER_CODE_SIZE, MEM_COMMIT, PAGE_EXECUTE_READWRITE);
-            c->chrMmx2FilterCode = VirtualAlloc(NULL, MAX_MMX2_FILTER_CODE_SIZE, MEM_COMMIT, PAGE_EXECUTE_READWRITE);
+            c->lumMmx2FilterCode = VirtualAlloc(NULL, c->lumMmx2FilterCodeSize, MEM_COMMIT, PAGE_EXECUTE_READWRITE);
+            c->chrMmx2FilterCode = VirtualAlloc(NULL, c->chrMmx2FilterCodeSize, MEM_COMMIT, PAGE_EXECUTE_READWRITE);
 #else
-            c->lumMmx2FilterCode = av_malloc(MAX_MMX2_FILTER_CODE_SIZE);
-            c->chrMmx2FilterCode = av_malloc(MAX_MMX2_FILTER_CODE_SIZE);
+            c->lumMmx2FilterCode = av_malloc(c->lumMmx2FilterCodeSize);
+            c->chrMmx2FilterCode = av_malloc(c->chrMmx2FilterCodeSize);
 #endif
 
-            c->lumMmx2Filter   = av_malloc((dstW        /8+8)*sizeof(int16_t));
-            c->chrMmx2Filter   = av_malloc((c->chrDstW  /4+8)*sizeof(int16_t));
-            c->lumMmx2FilterPos= av_malloc((dstW      /2/8+8)*sizeof(int32_t));
-            c->chrMmx2FilterPos= av_malloc((c->chrDstW/2/4+8)*sizeof(int32_t));
+            FF_ALLOCZ_OR_GOTO(c, c->lumMmx2Filter   , (dstW        /8+8)*sizeof(int16_t), fail);
+            FF_ALLOCZ_OR_GOTO(c, c->chrMmx2Filter   , (c->chrDstW  /4+8)*sizeof(int16_t), fail);
+            FF_ALLOCZ_OR_GOTO(c, c->lumMmx2FilterPos, (dstW      /2/8+8)*sizeof(int32_t), fail);
+            FF_ALLOCZ_OR_GOTO(c, c->chrMmx2FilterPos, (c->chrDstW/2/4+8)*sizeof(int32_t), fail);
 
             initMMX2HScaler(      dstW, c->lumXInc, c->lumMmx2FilterCode, c->lumMmx2Filter, c->lumMmx2FilterPos, 8);
             initMMX2HScaler(c->chrDstW, c->chrXInc, c->chrMmx2FilterCode, c->chrMmx2Filter, c->chrMmx2FilterPos, 4);
 
 #ifdef MAP_ANONYMOUS
-            mprotect(c->lumMmx2FilterCode, MAX_MMX2_FILTER_CODE_SIZE, PROT_EXEC | PROT_READ);
-            mprotect(c->chrMmx2FilterCode, MAX_MMX2_FILTER_CODE_SIZE, PROT_EXEC | PROT_READ);
+            mprotect(c->lumMmx2FilterCode, c->lumMmx2FilterCodeSize, PROT_EXEC | PROT_READ);
+            mprotect(c->chrMmx2FilterCode, c->chrMmx2FilterCodeSize, PROT_EXEC | PROT_READ);
 #endif
         }
 #endif /* defined(COMPILE_MMX2) */
@@ -2849,18 +2731,20 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
             (flags & SWS_CPU_CAPS_ALTIVEC) ? 8 :
             1;
 
-        initFilter(&c->vLumFilter, &c->vLumFilterPos, &c->vLumFilterSize, c->lumYInc,
-                   srcH      ,        dstH, filterAlign, (1<<12),
-                   (flags&SWS_BICUBLIN) ? (flags|SWS_BICUBIC)  : flags,
-                   srcFilter->lumV, dstFilter->lumV, c->param);
-        initFilter(&c->vChrFilter, &c->vChrFilterPos, &c->vChrFilterSize, c->chrYInc,
-                   c->chrSrcH, c->chrDstH, filterAlign, (1<<12),
-                   (flags&SWS_BICUBLIN) ? (flags|SWS_BILINEAR) : flags,
-                   srcFilter->chrV, dstFilter->chrV, c->param);
+        if (initFilter(&c->vLumFilter, &c->vLumFilterPos, &c->vLumFilterSize, c->lumYInc,
+                       srcH      ,        dstH, filterAlign, (1<<12),
+                       (flags&SWS_BICUBLIN) ? (flags|SWS_BICUBIC)  : flags,
+                       srcFilter->lumV, dstFilter->lumV, c->param) < 0)
+            goto fail;
+        if (initFilter(&c->vChrFilter, &c->vChrFilterPos, &c->vChrFilterSize, c->chrYInc,
+                       c->chrSrcH, c->chrDstH, filterAlign, (1<<12),
+                       (flags&SWS_BICUBLIN) ? (flags|SWS_BILINEAR) : flags,
+                       srcFilter->chrV, dstFilter->chrV, c->param) < 0)
+            goto fail;
 
 #ifdef COMPILE_ALTIVEC
-        c->vYCoeffsBank = av_malloc(sizeof (vector signed short)*c->vLumFilterSize*c->dstH);
-        c->vCCoeffsBank = av_malloc(sizeof (vector signed short)*c->vChrFilterSize*c->chrDstH);
+        FF_ALLOC_OR_GOTO(c, c->vYCoeffsBank, sizeof (vector signed short)*c->vLumFilterSize*c->dstH, fail);
+        FF_ALLOC_OR_GOTO(c, c->vCCoeffsBank, sizeof (vector signed short)*c->vChrFilterSize*c->chrDstH, fail);
 
         for (i=0;i<c->vLumFilterSize*c->dstH;i++) {
             int j;
@@ -2881,8 +2765,7 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
     // calculate buffer sizes so that they won't run out while handling these damn slices
     c->vLumBufSize= c->vLumFilterSize;
     c->vChrBufSize= c->vChrFilterSize;
-    for (i=0; i<dstH; i++)
-    {
+    for (i=0; i<dstH; i++) {
         int chrI= i*c->chrDstH / dstH;
         int nextSlice= FFMAX(c->vLumFilterPos[i   ] + c->vLumFilterSize - 1,
                            ((c->vChrFilterPos[chrI] + c->vChrFilterSize - 1)<<c->chrSrcVSubSample));
@@ -2896,19 +2779,26 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
     }
 
     // allocate pixbufs (we use dynamic allocation because otherwise we would need to
-    c->lumPixBuf= av_malloc(c->vLumBufSize*2*sizeof(int16_t*));
-    c->chrPixBuf= av_malloc(c->vChrBufSize*2*sizeof(int16_t*));
+    // allocate several megabytes to handle all possible cases)
+    FF_ALLOC_OR_GOTO(c, c->lumPixBuf, c->vLumBufSize*2*sizeof(int16_t*), fail);
+    FF_ALLOC_OR_GOTO(c, c->chrPixBuf, c->vChrBufSize*2*sizeof(int16_t*), fail);
     if (CONFIG_SWSCALE_ALPHA && isALPHA(c->srcFormat) && isALPHA(c->dstFormat))
-        c->alpPixBuf= av_malloc(c->vLumBufSize*2*sizeof(int16_t*));
+        FF_ALLOCZ_OR_GOTO(c, c->alpPixBuf, c->vLumBufSize*2*sizeof(int16_t*), fail);
     //Note we need at least one pixel more at the end because of the MMX code (just in case someone wanna replace the 4000/8000)
     /* align at 16 bytes for AltiVec */
-    for (i=0; i<c->vLumBufSize; i++)
-        c->lumPixBuf[i]= c->lumPixBuf[i+c->vLumBufSize]= av_mallocz(VOF+1);
-    for (i=0; i<c->vChrBufSize; i++)
-        c->chrPixBuf[i]= c->chrPixBuf[i+c->vChrBufSize]= av_malloc((VOF+1)*2);
+    for (i=0; i<c->vLumBufSize; i++) {
+        FF_ALLOCZ_OR_GOTO(c, c->lumPixBuf[i+c->vLumBufSize], VOF+1, fail);
+        c->lumPixBuf[i] = c->lumPixBuf[i+c->vLumBufSize];
+    }
+    for (i=0; i<c->vChrBufSize; i++) {
+        FF_ALLOC_OR_GOTO(c, c->chrPixBuf[i+c->vChrBufSize], (VOF+1)*2, fail);
+        c->chrPixBuf[i] = c->chrPixBuf[i+c->vChrBufSize];
+    }
     if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf)
-        for (i=0; i<c->vLumBufSize; i++)
-            c->alpPixBuf[i]= c->alpPixBuf[i+c->vLumBufSize]= av_mallocz(VOF+1);
+        for (i=0; i<c->vLumBufSize; i++) {
+            FF_ALLOCZ_OR_GOTO(c, c->alpPixBuf[i+c->vLumBufSize], VOF+1, fail);
+            c->alpPixBuf[i] = c->alpPixBuf[i+c->vLumBufSize];
+        }
 
     //try to avoid drawing green stuff between the right end and the stride end
     for (i=0; i<c->vChrBufSize; i++) memset(c->chrPixBuf[i], 64, (VOF+1)*2);
@@ -2917,8 +2807,7 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
 
     assert(c->chrDstH <= dstH);
 
-    if (flags&SWS_PRINT_INFO)
-    {
+    if (flags&SWS_PRINT_INFO) {
 #ifdef DITHER1XBPP
         const char *dither= " dithered";
 #else
@@ -2935,7 +2824,7 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
         else if (flags&SWS_POINT)
             av_log(c, AV_LOG_INFO, "Nearest Neighbor / POINT scaler, ");
         else if (flags&SWS_AREA)
-            av_log(c, AV_LOG_INFO, "Area Averageing scaler, ");
+            av_log(c, AV_LOG_INFO, "Area Averaging scaler, ");
         else if (flags&SWS_BICUBLIN)
             av_log(c, AV_LOG_INFO, "luma BICUBIC / chroma BILINEAR scaler, ");
         else if (flags&SWS_GAUSS)
@@ -2968,14 +2857,11 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
             av_log(c, AV_LOG_INFO, "using C\n");
     }
 
-    if (flags & SWS_PRINT_INFO)
-    {
-        if (flags & SWS_CPU_CAPS_MMX)
-        {
+    if (flags & SWS_PRINT_INFO) {
+        if (flags & SWS_CPU_CAPS_MMX) {
             if (c->canMMX2BeUsed && (flags&SWS_FAST_BILINEAR))
                 av_log(c, AV_LOG_VERBOSE, "using FAST_BILINEAR MMX2 scaler for horizontal scaling\n");
-            else
-            {
+            else {
                 if (c->hLumFilterSize==4)
                     av_log(c, AV_LOG_VERBOSE, "using 4-tap MMX scaler for horizontal luminance scaling\n");
                 else if (c->hLumFilterSize==8)
@@ -2990,9 +2876,7 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
                 else
                     av_log(c, AV_LOG_VERBOSE, "using n-tap MMX scaler for horizontal chrominance scaling\n");
             }
-        }
-        else
-        {
+        } else {
 #if ARCH_X86
             av_log(c, AV_LOG_VERBOSE, "using x86 asm scaler for horizontal scaling\n");
 #else
@@ -3002,15 +2886,12 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
                 av_log(c, AV_LOG_VERBOSE, "using C scaler for horizontal scaling\n");
 #endif
         }
-        if (isPlanarYUV(dstFormat))
-        {
+        if (isPlanarYUV(dstFormat)) {
             if (c->vLumFilterSize==1)
                 av_log(c, AV_LOG_VERBOSE, "using 1-tap %s \"scaler\" for vertical scaling (YV12 like)\n", (flags & SWS_CPU_CAPS_MMX) ? "MMX" : "C");
             else
                 av_log(c, AV_LOG_VERBOSE, "using n-tap %s scaler for vertical scaling (YV12 like)\n", (flags & SWS_CPU_CAPS_MMX) ? "MMX" : "C");
-        }
-        else
-        {
+        } else {
             if (c->vLumFilterSize==1 && c->vChrFilterSize==2)
                 av_log(c, AV_LOG_VERBOSE, "using 1-tap %s \"scaler\" for vertical luminance scaling (BGR)\n"
                        "      2-tap scaler for vertical chrominance scaling (BGR)\n", (flags & SWS_CPU_CAPS_MMX) ? "MMX" : "C");
@@ -3032,8 +2913,7 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
 
         av_log(c, AV_LOG_VERBOSE, "%dx%d -> %dx%d\n", srcW, srcH, dstW, dstH);
     }
-    if (flags & SWS_PRINT_INFO)
-    {
+    if (flags & SWS_PRINT_INFO) {
         av_log(c, AV_LOG_DEBUG, "lum srcW=%d srcH=%d dstW=%d dstH=%d xInc=%d yInc=%d\n",
                c->srcW, c->srcH, c->dstW, c->dstH, c->lumXInc, c->lumYInc);
         av_log(c, AV_LOG_DEBUG, "chr srcW=%d srcH=%d dstW=%d dstH=%d xInc=%d yInc=%d\n",
@@ -3042,12 +2922,17 @@ SwsContext *sws_getContext(int srcW, int srcH, enum PixelFormat srcFormat, int d
 
     c->swScale= getSwsFunc(c);
     return c;
+
+fail:
+    sws_freeContext(c);
+    return NULL;
 }
 
-static void reset_ptr(uint8_t* src[], int format){
+static void reset_ptr(const uint8_t* src[], int format)
+{
     if(!isALPHA(format))
         src[3]=NULL;
-    if(!isPlanarYUV(format)){
+    if(!isPlanarYUV(format)) {
         src[3]=src[2]=NULL;
         if(   format != PIX_FMT_PAL8
            && format != PIX_FMT_RGB8
@@ -3063,12 +2948,17 @@ static void reset_ptr(uint8_t* src[], int format){
  * swscale wrapper, so we don't need to export the SwsContext.
  * Assumes planar YUV to be in YUV order instead of YVU.
  */
-int sws_scale(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-              int srcSliceH, uint8_t* dst[], int dstStride[]){
+int sws_scale(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+              int srcSliceH, uint8_t* dst[], int dstStride[])
+{
     int i;
-    uint8_t* src2[4]= {src[0], src[1], src[2], src[3]};
+    const uint8_t* src2[4]= {src[0], src[1], src[2], src[3]};
     uint8_t* dst2[4]= {dst[0], dst[1], dst[2], dst[3]};
 
+    // do not mess up sliceDir if we have a "trailing" 0-size slice
+    if (srcSliceH == 0)
+        return 0;
+
     if (c->sliceDir == 0 && srcSliceY != 0 && srcSliceY + srcSliceH != c->srcH) {
         av_log(c, AV_LOG_ERROR, "Slices start in the middle!\n");
         return 0;
@@ -3077,27 +2967,27 @@ int sws_scale(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
         if (srcSliceY == 0) c->sliceDir = 1; else c->sliceDir = -1;
     }
 
-    if (usePal(c->srcFormat)){
-        for (i=0; i<256; i++){
+    if (usePal(c->srcFormat)) {
+        for (i=0; i<256; i++) {
             int p, r, g, b,y,u,v;
-            if(c->srcFormat == PIX_FMT_PAL8){
+            if(c->srcFormat == PIX_FMT_PAL8) {
                 p=((uint32_t*)(src[1]))[i];
                 r= (p>>16)&0xFF;
                 g= (p>> 8)&0xFF;
                 b=  p     &0xFF;
-            }else if(c->srcFormat == PIX_FMT_RGB8){
+            } else if(c->srcFormat == PIX_FMT_RGB8) {
                 r= (i>>5    )*36;
                 g= ((i>>2)&7)*36;
                 b= (i&3     )*85;
-            }else if(c->srcFormat == PIX_FMT_BGR8){
+            } else if(c->srcFormat == PIX_FMT_BGR8) {
                 b= (i>>6    )*85;
                 g= ((i>>3)&7)*36;
                 r= (i&7     )*36;
-            }else if(c->srcFormat == PIX_FMT_RGB4_BYTE){
+            } else if(c->srcFormat == PIX_FMT_RGB4_BYTE) {
                 r= (i>>3    )*255;
                 g= ((i>>1)&3)*85;
                 b= (i&1     )*255;
-            }else {
+            } else {
                 assert(c->srcFormat == PIX_FMT_BGR4_BYTE);
                 b= (i>>3    )*255;
                 g= ((i>>1)&3)*85;
@@ -3145,7 +3035,11 @@ int sws_scale(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
         int dstStride2[4]= {dstStride[0], dstStride[1], dstStride[2], dstStride[3]};
 
         reset_ptr(src2, c->srcFormat);
-        reset_ptr(dst2, c->dstFormat);
+        reset_ptr((const uint8_t**)dst2, c->dstFormat);
+
+        /* reset slice direction at end of frame */
+        if (srcSliceY + srcSliceH == c->srcH)
+            c->sliceDir = 0;
 
         return c->swScale(c, src2, srcStride2, srcSliceY, srcSliceH, dst2, dstStride2);
     } else {
@@ -3164,15 +3058,20 @@ int sws_scale(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
         dst2[3] += ( c->dstH                      -1)*dstStride[3];
 
         reset_ptr(src2, c->srcFormat);
-        reset_ptr(dst2, c->dstFormat);
+        reset_ptr((const uint8_t**)dst2, c->dstFormat);
+
+        /* reset slice direction at end of frame */
+        if (!srcSliceY)
+            c->sliceDir = 0;
 
         return c->swScale(c, src2, srcStride2, c->srcH-srcSliceY-srcSliceH, srcSliceH, dst2, dstStride2);
     }
 }
 
 #if LIBSWSCALE_VERSION_MAJOR < 1
-int sws_scale_ordered(SwsContext *c, uint8_t* src[], int srcStride[], int srcSliceY,
-                      int srcSliceH, uint8_t* dst[], int dstStride[]){
+int sws_scale_ordered(SwsContext *c, const uint8_t* src[], int srcStride[], int srcSliceY,
+                      int srcSliceH, uint8_t* dst[], int dstStride[])
+{
     return sws_scale(c, src, srcStride, srcSliceY, srcSliceH, dst, dstStride);
 }
 #endif
@@ -3183,24 +3082,26 @@ SwsFilter *sws_getDefaultFilter(float lumaGBlur, float chromaGBlur,
                                 int verbose)
 {
     SwsFilter *filter= av_malloc(sizeof(SwsFilter));
+    if (!filter)
+        return NULL;
 
-    if (lumaGBlur!=0.0){
+    if (lumaGBlur!=0.0) {
         filter->lumH= sws_getGaussianVec(lumaGBlur, 3.0);
         filter->lumV= sws_getGaussianVec(lumaGBlur, 3.0);
-    }else{
+    } else {
         filter->lumH= sws_getIdentityVec();
         filter->lumV= sws_getIdentityVec();
     }
 
-    if (chromaGBlur!=0.0){
+    if (chromaGBlur!=0.0) {
         filter->chrH= sws_getGaussianVec(chromaGBlur, 3.0);
         filter->chrV= sws_getGaussianVec(chromaGBlur, 3.0);
-    }else{
+    } else {
         filter->chrH= sws_getIdentityVec();
         filter->chrV= sws_getIdentityVec();
     }
 
-    if (chromaSharpen!=0.0){
+    if (chromaSharpen!=0.0) {
         SwsVector *id= sws_getIdentityVec();
         sws_scaleVec(filter->chrH, -chromaSharpen);
         sws_scaleVec(filter->chrV, -chromaSharpen);
@@ -3209,7 +3110,7 @@ SwsFilter *sws_getDefaultFilter(float lumaGBlur, float chromaGBlur,
         sws_freeVec(id);
     }
 
-    if (lumaSharpen!=0.0){
+    if (lumaSharpen!=0.0) {
         SwsVector *id= sws_getIdentityVec();
         sws_scaleVec(filter->lumH, -lumaSharpen);
         sws_scaleVec(filter->lumV, -lumaSharpen);
@@ -3235,20 +3136,31 @@ SwsFilter *sws_getDefaultFilter(float lumaGBlur, float chromaGBlur,
     return filter;
 }
 
-SwsVector *sws_getGaussianVec(double variance, double quality){
+SwsVector *sws_allocVec(int length)
+{
+    SwsVector *vec = av_malloc(sizeof(SwsVector));
+    if (!vec)
+        return NULL;
+    vec->length = length;
+    vec->coeff  = av_malloc(sizeof(double) * length);
+    if (!vec->coeff)
+        av_freep(&vec);
+    return vec;
+}
+
+SwsVector *sws_getGaussianVec(double variance, double quality)
+{
     const int length= (int)(variance*quality + 0.5) | 1;
     int i;
-    double *coeff= av_malloc(length*sizeof(double));
     double middle= (length-1)*0.5;
-    SwsVector *vec= av_malloc(sizeof(SwsVector));
+    SwsVector *vec= sws_allocVec(length);
 
-    vec->coeff= coeff;
-    vec->length= length;
+    if (!vec)
+        return NULL;
 
-    for (i=0; i<length; i++)
-    {
+    for (i=0; i<length; i++) {
         double dist= i-middle;
-        coeff[i]= exp(-dist*dist/(2*variance*variance)) / sqrt(2*variance*PI);
+        vec->coeff[i]= exp(-dist*dist/(2*variance*variance)) / sqrt(2*variance*PI);
     }
 
     sws_normalizeVec(vec, 1.0);
@@ -3256,26 +3168,28 @@ SwsVector *sws_getGaussianVec(double variance, double quality){
     return vec;
 }
 
-SwsVector *sws_getConstVec(double c, int length){
+SwsVector *sws_getConstVec(double c, int length)
+{
     int i;
-    double *coeff= av_malloc(length*sizeof(double));
-    SwsVector *vec= av_malloc(sizeof(SwsVector));
+    SwsVector *vec= sws_allocVec(length);
 
-    vec->coeff= coeff;
-    vec->length= length;
+    if (!vec)
+        return NULL;
 
     for (i=0; i<length; i++)
-        coeff[i]= c;
+        vec->coeff[i]= c;
 
     return vec;
 }
 
 
-SwsVector *sws_getIdentityVec(void){
+SwsVector *sws_getIdentityVec(void)
+{
     return sws_getConstVec(1.0, 1);
 }
 
-double sws_dcVec(SwsVector *a){
+double sws_dcVec(SwsVector *a)
+{
     int i;
     double sum=0;
 
@@ -3285,94 +3199,86 @@ double sws_dcVec(SwsVector *a){
     return sum;
 }
 
-void sws_scaleVec(SwsVector *a, double scalar){
+void sws_scaleVec(SwsVector *a, double scalar)
+{
     int i;
 
     for (i=0; i<a->length; i++)
         a->coeff[i]*= scalar;
 }
 
-void sws_normalizeVec(SwsVector *a, double height){
+void sws_normalizeVec(SwsVector *a, double height)
+{
     sws_scaleVec(a, height/sws_dcVec(a));
 }
 
-static SwsVector *sws_getConvVec(SwsVector *a, SwsVector *b){
+static SwsVector *sws_getConvVec(SwsVector *a, SwsVector *b)
+{
     int length= a->length + b->length - 1;
-    double *coeff= av_malloc(length*sizeof(double));
     int i, j;
-    SwsVector *vec= av_malloc(sizeof(SwsVector));
-
-    vec->coeff= coeff;
-    vec->length= length;
+    SwsVector *vec= sws_getConstVec(0.0, length);
 
-    for (i=0; i<length; i++) coeff[i]= 0.0;
+    if (!vec)
+        return NULL;
 
-    for (i=0; i<a->length; i++)
-    {
-        for (j=0; j<b->length; j++)
-        {
-            coeff[i+j]+= a->coeff[i]*b->coeff[j];
+    for (i=0; i<a->length; i++) {
+        for (j=0; j<b->length; j++) {
+            vec->coeff[i+j]+= a->coeff[i]*b->coeff[j];
         }
     }
 
     return vec;
 }
 
-static SwsVector *sws_sumVec(SwsVector *a, SwsVector *b){
+static SwsVector *sws_sumVec(SwsVector *a, SwsVector *b)
+{
     int length= FFMAX(a->length, b->length);
-    double *coeff= av_malloc(length*sizeof(double));
     int i;
-    SwsVector *vec= av_malloc(sizeof(SwsVector));
-
-    vec->coeff= coeff;
-    vec->length= length;
+    SwsVector *vec= sws_getConstVec(0.0, length);
 
-    for (i=0; i<length; i++) coeff[i]= 0.0;
+    if (!vec)
+        return NULL;
 
-    for (i=0; i<a->length; i++) coeff[i + (length-1)/2 - (a->length-1)/2]+= a->coeff[i];
-    for (i=0; i<b->length; i++) coeff[i + (length-1)/2 - (b->length-1)/2]+= b->coeff[i];
+    for (i=0; i<a->length; i++) vec->coeff[i + (length-1)/2 - (a->length-1)/2]+= a->coeff[i];
+    for (i=0; i<b->length; i++) vec->coeff[i + (length-1)/2 - (b->length-1)/2]+= b->coeff[i];
 
     return vec;
 }
 
-static SwsVector *sws_diffVec(SwsVector *a, SwsVector *b){
+static SwsVector *sws_diffVec(SwsVector *a, SwsVector *b)
+{
     int length= FFMAX(a->length, b->length);
-    double *coeff= av_malloc(length*sizeof(double));
     int i;
-    SwsVector *vec= av_malloc(sizeof(SwsVector));
+    SwsVector *vec= sws_getConstVec(0.0, length);
 
-    vec->coeff= coeff;
-    vec->length= length;
-
-    for (i=0; i<length; i++) coeff[i]= 0.0;
+    if (!vec)
+        return NULL;
 
-    for (i=0; i<a->length; i++) coeff[i + (length-1)/2 - (a->length-1)/2]+= a->coeff[i];
-    for (i=0; i<b->length; i++) coeff[i + (length-1)/2 - (b->length-1)/2]-= b->coeff[i];
+    for (i=0; i<a->length; i++) vec->coeff[i + (length-1)/2 - (a->length-1)/2]+= a->coeff[i];
+    for (i=0; i<b->length; i++) vec->coeff[i + (length-1)/2 - (b->length-1)/2]-= b->coeff[i];
 
     return vec;
 }
 
 /* shift left / or right if "shift" is negative */
-static SwsVector *sws_getShiftedVec(SwsVector *a, int shift){
+static SwsVector *sws_getShiftedVec(SwsVector *a, int shift)
+{
     int length= a->length + FFABS(shift)*2;
-    double *coeff= av_malloc(length*sizeof(double));
     int i;
-    SwsVector *vec= av_malloc(sizeof(SwsVector));
-
-    vec->coeff= coeff;
-    vec->length= length;
+    SwsVector *vec= sws_getConstVec(0.0, length);
 
-    for (i=0; i<length; i++) coeff[i]= 0.0;
+    if (!vec)
+        return NULL;
 
-    for (i=0; i<a->length; i++)
-    {
-        coeff[i + (length-1)/2 - (a->length-1)/2 - shift]= a->coeff[i];
+    for (i=0; i<a->length; i++) {
+        vec->coeff[i + (length-1)/2 - (a->length-1)/2 - shift]= a->coeff[i];
     }
 
     return vec;
 }
 
-void sws_shiftVec(SwsVector *a, int shift){
+void sws_shiftVec(SwsVector *a, int shift)
+{
     SwsVector *shifted= sws_getShiftedVec(a, shift);
     av_free(a->coeff);
     a->coeff= shifted->coeff;
@@ -3380,7 +3286,8 @@ void sws_shiftVec(SwsVector *a, int shift){
     av_free(shifted);
 }
 
-void sws_addVec(SwsVector *a, SwsVector *b){
+void sws_addVec(SwsVector *a, SwsVector *b)
+{
     SwsVector *sum= sws_sumVec(a, b);
     av_free(a->coeff);
     a->coeff= sum->coeff;
@@ -3388,7 +3295,8 @@ void sws_addVec(SwsVector *a, SwsVector *b){
     av_free(sum);
 }
 
-void sws_subVec(SwsVector *a, SwsVector *b){
+void sws_subVec(SwsVector *a, SwsVector *b)
+{
     SwsVector *diff= sws_diffVec(a, b);
     av_free(a->coeff);
     a->coeff= diff->coeff;
@@ -3396,7 +3304,8 @@ void sws_subVec(SwsVector *a, SwsVector *b){
     av_free(diff);
 }
 
-void sws_convVec(SwsVector *a, SwsVector *b){
+void sws_convVec(SwsVector *a, SwsVector *b)
+{
     SwsVector *conv= sws_getConvVec(a, b);
     av_free(a->coeff);
     a->coeff= conv->coeff;
@@ -3404,20 +3313,21 @@ void sws_convVec(SwsVector *a, SwsVector *b){
     av_free(conv);
 }
 
-SwsVector *sws_cloneVec(SwsVector *a){
-    double *coeff= av_malloc(a->length*sizeof(double));
+SwsVector *sws_cloneVec(SwsVector *a)
+{
     int i;
-    SwsVector *vec= av_malloc(sizeof(SwsVector));
+    SwsVector *vec= sws_allocVec(a->length);
 
-    vec->coeff= coeff;
-    vec->length= a->length;
+    if (!vec)
+        return NULL;
 
-    for (i=0; i<a->length; i++) coeff[i]= a->coeff[i];
+    for (i=0; i<a->length; i++) vec->coeff[i]= a->coeff[i];
 
     return vec;
 }
 
-void sws_printVec2(SwsVector *a, AVClass *log_ctx, int log_level){
+void sws_printVec2(SwsVector *a, AVClass *log_ctx, int log_level)
+{
     int i;
     double max=0;
     double min=0;
@@ -3431,8 +3341,7 @@ void sws_printVec2(SwsVector *a, AVClass *log_ctx, int log_level){
 
     range= max - min;
 
-    for (i=0; i<a->length; i++)
-    {
+    for (i=0; i<a->length; i++) {
         int x= (int)((a->coeff[i]-min)*60.0/range +0.5);
         av_log(log_ctx, log_level, "%1.3f ", a->coeff[i]);
         for (;x>0; x--) av_log(log_ctx, log_level, " ");
@@ -3441,19 +3350,22 @@ void sws_printVec2(SwsVector *a, AVClass *log_ctx, int log_level){
 }
 
 #if LIBSWSCALE_VERSION_MAJOR < 1
-void sws_printVec(SwsVector *a){
+void sws_printVec(SwsVector *a)
+{
     sws_printVec2(a, NULL, AV_LOG_DEBUG);
 }
 #endif
 
-void sws_freeVec(SwsVector *a){
+void sws_freeVec(SwsVector *a)
+{
     if (!a) return;
     av_freep(&a->coeff);
     a->length=0;
     av_free(a);
 }
 
-void sws_freeFilter(SwsFilter *filter){
+void sws_freeFilter(SwsFilter *filter)
+{
     if (!filter) return;
 
     if (filter->lumH) sws_freeVec(filter->lumH);
@@ -3464,25 +3376,24 @@ void sws_freeFilter(SwsFilter *filter){
 }
 
 
-void sws_freeContext(SwsContext *c){
+void sws_freeContext(SwsContext *c)
+{
     int i;
     if (!c) return;
 
-    if (c->lumPixBuf)
-    {
+    if (c->lumPixBuf) {
         for (i=0; i<c->vLumBufSize; i++)
             av_freep(&c->lumPixBuf[i]);
         av_freep(&c->lumPixBuf);
     }
 
-    if (c->chrPixBuf)
-    {
+    if (c->chrPixBuf) {
         for (i=0; i<c->vChrBufSize; i++)
             av_freep(&c->chrPixBuf[i]);
         av_freep(&c->chrPixBuf);
     }
 
-    if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf){
+    if (CONFIG_SWSCALE_ALPHA && c->alpPixBuf) {
         for (i=0; i<c->vLumBufSize; i++)
             av_freep(&c->alpPixBuf[i]);
         av_freep(&c->alpPixBuf);
@@ -3504,11 +3415,11 @@ void sws_freeContext(SwsContext *c){
 
 #if ARCH_X86 && CONFIG_GPL
 #ifdef MAP_ANONYMOUS
-    if (c->lumMmx2FilterCode) munmap(c->lumMmx2FilterCode, MAX_MMX2_FILTER_CODE_SIZE);
-    if (c->chrMmx2FilterCode) munmap(c->chrMmx2FilterCode, MAX_MMX2_FILTER_CODE_SIZE);
+    if (c->lumMmx2FilterCode) munmap(c->lumMmx2FilterCode, c->lumMmx2FilterCodeSize);
+    if (c->chrMmx2FilterCode) munmap(c->chrMmx2FilterCode, c->chrMmx2FilterCodeSize);
 #elif HAVE_VIRTUALALLOC
-    if (c->lumMmx2FilterCode) VirtualFree(c->lumMmx2FilterCode, MAX_MMX2_FILTER_CODE_SIZE, MEM_RELEASE);
-    if (c->chrMmx2FilterCode) VirtualFree(c->chrMmx2FilterCode, MAX_MMX2_FILTER_CODE_SIZE, MEM_RELEASE);
+    if (c->lumMmx2FilterCode) VirtualFree(c->lumMmx2FilterCode, c->lumMmx2FilterCodeSize, MEM_RELEASE);
+    if (c->chrMmx2FilterCode) VirtualFree(c->chrMmx2FilterCode, c->chrMmx2FilterCodeSize, MEM_RELEASE);
 #else
     av_free(c->lumMmx2FilterCode);
     av_free(c->chrMmx2FilterCode);