analysis: Fix the "key-press-event" callback signature

We were returning FALSE in a function declared as returning a gpointer.
Turns out that the function should return a gboolea instead.

analysis: None used in pointer context

While this is totally fine (None is 0L and, in the pointer context, will
be converted in the right internal NULL representation, which could be a
value with some bits to 1), I believe it's clearer to use NULL instead
of None when we talk about pointers.

analysis: FALSE/0 used in pointer context

While this is totally fine (0 in the pointer context will be converted
in the right internal NULL representation, which could be a value with
some bits to 1), I believe it's clearer to use NULL in the pointer
context.

It seems that, in most case, it's more an overlook than a deliberate
choice to use FALSE/0 as NULL, eg. copying a _COGL_GET_CONTEXT (ctx, 0)
or a g_return_val_if_fail (cond, 0) from a function returning a
gboolean.

Update git ignore file

buffer: Remove use of CoglHandle in the CoglBuffer API

This replaces the use of CoglHandle with strongly type CoglBuffer *
pointers instead. The only function not converted for now is
cogl_is_buffer which will be done in a later commit.

Start eradicating the CoglHandle typedef

CoglHandle is a common source of complaints and confusion because people
expect a "handle" to be some form of integer type with some indirection
to lookup the corresponding objects as opposed to a direct pointer.

This patch starts by renaming CoglHandle to CoglObject * and creating
corresponding cogl_object_ APIs to replace the cogl_handle ones.

The next step though is to remove all use of CoglHandle in the Cogl APIs
and replace with strongly typed pointer types such as CoglMaterial * or
CoglTexture * etc also all occurrences of COGL_INVALID_HANDLE can just
use NULL instead.

After this we will consider switching to GTypeInstance internally so we
can have inheritance for our types and hopefully improve how we handle
bindings.

Note all these changes will be done in a way that maintains the API and
ABI.

cogl-texture-2d: Regenerate the mipmaps when the texture is modified

When the texture is modified by cogl_texture_set_region, it wasn't
setting mipmaps_dirty so it would never update the mipmaps.

texture: don't use a static bool to gate warning

in create_pick_material we were using a static boolean to gate when we
show a warning, but that would mean if the problem recurs between
different textures then the warning will only be shown once. We now have
a private bitfield flag instead, just so we don't spew millions of
warnings if the problem is recurring.

[texture] Support picking textures according to their alpha channel

This adds a boolean "pick-with-alpha" property to ClutterTexture and when
true, it will use the textures alpha channel to define the actors shape when
picking.

Users should be aware that it's a bit more expensive to pick textures like
this (so probably best not to blindly enable it on *all* your textures)
since it implies rasterizing the texture during picking whereas we would
otherwise just send a solid filled quad to the GPU.  It will also interrupt
the internal batching of geometry for pick renders which can otherwise often
be done in a single draw call.

[material] fix _set_alpha_test_function: missing a glEnable(GL_ALPHA_TEST)

Since the default alpha test function of GL_ALWAYS is equivalent to
GL_ALPHA_TEST being disabled we don't need to worry about Enabling/Disabling
it when flushing material state, instead it's enough to leave it always
enabled.  We will assume that any driver worth its salt wont incur any
additional cost for glEnable (GL_ALPHA_TEST) + GL_ALWAYS vs
glDisable (GL_ALPHA_TEST).

This patch simply calls glEnable (GL_ALPHA_TEST) in cogl_create_context

[texture] remove redundant if (paint_opacity == 0) bailout

clutter_texture_paint shouldn't need to optimize the case where
paint_opacity == 0 and bailout, since we've been doing this optimization for
all actors in clutter_actor_paint for a while now.

test-scrolling: Use a better looking bounce easing

cogl-vertex-buffer: Don't disable any texture coord arrays

The enabled state of the texture coord arrays is cached in the cogl
context so there is no need to disable them when we've finished
rendering a vbo.

cogl: Record new enabled arrays in _cogl_disable_other_texcoord_arrays

When _cogl_disable_other_texcoord_arrays is called it disables the
neccessary texcoord arrays and then removes the bits for the disabled
arrays in ctx->texcoord_arrays_enabled. However none of the places
that call the function then set any bits in ctx->texcoord_arrays_enabled
so the arrays would never get marked and they would never get disabled
again.

