Signed-off-by: NDiego Biurrun <diego@biurrun.de>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This makes it consistent with the msvc builds which automatically set
the DEP and ASLR flags by default.  There really is no good reason why
they shouldn't be set.

The fact that binutils does not set them on by default boggles the mind.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Move the local variables it uses into the macro, enclosing them
in a do {} while (0) scope.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This fixes out of array accesses.

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NMartin Storsjö <martin@martin.st>

This is required if we return other error codes than explicitly
-1, which so far has been the only other possible return value
besides 0.
Signed-off-by: NMartin Storsjö <martin@martin.st>

No sample tried shows a difference.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This fixes several files from VLC ticket 5887.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This replaces a large number of checks for the second field by
fixing the pointers when they are setup.

This should also fix I/BI field pictures.

Changes checksums for vc1_sa10143, the file becomes slightly closer
to what the reference decoder outputs.

Based on "vc1dec: the second field is written wrong to the picture"
by Sebastian Sandberg <sebastiand.sandberg@gmail.com>.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Currently it would incorrectly trigger on a string that contains a '='
but does not contain a ':', e.g. flags=<flags>.

There is now no need to explicitly pass 0:0 as width/height to scale,
those are the defaults.

Signed-off-by: NAnton Khirnov <anton@khirnov.net>

Using Optional Data chunk introduced changes the hash.

Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Also eliminate an unnecessary cast. Fixes the following warning:
jpeg2000dec.c:1200:20: warning: format ‘%lX’ expects argument of type ‘long unsigned int’, but argument 5 has type ‘long long unsigned int’

Fixes read of uninitialized memory.

Based on a patch by Michael Niedermayer.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This is not implemented and doesn't work.
Signed-off-by: NMartin Storsjö <martin@martin.st>

The fieldtx_plane is not cleared for interlaced fields.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This was due to a miscomputation of s->cur_channel, which led to
psy-based encoders using the psy coefficients for the wrong channel.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This fixes a case where multichannel bitrate isn't accurately
targetted by psy model alone, never achieving the target bitrate.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

The C code treats the data as arrays of uint16_t so strides must not
be in bytes but in pixels.
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

Always use the special filter for the first and last 3 columns (only).

Changes made in 64ed3976 slowed the filter to just under 3/4 of what it
was.  This commit restores the speed while maintaining identical output.

For reference, on my Athlon64:
1733222 decicycles in old
2358563 decicycles in new
1727558 decicycles in this
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

This fixes https://bugzilla.libav.org/show_bug.cgi?id=514.
Signed-off-by: NMartin Storsjö <martin@martin.st>

If this is explicitly disabled for win32/mingw, it should also
be disabled for cygwin, for consistency and for the same reasons
as for win32/mingw.
Signed-off-by: NMartin Storsjö <martin@martin.st>

The header is not (anymore) MMX-specific.

Also remove some pointless comments.