Pre-release version bump

Don't assert when malformed regions are detected.

Instead print a message to stderr so that it will end up in the X log
file.

Fix another search and replace issue

Fix search-and-replace issue pointed out by Koen Kooi.

Add implementation of MMX __m64 functions for MSVC x64.

Microsoft C++ does not define __m64 and all related MMX functions in
x64.  However, it succeeds in generating object files for SSE2 code
inside pixman.

The real problem happens during linking, when it cannot find MMX functions
(which are not defined as intrinsics for AMD64 platform).

I have implemented those missing functions using general programming.

MMX __m64 is used relatively scarcely within SSE2 implementation, and the
performance impact probably is negligible.

Bug 22390.

Fix read of BITS members from a solid image.

During the fast-path query, the read_func and write_func from the bits
structure are queried for the solid image.

==32723== Conditional jump or move depends on uninitialised value(s)
==32723==    at 0x412AF20: _pixman_run_fast_path (pixman-utils.c:681)
==32723==    by 0x4136319: sse2_composite (pixman-sse2.c:5554)
==32723==    by 0x4100CD2: _pixman_implementation_composite
(pixman-implementation.c:227)
==32723==    by 0x412396E: pixman_image_composite (pixman.c:140)
==32723==    by 0x4123D64: pixman_image_fill_rectangles (pixman.c:322)
==32723==    by 0x40482B7: _cairo_image_surface_fill_rectangles
(cairo-image-surface.c:1180)
==32723==    by 0x4063BE7: _cairo_surface_fill_rectangles
(cairo-surface.c:1883)
==32723==    by 0x4063E38: _cairo_surface_fill_region
(cairo-surface.c:1840)
==32723==    by 0x4067FDC: _clip_and_composite_trapezoids
(cairo-surface-fallback.c:625)
==32723==    by 0x40689C5: _cairo_surface_fallback_paint
(cairo-surface-fallback.c:835)
==32723==    by 0x4065731: _cairo_surface_paint (cairo-surface.c:1923)
==32723==    by 0x4044098: _cairo_gstate_paint (cairo-gstate.c:900)
==32723==  Uninitialised value was created by a heap allocation
==32723==    at 0x402732D: malloc (vg_replace_malloc.c:180)
==32723==    by 0x410099F: _pixman_image_allocate (pixman-image.c:100)
==32723==    by 0x41265B8: pixman_image_create_solid_fill
(pixman-solid-fill.c:75)
==32723==    by 0x4123CE1: pixman_image_fill_rectangles (pixman.c:314)
==32723==    by 0x40482B7: _cairo_image_surface_fill_rectangles
(cairo-image-surface.c:1180)
==32723==    by 0x4063BE7: _cairo_surface_fill_rectangles
(cairo-surface.c:1883)
==32723==    by 0x4063E38: _cairo_surface_fill_region
(cairo-surface.c:1840)
==32723==    by 0x4067FDC: _clip_and_composite_trapezoids
(cairo-surface-fallback.c:625)
==32723==    by 0x40689C5: _cairo_surface_fallback_paint
(cairo-surface-fallback.c:835)
==32723==    by 0x4065731: _cairo_surface_paint (cairo-surface.c:1923)
==32723==    by 0x4044098: _cairo_gstate_paint (cairo-gstate.c:900)
==32723==    by 0x403C10B: cairo_paint (cairo.c:2052)

Only apply the workaround if the clip region extends beyond the drawable.

This works because the X server always attempts to set a clip region
within the bounds of the drawable, and it only fails at it when it is
computing the wrong translation and therefore needs the workaround.

Rework the workaround for bogus X server images.

Bug 22844 demonstrates that it is not sufficient to play tricks with
the clip regions to work around the bogus images from the X
server. The problem there is that if the operation hits the general
path and the destination has a different format than a8r8g8b8, the
destination pixels will be fetched into a temporary array. But because
those pixels would be outside the clip region, they would be fetched
as black. The previous workaround was relying on fast paths fetching
those pixels without checking the clip region.

In the new scheme we work around the problem at the
pixman_image_composite() level. If an image is determined to need a
work around, we translate both the bits pointer, the coordinates, and
the clip region, thus effectively undoing the X server's broken
computation.

Add test case for bug 22844.

Fix typo in sse2_combine_over_pixbuf_0565()

Fix NEON build for older ARM CPUs

The pld instruction used in the NEON assembler code is only available
for ARMv5e and >= ARMv6.

Set -mcpu=cortex-a8 when compiling the source file (similar to what is
already done for the SIMD build).