This patch just changes it so that _cogl_disable_other_texcoord_arrays
also sets the corresponding bits in ctx->texcoord_arrays_enabled.

test-scrolling: Animate intermediate positions

Animating intermediate positions in the viewport panning is a neat
effect.

drag-action: Re-use press_button in emit_drag_end()

Since emit_drag_end() can be called from a MOTION event capture we
cannot call clutter_event_get_button(). We should, instead, use the
press_button value because if we're emitting ::drag-end it means we
also emitted ::drag-begin and the value is valid.

actor: Add accessors pair for :clip-to-allocation

We should have an accessors pair for the :clip-to-allocation property,
to avoid going through the generic GObject accessors.

Add a scrolling interactive example

We can use a container with :clip-to-allocation set to TRUE and a
DragAction to create a panning viewport.

Use ClickAction in test-animation

Instead of using ::button-press-event.

actor: Fix a type check thinko in add_action_with_name()

docs: Description fixes for Action and Constraint

state: Minor fixes and documentation additions

Clean up the code a bit, and move common code paths into separate
function. Also, document clutter_state_set() with examples.

introspection: Add --pkg atk

We need to tell the introspection scanner all the dependencies we
require, including the pkg-config name to use when compiling the
GIR file into a typelib object.

click-action: Handle the event

In case we did handle the event, we should return TRUE from the ::event
signal handler.

docs: Fix DragAction documentation

The ::drag-motion signal does not have a ModifierType argument; also,
there's no need to connect to the ::drag-motion signal any more.

Fix typo in get_accessible()

actor: Put get_accessible() at the end of the class structure

New virtual functions cannot go wherever they want, if we need to
preserve the ABI.

Also, the coding style should match the rest of ClutterActor and
Clutter's own coding style.

docs: Add ClickAction to the API reference

And clean up the reference index.

actor-meta: Disconnect on actor destroy

When destroying an Actor the various ActorMeta instance should already
be disposed - unless something is holding a reference to them, in which
case we should use the ::destroy signal to unset the ActorMeta:actor
back pointer.

action: Add ClickAction

ClickAction adds "clickable" semantics to an actor. It provides all
the business logic to emit a high-level "clicked" signal from the
various low-level signals inside ClutterActor.

drag: Add a default drag-motion behaviour

The DragAction should, by default, drag the actor to which it has been
applied, instead of delegating what to do to the developer. If custom
code need to override it, g_signal_stop_emission_by_name() can be called
to stop the default handler to ever running.

Post-release version bump to 1.3.3

Release 1.3.2 (snapshot)

docs: Add Actor.get_accessible() to the API reference

docs: Update NEWS

state: replaced test with a more complex one

state: fix clutter_state_change (state, "state-name", FALSE)

Properties were not being updated correctly.

cogl: Use a CoglBitmask to store the list of used texcoord arrays

Instead of directly using a guint32 to store a bitmask for each used
texcoord array, it now stores them in a CoglBitmask. This removes the
limitation of 32 layers (although there are still other places in Cogl
that imply this restriction). To disable texcoord arrays code should
call _cogl_disable_other_texcoord_arrays which takes a bitmask of
texcoord arrays that should not be disabled. There are two extra
bitmasks stored in the CoglContext which are used temporarily for this
function to avoid allocating a new bitmask each time.

http://bugzilla.openedhand.com/show_bug.cgi?id=2132

cogl: Add an internal CoglBitmask type

This implements a growable array of bits called CoglBitmask. The
CoglBitmask is intended to be cheap if less than 32 bits are used. If
more bits are required it will allocate a GArray. The type is meant to
be allocated on the stack but because it can require additional
resources it also has a destroy function.

http://bugzilla.openedhand.com/show_bug.cgi?id=2132

cogl-material: Fix the count of the number of layers

Previously the counter for the number of layers was only updated
whenever the texture handle for a layer changes. However there are
many other ways for a new layer to be created for example by setting a
layer combine constant. Also by default the texture on a layer is
COGL_INVALID_HANDLE so if the application tries to create an explicit
layer with no texture by calling cogl_material_set_layer with
COGL_INVALID_HANDLE then it also wouldn't update the count.