Some formatting changes to pixman-arm-neon.c

ARM: Fixes for the inline assembly constraints in pixman_fill_neon

Some of the variables in the inline assembly arguments list are
actually modified by the assembly code, they are now marked
appropriately.

ARM: Workaround cs2007q3 compiler bug for NEON registers clobber list

128-bit registers "qX" are incorrectly handled in inline assembly
clobber list for codesourcery cs2007q3 gcc toolchain. Only the
first 64-bit half is saved and restored by gcc. Changing clobber
list to use only 64-bit register aliases can solve this problem.
For example, 128-bit register q0 is mapped to two 64-bit
registers d0 and d1, q1 is mapped to d2 and d3, etc.

ARM: Commented out the rest of buggy NEON optimizations

These functions have problems with invalid memory accesses and often
crash X server

ARM: Use Ian's variant of 'neon_composite_over_n_8_0565' function again

This patch effectively reverts the changes done by commit
8eeeca993252edc39da9c5c57545b81215fafc81 which was causing
severe stability issues, and restores old variant of
'neon_composite_over_n_8_0565' function, which used to work
correctly.

Update Makefile.win32 to make it work again.

Add a lot more sanity checks to region code

- Introduce a GOOD_RECT() macro that checks that a pixman_box_t is not
  empty or degenerate an use it.
- Use GOOD_RECT() instead of magic if statements for funtions that take
  x, y, width, height arguments
- Use GOOD_RECT() in _reset(). The checks in the previous code seemed to
  allow an empty box, but then created a broken region from it.
- Add GOOD(region) check at the end of _translate()

Handle degenerate case in pixman_init_extents()

Create an empty region instead

Replace // comments with /* */ comments in various places

Align the stack in _pixman_implementation_create_sse2()

When compiled without optimization, GCC will place various temporaries
on the stack. Since Firefox sometimes causes the stack to be aligned
to four bytes, this causes movdqa to generate faults.

Check whether the linker understands the hwcap file before using it.

If we're trying to use the GNU linker on Solaris we shouldn't use
our solaris-hwcap.mapfile since it doesn't grok the mapfile format.

Move read and write functions to the bits_image_t struct.

Those fields were duplicated between image_common and bits_image_t
before.

Handle degenerate case in pixman_init_rect()

Create an empty region in that case.

Add back check for need_workaround that got removed during reindenting

Fix combine_src_ca() to fill out all of the destination line.

Change composite test to use a rainbow gradient as source.

Also make the destination a yellow patch. This makes the output a bit
more comparable to the image in the PDF specification.

Various minor formatting changes

Change name of macro from RGB16_TO_ENTRY to RGB15_TO_ENTRY

Clarify color burn code - no semantical changes

- Improve documentation to be equal to Proposed ISO32000 Extension
- Simplify code in default case

Clarify color dodge code - no semantical changes

- Improve documentation to be equal to Proposed ISO32000 Extension
- Simplify code in default case

Fix terms in comments describing the blend modes

Add -fno-strict-aliasing

Post-release version bump

Pre-release version bump

Reindent and rename identifiers in scaling-test.c

Reformat pixman.h

Reindent and reformat pixman-private.h

Reindent and reformat pixman-combine.h.template

Reindent and reformat pixman-combine.c.template

Reindent and reformat pixman-vmx.c

Reformat and reindent pixman-utils.c

Reformat and reindent pixman-trap.c

Reformat pixman-timer.c

Reformat and reindent pixman-sse2.c

Reformat and reindent pixman-solid-fill.c

Reformat and reindent pixman-region.c

Reindent and reformat pixman-radial-gradient.c

Reindent and reformat pixman-mmx.c

Reindent and reformat pixman-matrix.c

Reindent and reformat pixman-linear-gradient.c

Reindent and reformat pixman-implementation.c

Reindent and reformat pixman-image.c

Reindent and reformat pixman-gradient-walker.c

Reindent and reformat pixman-general.c

Reindent and reformat pixman-fastpath.c

Reindent and reformat pixman-edge.c

Reindent and reformat pixman-cpu.c

Reindent and reformat pixman-conical-gradient.c

Reindent and reformat pixman.c

Reindent and reformat pixman-bits-image.c

Reindent and reformat pixman-arm-simd.c

Reindent and reformat pixman-arm-neon.c

Reindent and reformat pixman-access.c.

Fix burn and dodge operators to match acroread output

Return immediately if the operator is CONJOINT_DST or DISJOINT_DST

These are noops just like plain DST is.

Fix bits_image_fetch_{un}transformed() for 64 bit buffers.

The buffer pointer has to advance twice as far as in the 32 bit case.

Make sure we get all 64 bits in bits_image_fetch_solid_64()

Previously we would only store the first 32 bits.

Fix typo in bits_image_fetch_solid_64().

Found by blitters-test.

Only destroy the regions when pixman_compute_composite_region() returns TRUE

Move workaround logic to pixman-bits-image.c.

Instead of computing whether a workaround is needed on every call to
_pixman_run_fast_path(), just cache this information in the image.

Also, when workarounds are needed, clip against the source geometry to
prevent out of bound reads.

Only apply the workaround to source images when out_of_bounds_workaround is set.

Pointed out by Siarhei Siamashka.

Return nonzero code from scaling-test program in case of failure.

This can potentially help in tests automation.

Fix scaling-test to work on big endian systems

Convert some leftover instances of Alpha() in pixman-vmx.c.

They were probably missed due to the space before the parens.

Add workarounds for X servers doing out-of-bounds accesses.

Old X servers rely on out-of-bounds accesses when they are asked
to composite with a window as the source. They create a pixman image
pointing to some bogus position in memory, but then they set a clip
region to the position where the actual bits are.

Due to a bug in old versions of pixman, where it would not clip
against the image bounds when a clip region was set, this would
actually work.

The workaround added by this commit is to try and detect whether a
source drawable is actually a window without a client clip set. Such a
window will generally have a clip region that corresponds exactly to
the hierarchy clip in the server, whereas pixmaps will have a clip
region that is an exact match to the drawable.

When we detect such a window, we allow a fast path to run that would
normally be rejected due to the sources not completely subsuming the
composite region.

Fixed X servers should call the new function
pixman_disable_out_of_bounds_workaround() to disable the workaround.

This was reported in bug 22484.

Add a BUILT_SOURCES with pixman-combine{32,64}.{c.h}

Bug 22681.

Fix the NEON build after "Convert CamelCase names to underscore_names."

    This patch fixes the following build error caused
    by commit a98b71eff4041df58c9dcc2b1e25cefa38f364ff
    (Convert CamelCase names to underscore_names.):

    <--   snip  -->

    ...
    pixman-arm-neon.c: In function 'neon_composite_over_n_8_0565':
    pixman-arm-neon.c:1784: error: 'x_dst' undeclared (first use in this function)
    pixman-arm-neon.c:1784: error: (Each undeclared identifier is reported only once
    pixman-arm-neon.c:1784: error: for each function it appears in.)
    pixman-arm-neon.c:1785: error: 'p_dst' undeclared (first use in this function)
    pixman-arm-neon.c: In function 'neon_composite_over_n_0565':
    pixman-arm-neon.c:1937: error: 'x_dst' undeclared (first use in this function)
    pixman-arm-neon.c:1938: error: 'p_dst' undeclared (first use in this function)
    pixman-arm-neon.c: In function 'neon_composite_over_8888_0565':
    pixman-arm-neon.c:2074: error: 'x_dst' undeclared (first use in this function)
    pixman-arm-neon.c:2075: error: 'p_dst' undeclared (first use in this function)
    make[3]: *** [libpixman_arm_neon_la-pixman-arm-neon.lo] Error 1
    ...

    <--  snip  -->

Update the CRC value in scaling-test.c.

The changes in pixman behavior justifying this are:

- New clipping rules

- Bug fixes in region code. In particular, when
  pixman_region_init_rects() is called on these two boxes:

{ 2, 6, 7, 6 }
{ 4, 1, 6, 7 }

  it now ignores the first one, which is empty and produces

{ 4, 1, 6, 7 }

  Previously, it would produce:

{ 2, 1, 7, 7 }

  for some reason.

Fix bug in pixman-region.c where empty regions would not be properly initialized.

Also add a couple more tests to region-test.c.

Fix a couple of problems with the tests when HAVE_GTK is not defined.

- Make sure the non-gtk+ test programs are added to noinst_PROGRAMS
  when HAVE_GTK is not set.

- Don't include glib.h in oob-test.c

Merge branch 'naming'

Eliminate boxes with x1 > x2 or y1 > y2 in pixman_region_init_rects().

Add a box with y2 < y1 in region-test.c

Fix up some overeager search-and-replace renamings

Rename num_rects back to numRects.

The name numRects is public API and can't be changed.

Rename combine_*_c to combine_*_ca