This patch fixes that by incrementing the count in
cogl_material_get_layer instead. This function is called by all
functions that may end up creating a layer so it seems like the most
appropriate place.

http://bugzilla.openedhand.com/show_bug.cgi?id=2132

cogl-vertex-buffer: Don't disable layers with no texture coords

It should be quite acceptable to use a texture without defining any
texture coords. For example a shader may be in use that is doing
texture lookups without referencing the texture coordinates. Also it
should be possible to replace the vertex colors using a texture layer
without a texture but with a constant layer color.

enable_state_for_drawing_buffer no longer sets any disabled layers in
the overrides. Instead of counting the number of units with texture
coordinates it now keeps them in a mask. This means there can now be
gaps in the list of enabled texture coordinate arrays. To cope with
this, the Cogl context now also stores a mask to track the enabled
arrays. Instead of code manually iterating each enabled array to
disable them, there is now an internal function called
_cogl_disable_texcoord_arrays which disables a given mask.

I think this could also fix potential bugs when a vertex buffer has
gaps in the texture coordinate attributes that it provides. For
example if the vertex buffer only had texture coordinates for layer 2
then the disabling code would not disable the coordinates for layers 0
and 1 even though they are not used. This could cause a crash if the
previous data for those arrays is no longer valid.

http://bugzilla.openedhand.com/show_bug.cgi?id=2132

test-cogl-materials: Test invalid layers with a constant color

This adds a test using a layer with no texture but that replaces the
fragment color with the layer constant. This currently causes the test
to fail.

http://bugzilla.openedhand.com/show_bug.cgi?id=2132

test-cogl-materials: Run the 'all layers' test with different prims

The drawing code from test_invalid_texture_layers which draws a
rectangle, a polygon and a vertex buffer has been split out to
separate function. test_using_all_layers now also uses this so that it
will also test the other two primitives. This causes the test to fail
when all of the layers are drawn using a vertex buffer.

http://bugzilla.openedhand.com/show_bug.cgi?id=2132

Implement AtkImplementor on ClutterActor

Implements AtkImplementor interface on ClutterActor, and his method
atk_implementor_ref_accessible.

More information:
http://library.gnome.org/devel/atk/stable/AtkObject.html#AtkImplementor

Implemented clutter_actor_get_accessible

Added the implementation for clutter_actor_get_accessible, virtual
ClutterActor function, used to obtain the accessible object of
any ClutterActor.

As it is defined virtual, it would be possible to redefine it, so
any custom clutter actor could implement their accessibility object,
withouth relying totally on a accessibility implementation module.

See gtkiconview as example.

http://bugzilla.openedhand.com/show_bug.cgi?id=2070

Added Atk dependency

Added Atk dependency, in order to implement a built-in
accessibility support on Clutter.

build: Remove shave files from EXTRA_DIST

We don't use Shave any more, and require automake 1.11 instead.

state: Use Interval.compute() instead of compute_value()

Use the newly-added compute() variant to avoid creating a GValue only
for getting it copied by g_object_set_property().

interval: Add a pointer variant to compute_value()

ClutterInterval.compute_value() computes the new value given a progress
and copies it to a given GValue. Since most of the time we want to pass
that very same value to another function that copies it again, we should
have a compute_value() variant that stores that computed value inside
ClutterInterval and returns a pointer to it. This way we initialize the
result GValue just once and we never copy it, as long as the Interval
instance is valid.

docs: Align parameter names between header and source code

docs: Add ClutterState to the API reference

Merge branch 'wip/state-machine'

* wip/state-machine:
  Do not use wildcards in test-state
  script: Implement State deserialization
  state: added a "target-state" property
  state: documented data structures
  Add State interactive tests to the ignore file
  state: Documentation and introspection annotation fixes
  state: Minor coding style fixes
  state: Clean up the header's documentation
  state: Constify StateKey accessors
  Do not include clutter.h from a Clutter header file
  state-machine: made clutter_state_change take a boolean animate argument
  state-machine: use clutter_timeline_get_progress
  state-machine: add completed signal
  state machine: added state machine