s/combine_(.+)_c([^a-z0-9A-Z])/combine_$1_ca$2/g;

Various sse2 renamings

s/sse2combine/sse2_combine/g

Convert CamelCase names to underscore_names.

s/sizeRI/size_ri/g;
s/numRI/num_ri/g;
s/RepeatNone/REPEAT_NONE/g;
s/fbOver/over/g;
s/fbIn/in/g;
s/iSrc/src_image/g;
s/iMask/mask_image/g;
s/iDst/dest_image/g;
s/SaDa/Sa.Da/g;
s/FbMaskBits/MASK_BITS/g;
s/RenderSamplesX/RENDER_SAMPLES_X/g;
s/MMXData/mmx_data_t/g;
s/RegionInfo/region_info_t/g;

s/([^0x])([a-z])([A-Z])/$1$2_\l$3/g;
s/([^0x])([A-Z])([A-Z])([a-z])/$1$2_\l$3$4/g;
s/([^0x])([A-Z])([a-z]+)_([a-z])/$1\l$2$3_$4/g;
s/([a-z])_([A-Z])/$1_\l$2/g;

s/su_sE/SuSE/g;
s/X_Free86/XFree86/g;
s/X_free86/XFree86/g;

s/_ULL/ULL/g;
s/_uLL/ULL/g;

s/U_nc/UNc/g;
s/combine ##/combine_ ##/g;
s/## U/## _u/g;
s/## C/## _c/g;
s/UNc_aDD/UNc_ADD/g;

s/BLEND_MODE \((.+)\)/BLEND_MODE (\l$1)/g;
s/blend_(.+)/blend_\l$1/g;

s/AN_ds/ANDs/g;
s/O_rs/ORs/g;
s/over565/over_565/g;
s/8pix/8_pix/g;
s/Over565/over_565/g;
s/inU/in_u/g;
s/inPart/in_part/g;
s/inC/in_c/g;
s/inreverse/in_reverse/g;
s/get_exception_code/GetExceptionCode/g; # GetExceptionCode is WinCE API
s/CP_us/CPUs/g;
s/authentic_aMD/AuthenticAMD/g;
s/op_sR_cx_mAS_kx_dST/op_src_mask_dest/g;
s/no_VERBOSE/noVERBOSE/g;
s/mc_cormack/McCormack/g;
s/r1band/r1_band/g;
s/r2band/r2_band/g;
s/as GOOD things/as good things/g;
s/brokendata/broken_data/g;
s/X_render/XRender/g;
s/__open_bSD__/__OpenBSD__/g;
s/^Quick/quick/g;
s/NextRect/next_rect/g;
s/RectIn/rect_in/g;
s/pboxout/pbox_out/g;
s/F_sorted/FSorted/g;
s/usse2/u_sse2/g;
s/csse2/c_sse2/g;
s/cPixelsse2/c_pixel_sse2/g;
s/Mask565/mask_565/g;
s/565fix_rB/565_fix_rb/g;
s/565fix_g/565_fix_g/g;
s/565r/565_r/g;
s/565g/565_g/g;
s/565b/565_b/g;
s/uPixelsse2/u_pixel_sse2/g;
s/Mask00ff/mask_00ff/g;
s/Mask0080/mask_0080/g;
s/Mask0101/mask_0101/g;
s/Maskffff/mask_ffff/g;
s/Maskff000000/mask_ff000000/g;
s/load128Aligned/load_128_aligned/g;
s/load128Unaligned/load_128_unaligned/g;
s/save128Aligned/save_128_aligned/g;
s/save128Unaligned/save_128_unaligned/g;
s/fillsse2/fill_sse2/g;
s/unpack565/unpack_565/g;
s/pack565/pack_565/g;
s/bltsse2/blt_sse2/g;
s/x565Unpack/x565_unpack/g;
s/r1End/r1_end/g;
s/r2End/r2_end/g;
s/argb8Pixels/argb8_pixels/g;

Remove reference to 8888_RevNP

Remove reference to 8888RevNP

NoFeatures => NO_FEATURES

s/FbScrRight/SCREEN_SHIFT_RIGHT/g

CPUFeatures => cpu_features

Rename FbGet8 to GET8

Rename RBmask/Gmask => rb_mask/g_mask in pixman-arm-neon.c

Use ALPHA_8 in pixman-image.c instead of Alpha

Uppercase a few more macros in pixman-combine.c.template

Rename macros for non-separable blend modes

Lum => LUM
Sat => SAT
Min => CH_MIN
Max => CH_MAX