Conflicts:
.gitignore

Merge branch 'wip/framebuffer-bits'

* wip/framebuffer-bits:
  Implement accessors for the color bits in a framebuffer

Do not use wildcards in test-state

Use named states, like in a real case.

script: Implement State deserialization

It should be possible to describe ClutterState transitions using
ClutterScript in a similar way as ClutterAnimator.

Merge branch 'wip/constraints'

* wip/constraints: (24 commits)
  Add the Cogl API reference to the fixxref extra directories
  Document the internal MetaGroup class
  Remove the construct-only flag from ActorMeta:name
  doc: Remove gtk-doc annotations from the json-glib copy
  doc: Fix parameter documentation
  Add named modifiers for Action and Constraint
  Remove a redundant animation
  Set the stage resizable in test-constraints
  Use a 9 grid for the constraints test
  Miscellaneous documentation fixes
  docs: Document animating action and constraint properties
  docs: Document BindConstraint and AlignConstraint
  constraint: Rename BindConstraint:bind-axis
  constraints: Add AlignConstraint
  tests: Add a constraints interactive test
  constraint: Add BindConstraint
  actor: Implement Animatable
  animation: Use the new Animatable API for custom properties
  animatable: Add custom properties to Animatable
  constraint: Add ClutterConstraint base class
  ...

Conflicts:
configure.ac

state: added a "target-state" property

When set the ClutterState transitions to the new state.

state: documented data structures

Add State interactive tests to the ignore file

state: Documentation and introspection annotation fixes

state: Minor coding style fixes

state: Clean up the header's documentation

state: Constify StateKey accessors

Since the accessors do not modify the StateKey we should constify the
argument.

Do not include clutter.h from a Clutter header file

Clutter header files should just include the needed headers.

math: Adds an experimental cogl_vector3_* API

This adds a math utility API for handling 3 component, single precision
float vectors with the following; mostly self explanatory functions:

cogl_vector3_init
cogl_vector3_init_zero
cogl_vector3_equal
cogl_vector3_equal_with_epsilon
cogl_vector3_copy
cogl_vector3_free
cogl_vector3_invert
cogl_vector3_add
cogl_vector3_subtract
cogl_vector3_multiply_scalar
cogl_vector3_divide_scalar
cogl_vector3_normalize
cogl_vector3_magnitude
cogl_vector3_cross_product
cogl_vector3_dot_product
cogl_vector3_distance

Since the API is experimental you will need to define
COGL_ENABLE_EXPERIMENTAL_API before including cogl.h if you want to use
the API.

state-machine: made clutter_state_change take a boolean animate argument

Most of the time this will be TRUE, pass FALSE to change to the target
state immediately.

state-machine: use clutter_timeline_get_progress

state-machine: add completed signal

Added a completed signal to the animator

state machine: added state machine

cookbook: Add a recipe for CairoTexture

The Clutter cookbook has a chapter for textures. It would be useful to
provide a recipe on simple uses of ClutterCairoTexture as part of that.

Some suggested content is attached.

cogl-texture-2d: Optimise _cogl_texture_2d_is_pot

This takes the simple algorithm from here:

http://graphics.stanford.edu/~seander/bithacks.html#DetermineIfPowerOf2

This avoids a loop and is much faster.

Add the Cogl API reference to the fixxref extra directories

Document the internal MetaGroup class

Just because it's an internal class, ClutterMetaGroup should not be left
undocumented.

Remove the construct-only flag from ActorMeta:name

The ClutterActorMeta:name property should be read-write, and be
writeable at any time.

doc: Remove gtk-doc annotations from the json-glib copy

There's no need to use gtk-doc annotations: the API is internal to
Clutter.

doc: Fix parameter documentation

Add named modifiers for Action and Constraint

The ClutterActor API should have modifier methods for adding, removing
and retrieving Actions and Constraints using the ClutterActorMeta:name
property - mostly, for convenience.

Remove a redundant animation

Set the stage resizable in test-constraints

To demonstrate that constraints can be used to reposition actors in a
resizable stage, set the :user-resizable property to TRUE, and spend the
next 15 minutes playing with the stage size.

Use a 9 grid for the constraints test

Let's complicate the interactive constraints test a little bit by adding
more actors and animating two constraints at a time.

Miscellaneous documentation fixes

Stubs out _cogl_xlib_handle_event

This stubs out an xlib event handling mechanism for Cogl. The intention
is for Clutter to use this to forward all x11 events to Cogl. As we move
winsys functionality down into Cogl, Cogl will become responsible for
handling a number of X events: ConfigureNotify events for onscreen
framebuffers, swap events and Damage events for cogl_x11_texture_pixmap.

docs: Document animating action and constraint properties

Add a refsect for the syntax to be used when animating action and
constraint properties with ClutterAnimation.

build: expose automake conditionals for cogl winsys

Expose the ./configured window system/backend options to the Cogl
automake files via some new SUPPORT_XYZ conditionals.

configure.ac: Adds some COGL_HAS_XYZ_SUPPORT defines

This exposes the ./configured window system/backend options to Cogl via
a set of new COGL_HAS_XYZ_SUPPORT defines:

COGL_HAS_{X11,XLIB,GLX,EGL,EGL_PLATFORM_XYZ,OSX,WIN32,WGL}_SUPPORT

docs: Document BindConstraint and AlignConstraint

Add the missing gtk-doc annotations for BindConstraint and
AlignConstraint, plus the licensing blurb.

constraint: Rename BindConstraint:bind-axis

We're not binding an axis: we're really binding a coordinate of an actor
to the coordinate of another one.

constraints: Add AlignConstraint

AlignConstraint is a simple constraint that keeps an actor's position
aligned to the width or height of another actor, multiplied by an
alignment factor.

tests: Add a constraints interactive test

Show how to use constraints and how to animate them.

constraint: Add BindConstraint

The BindConstraint object is a constraint that binds the current
position of an actor on a given axis to the actor that has the
constraint applied.

actor: Implement Animatable

By implementing the newly added support for custom animatable
properties, we can allow addressing action and constraint properties
from ClutterAnimation and clutter_actor_animate().

animation: Use the new Animatable API for custom properties

The Animation class should use the Animatable API for custom properties
to override finding a property definition, getting the initial state and
setting the final state of an object.

animatable: Add custom properties to Animatable

Like ClutterScriptable, it would be good to have the Animatable
interface allow defining custom properties that can be animated.

constraint: Add ClutterConstraint base class

The Constraint base, abstract class should be used to implement Actor
modifiers that affect the way an actor is sized or positioned inside a
fixed layout manager.

eglx: implement cogl_get_proc_address()

http://bugzilla.openedhand.com/show_bug.cgi?id=2124

eglx: Rename EGLNative{Window,Display}Type back to Native{Window,Display}Type

Commit e2a990d renamed these types to new names from EGL 1.3. However
it still works to use the old names under EGL 1.3 so let's just use
those to keep compatibility.

clutter-backend-egl: Chain up in dispose before destroying the context

clutter_backend_egl_dispose now chains up before disposing its own
resources so that ClutterBackendX11 will destroy all of the stages
before we destroy the egl context. Otherwise the actors may try to
make GL calls during destruction which causes a crash.

clutter-stage-egl.c: Don't call eglWait*()

According to the docs, eglSwapBuffers performs an implicit glFlush
before swapping so there should be no need to wait.

test-cogl-materials: Fix compilation under GLES 2

GLES 2 doesn't have GL_MAX_TEXTURE_UNITS. Instead the cogl backend
uses GL_MAX_TEXTURE_IMAGE_UNITS with a maximum limit of 16. The same
restriction is now used in the test.

fix case when EGL driver doesn't support EGL_NATIVE_VISUAL_ID

Some EGL drivers, such as the PowerVR simulator (and some proprietary drivers)
return zero when the EGLConfig is queried for the EGL_NATIVE_VISUAL_ID
attribute via eglGetConfigAttrib().

This patch detects and attempts to work around that situation by picking a
visual with the same color depth.

http://bugzilla.openedhand.com/show_bug.cgi?id=2123