profile/ivi/clutter.git
14 years agoAdd some more argument validation
Emmanuele Bassi [Thu, 23 Sep 2010 15:17:24 +0000 (16:17 +0100)]
Add some more argument validation

This is all internal, so we shouldn't need it; unfortunately, it seems
we're passing invalid data internally, so for the time being catching
inconsistencies should at least emit a warning for us to backtrace.

14 years agoactor: don't pass NULL to _stage_set_pick_buffer_valid
Robert Bragg [Thu, 23 Sep 2010 15:03:06 +0000 (16:03 +0100)]
actor: don't pass NULL to _stage_set_pick_buffer_valid

This adds a check in clutter_actor_real_queue_redraw after calling
_clutter_actor_get_stage_internal to check in case the actor doesn't yet
have an associated stage so we can avoid passing a NULL stage pointer to
_clutter_stage_set_pick_buffer_valid which could cause a crash.

14 years agodocs: Put deprecated Cogl symbols into new section
Robert Bragg [Thu, 23 Sep 2010 14:45:27 +0000 (15:45 +0100)]
docs: Put deprecated Cogl symbols into new section

This adds a "Cogl deprecated API" chapter to the Cogl reference manual
so we can group all the documentation for deprecated symbols together
instead of having them clutter up the documentation of symbols we would
rather developers used.

14 years agodocs: update the overview paragraph for Cogl
Robert Bragg [Wed, 25 Aug 2010 21:31:30 +0000 (22:31 +0100)]
docs: update the overview paragraph for Cogl

Instead of describing OpenGL as "a low level OpenGL abstraction
library" it is now summarised as "modern 3D graphics API".

14 years agodocs: cogl-texture-3d wasn't listed as experimental
Robert Bragg [Wed, 25 Aug 2010 19:36:20 +0000 (20:36 +0100)]
docs: cogl-texture-3d wasn't listed as experimental

The CoglTexture3D API is only available when defining
COGL_ENABLE_EXPERIMENTAL_API so it should be listed in the experimental
section of the API reference.

14 years agodocs: Use "Cogl" not "COGL" in Cogl API reference
Robert Bragg [Wed, 25 Aug 2010 19:33:27 +0000 (20:33 +0100)]
docs: Use "Cogl" not "COGL" in Cogl API reference

Cogl isn't an acronym so we should use "Cogl" instead of "COGL" in
our documentation.

14 years agotable-layout: Invert row/column in arguments
Emmanuele Bassi [Thu, 23 Sep 2010 14:15:37 +0000 (15:15 +0100)]
table-layout: Invert row/column in arguments

                *** This is an API change ***

The general pattern for axis-aligned arguments is:

        x argument
        y argument

If we consider columns an x-aligned argument, and row a y-aligned
argument, then we need to update the TableLayout functions to be:

        column
        row

and not:

        row
        column

14 years agodocs: Mention Uncrustify in the coding style document
Emmanuele Bassi [Thu, 23 Sep 2010 12:36:56 +0000 (13:36 +0100)]
docs: Mention Uncrustify in the coding style document

14 years agoUpdate the uncrustify configuration
Emmanuele Bassi [Thu, 23 Sep 2010 12:36:41 +0000 (13:36 +0100)]
Update the uncrustify configuration

14 years agoAdd uncrustify configuration file
Emmanuele Bassi [Thu, 23 Sep 2010 12:00:06 +0000 (13:00 +0100)]
Add uncrustify configuration file

Patch submission should include a pass of uncrustify to conform to the
coding style.

Uncrustify is not perfect - but at least it's a start.

14 years agopicking: Fix tracking of pick buffer validity
Robert Bragg [Fri, 10 Sep 2010 23:29:05 +0000 (00:29 +0100)]
picking: Fix tracking of pick buffer validity

We have an optimization to track when there are multiple picks per
frame so we can do a full render of the pick buffer to reduce the
number of pick renders for a static scene.

There was a problem though in that we were tracking this information in
the ClutterMainContext, but conceptually this doesn't really make sense
because the pick buffer is associated with a stage framebuffer and there
can be multiple stages for one context.

This patch moves the state tracking to ClutterStage.

14 years agoRevert "picking: Fix tracking of pick buffer validity"
Robert Bragg [Thu, 23 Sep 2010 10:35:42 +0000 (11:35 +0100)]
Revert "picking: Fix tracking of pick buffer validity"

This reverts commit d7e86e26960f4cb2f5f0600357f5df89bd1c46c1.

This was a half baked patch that was pushed a bit early since it broke
test-texture-pick-with-alpha + the commit message refers to a change on
the wip/paint-box branch that hasn't happened yet.

14 years agopicking: Fix tracking of pick buffer validity
Robert Bragg [Fri, 10 Sep 2010 23:29:05 +0000 (00:29 +0100)]
picking: Fix tracking of pick buffer validity

We have an optimization to track when there are multiple picks per
frames so we can do a full render of the pick buffer to reduce the
number of pick renders for a static scene.

There were two problems with how we were tracking this state though.
Firstly we were tracking this information in the ClutterMainContext, but
conceptually this doesn't really make sense because the pick buffer is
associated with a stage framebuffer and there can be multiple stages for
one context.  Secondly - since the change to how redraws are queued - we
weren't marking the pick buffer as invalid when a queuing a redraw, we
were only marking the buffer invalid when signaling/finishing the
queue-redraw process, which is now deferred until just before a paint.
This meant using clutter_stage_get_actor_at_pos after a scenegraph
change could give a wrong result if it just read from an existing (but
technically invalid) pick buffer.

This patch moves the state tracking to ClutterStage, and ensures the
buffer is invalidated in _clutter_stage_queue_actor_redraw.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2283

Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>
14 years agobox-layout: Small cleanups
Emmanuele Bassi [Wed, 22 Sep 2010 11:46:47 +0000 (12:46 +0100)]
box-layout: Small cleanups

14 years agodocs: Update the coding style
Emmanuele Bassi [Wed, 22 Sep 2010 09:07:42 +0000 (10:07 +0100)]
docs: Update the coding style

Make sure to document:

  • nested if
  • conditions
  • interface definition

14 years agotest-conform: Delay initializing Clutter until a test is run
Neil Roberts [Tue, 21 Sep 2010 12:17:53 +0000 (13:17 +0100)]
test-conform: Delay initializing Clutter until a test is run

Instead of calling clutter_init immediately, test-conformance now only
calls it as part of test_conform_simple_fixture_setup. The conformance
tests assert that only one test is run per instance of
test-conformance so it should never end up calling clutter_init
twice. Delaying clutter_init has the advantage that calling
"test-conformance -l" will still work even on systems with no X
server. This could be useful for automated build systems.

14 years agoeffects: Make sure we're using GLSL 1.10
Emmanuele Bassi [Tue, 21 Sep 2010 12:32:31 +0000 (13:32 +0100)]
effects: Make sure we're using GLSL 1.10

14 years agointrospection: Build ClutterJson before Clutter
Emmanuele Bassi [Tue, 21 Sep 2010 12:17:51 +0000 (13:17 +0100)]
introspection: Build ClutterJson before Clutter

Since the latter requires the former.

14 years agoRequire the installed JSON-GLib
Emmanuele Bassi [Tue, 21 Sep 2010 12:15:56 +0000 (13:15 +0100)]
Require the installed JSON-GLib

Continue to provide the internal copy for 1.4, but require an explicit
override to use it.

The internal copy will be removed for Clutter 1.6.

14 years agobuild: Update the remote publish path for the cookbook
Emmanuele Bassi [Tue, 21 Sep 2010 11:02:41 +0000 (12:02 +0100)]
build: Update the remote publish path for the cookbook

14 years agoclutter-box-layout: Swap the default request mode
Neil Roberts [Mon, 20 Sep 2010 11:04:51 +0000 (12:04 +0100)]
clutter-box-layout: Swap the default request mode

The request mode set by the box layout was previously width-for-height
in a vertical layout and height-for-width in a horizontal layout which
seems to be wrong. For example, if width-for-height is used in a
vertical layout then the width request will come second with the
for_height parameter set. However a vertical layout doesn't pass the
for_height parameter on to its children so doing the requests in that
order doesn't help. If the layout contains a ClutterText then both the
width and height request for it will have -1 for the for_width and
for_height parameters so the text would end up allocated too small.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2328

14 years agoMerge branch 'cookbook-layouts-bind-constraint'
Emmanuele Bassi [Mon, 20 Sep 2010 13:33:26 +0000 (14:33 +0100)]
Merge branch 'cookbook-layouts-bind-constraint'

* cookbook-layouts-bind-constraint:
  cookbook: Add recipe about sync'ing actor sizes
  cookbook: Example using allocation-changed to sync actor size
  cookbook: Simple example to demonstrate bind constraint
  cookbook: Example of using a bind constraint for an overlay

14 years agodocs: Description fixes for State.set_animator()
Emmanuele Bassi [Mon, 20 Sep 2010 13:31:53 +0000 (14:31 +0100)]
docs: Description fixes for State.set_animator()

14 years agostate: Request the animator for the default state
Bastian Winkler [Fri, 17 Sep 2010 11:21:03 +0000 (13:21 +0200)]
state: Request the animator for the default state

Try to use the default-state animator in case there is no animator for
this specific state change request.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2325

14 years agostate: Fix the usage of ClutterAnimator in ClutterScript
Bastian Winkler [Wed, 15 Sep 2010 13:59:39 +0000 (15:59 +0200)]
state: Fix the usage of ClutterAnimator in ClutterScript

Fix the transition parser to allow transitions that have only an
animator and no keys defined.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2325

14 years agostate: Create a new target state in clutter_state_set_animator
Bastian Winkler [Wed, 15 Sep 2010 12:46:19 +0000 (14:46 +0200)]
state: Create a new target state in clutter_state_set_animator

clutter_state_set_animator needs to create a new state in order to use a
ClutterAnimator with a target state that doesn't exist yet.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2325

14 years agobuild: Pass CLUTTER_CFLAGS to g-ir-scanner
Dominique Leuenberger [Mon, 20 Sep 2010 13:13:49 +0000 (14:13 +0100)]
build: Pass CLUTTER_CFLAGS to g-ir-scanner

http://bugzilla.clutter-project.org/show_bug.cgi?id=2327

Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>
14 years agodocs: API reference fixes
Emmanuele Bassi [Mon, 20 Sep 2010 12:15:44 +0000 (13:15 +0100)]
docs: API reference fixes

14 years agobuild: Remove unnecessary newline
Emmanuele Bassi [Mon, 20 Sep 2010 10:30:23 +0000 (11:30 +0100)]
build: Remove unnecessary newline

14 years agopo: Update German translation
Frederik Hahne [Mon, 20 Sep 2010 10:23:31 +0000 (11:23 +0100)]
po: Update German translation

14 years agotexture: Add more validation on the material
Emmanuele Bassi [Sat, 18 Sep 2010 07:41:52 +0000 (08:41 +0100)]
texture: Add more validation on the material

If set_cogl_texture() is called after unsetting the Texture's material
then we really want to make a copy of the template.

Also, we should assert more often if the internal state goes horribly
wrong: at least, we'll have a backtrace.

14 years agoFix the ordering of the arguments for clutter_table_layout_set_span
Neil Roberts [Fri, 17 Sep 2010 16:10:39 +0000 (17:10 +0100)]
Fix the ordering of the arguments for clutter_table_layout_set_span

The order of the row_span and column_span arguments was different in
the declaration from that in the definition. This was causing the
gtk-doc to also have the wrong order.

14 years agocogl-object-private.h: Include cogl-debug.h
Neil Roberts [Fri, 17 Sep 2010 16:18:09 +0000 (17:18 +0100)]
cogl-object-private.h: Include cogl-debug.h

If COGL_OBJECT_DEBUG is defined then cogl-object-private.h will call
COGL_NOTE in the ref and unref macros. For this to work the debug
header needs to also be included or COGL_NOTE won't necessarily be
defined.

14 years agocookbook: Add recipe about sync'ing actor sizes
Elliot Smith [Thu, 16 Sep 2010 14:55:21 +0000 (15:55 +0100)]
cookbook: Add recipe about sync'ing actor sizes

The recipe covers how to use ClutterBindConstraint
to bind actor sizes together.

It gives some examples of where this approach is appropriate,
as well as explaining an alternative using allocation-changed
or notify::* signals.

Three examples are given:

1. Resizing a texture to the stage.
2. Resizing a rectangle to act as a transparent overlay on
top of a texture (using constraints).
3. Resizing a rectangle to act as a transparent overlay on
top of a texture, but with a size proportional to the texture
(using a handler connected to allocation-changed signals
emitted by the texture).

14 years agocookbook: Example using allocation-changed to sync actor size
Elliot Smith [Thu, 16 Sep 2010 12:42:09 +0000 (13:42 +0100)]
cookbook: Example using allocation-changed to sync actor size

An alternative method (not using constraints) to bind
one actor's size and position to another. Used as
an example in the recipe about resizing one actor in
sync with a source actor.

14 years agocookbook: Simple example to demonstrate bind constraint
Elliot Smith [Wed, 15 Sep 2010 15:05:15 +0000 (16:05 +0100)]
cookbook: Simple example to demonstrate bind constraint

A simple example showing how to scale an actor to the stage.

Demonstrates ClutterBindConstraint and ClutterAlignConstraint
in a fashion suitable for a short recipe.

14 years agocookbook: Example of using a bind constraint for an overlay
Elliot Smith [Wed, 15 Sep 2010 14:50:32 +0000 (15:50 +0100)]
cookbook: Example of using a bind constraint for an overlay

Example code which loads an image into a texture, and resizes
the image in response to +/- key presses. The overlay is
a transparent rectangle which is bound to the height and
width of the texture; on clicking the texture, the overlay
is made visible by increasing its opacity.

This demonstrates how to use constraints to simplify code
for resizing an actor which is "dependent" on another actor.

14 years agodocs: Add sub-classing notes on ActorMeta and Constraint
Emmanuele Bassi [Fri, 17 Sep 2010 13:54:31 +0000 (14:54 +0100)]
docs: Add sub-classing notes on ActorMeta and Constraint

14 years agoflow-layout: Blow the cached preferred size if needed
Emmanuele Bassi [Fri, 17 Sep 2010 11:43:23 +0000 (12:43 +0100)]
flow-layout: Blow the cached preferred size if needed

If the FlowLayout layout manager wasn't allocated the same size it
requested then it should blow its caches and recompute the layout
with the given allocation size.

14 years agoconstraint: Re-implement using update_allocation()
Emmanuele Bassi [Fri, 17 Sep 2010 11:13:29 +0000 (12:13 +0100)]
constraint: Re-implement using update_allocation()

Instead of using the fixed position and size API, use the newly added
update_allocation() virtual function in ClutterConstraint to change the
allocation of a ClutterActor. This allows using constraints inside
layout managers, and also allows Constraints to react to changes in the
size of an actor without causing relayout cycles.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2319

14 years agoconstraint: Add ::update_allocation()
Emmanuele Bassi [Fri, 17 Sep 2010 11:09:56 +0000 (12:09 +0100)]
constraint: Add ::update_allocation()

The Constraint should plug directly into the allocation mechanism, and
modify the allocation of the actor to which they are applied to. This is
similar to the mechanism used by the Effect class to modify the paint
sequence of an actor.

14 years agotest-bind: Fix colors and set the name of the boxes
Emmanuele Bassi [Fri, 17 Sep 2010 11:08:52 +0000 (12:08 +0100)]
test-bind: Fix colors and set the name of the boxes

14 years agobuild: Use the power of abs_builddir
Emmanuele Bassi [Fri, 17 Sep 2010 10:40:10 +0000 (11:40 +0100)]
build: Use the power of abs_builddir

When calling test-conformance to get the list of conformance tests.

14 years agobuild: Enable COGL_OBJECT_DEBUG in the debug flags
Emmanuele Bassi [Fri, 17 Sep 2010 10:39:20 +0000 (11:39 +0100)]
build: Enable COGL_OBJECT_DEBUG in the debug flags

The CoglObject debugging code is enabled by a separate flag.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2318

14 years agooffscreen-effect: Update handling of transforms
Robert Bragg [Thu, 16 Sep 2010 11:58:42 +0000 (12:58 +0100)]
offscreen-effect: Update handling of transforms

In line with the changes made in f5f066df9ce7 to clean up how Clutter
deals with transformations of actors this patch updates the code in
clutter-offscreen-effect.c. We now query the projection matrix from the
stage instead of the perspective and instead of duplicating the logic to
setup the stage view transform we now use
_clutter_actor_apply_modelview_transform for the stage instead.

14 years agoclutter.modules: Specify the checkoutdir for the gtk2 module
Neil Roberts [Thu, 16 Sep 2010 10:12:51 +0000 (11:12 +0100)]
clutter.modules: Specify the checkoutdir for the gtk2 module

If no checkoutdir is specified then jhbuild seems to use the name of
the module which in this case would be 'gtk+'. This ends up
overwriting the checkout of the master branch of gtk+ and causes all
kinds of build problems. This patch adds a checkoutdir attribute to
the gtk2 module to force it to checkout into the gtk2 directory.

14 years agoPost-release version bump to 1.3.15
Emmanuele Bassi [Wed, 15 Sep 2010 15:46:06 +0000 (16:46 +0100)]
Post-release version bump to 1.3.15

14 years agoRelease Clutter 1.3.14 (developers snapshot)
Emmanuele Bassi [Wed, 15 Sep 2010 15:12:39 +0000 (16:12 +0100)]
Release Clutter 1.3.14 (developers snapshot)

14 years agobuild: Add cogl-debug-options.h
Emmanuele Bassi [Wed, 15 Sep 2010 14:56:42 +0000 (15:56 +0100)]
build: Add cogl-debug-options.h

14 years agodocs: Fix the name of the parameters
Emmanuele Bassi [Wed, 15 Sep 2010 14:35:05 +0000 (15:35 +0100)]
docs: Fix the name of the parameters

14 years agocogl: Make cogl_util_next_p2 internal and fix the documentation
Neil Roberts [Wed, 15 Sep 2010 13:39:05 +0000 (14:39 +0100)]
cogl: Make cogl_util_next_p2 internal and fix the documentation

cogl_util_next_p2 is declared in cogl-util.h which is a private header
so it shouldn't be possible for an application to use it. It's
probably not a function we'd like to export from Cogl so it seems
better to keep it private. This patch renames it to _cogl_util_next_p2
so that it won't be exported from the shared library.

The documentation for the function is also slightly wrong because it
stated that the function returned the next power greater than
'a'. However the code would actually return 'a' if it's already a
power of two. I think the actual behaviour is more useful so this
patch changes the documentation rather than the code.

14 years agocogl-vertex-buffer: Don't always set COGL_MATERIAL_FLUSH_FALLBACK_MASK
Neil Roberts [Wed, 15 Sep 2010 13:07:30 +0000 (14:07 +0100)]
cogl-vertex-buffer: Don't always set COGL_MATERIAL_FLUSH_FALLBACK_MASK

Previously CoglVertexBuffer would always set the flush options flags
to at least contain COGL_MATERIAL_FLUSH_FALLBACK_MASK. The code then
later checks whether any flags are set before deciding whether to copy
the material to implement the overrides. This means that it would
always end up copying the material even if there are no fallback
layers. This patch changes it so that it only sets
COGL_MATERIAL_FLUSH_FALLBACK_MASK if fallback_layers != 0.

14 years agomaterial-arbfp: fix updating params if sharing progs
Robert Bragg [Tue, 14 Sep 2010 19:25:23 +0000 (20:25 +0100)]
material-arbfp: fix updating params if sharing progs

If a single arbfp program is being shared between multiple CoglMaterials
then we need to make sure we update all program.local params when
switching between materials. Previously we had a dirty flag to track
when combine_constant params were changed but didn't take in to account
that different materials sharing the same program may have different
combine constants.

14 years agomaterial-arbfp: Another pass at simplifying the code
Robert Bragg [Tue, 14 Sep 2010 17:50:50 +0000 (18:50 +0100)]
material-arbfp: Another pass at simplifying the code

Previously the backend private state was used to either link to an
authority material or provide authoritative program state. The mechanism
seemed overly complex and felt very fragile. I made a recent comment
which added a lot of documentation to make it easier to understand but
still it didn't feel very elegant.

This patch takes a slightly different approach; we now have a
ref-counted ArbfpProgramState object which encapsulates a single ARBfp
program and the backend private state now just has a single member which
is a pointer to one of these arbfp_program_state objects. We no longer
need to cache pointers to our arbfp-authority and so we can get rid of
a lot of awkward code that ensured these pointers were
updated/invalidated at the right times. The program state objects are
not tightly bound to a material so it will also allow us to later
implement a cache mechanism that lets us share state outside a materials
ancestry. This may help to optimize code not following the
recommendations of deriving materials from templates, avoiding one-shot
materials and not repeatedly modifying materials because even if a
material's ancestry doesn't naturally lead us to shareable state we can
fallback to searching for shareable state using central hash tables.

14 years agomaterial: Adds experimental cogl_material_foreach_layer API
Robert Bragg [Mon, 5 Jul 2010 20:33:26 +0000 (21:33 +0100)]
material: Adds experimental cogl_material_foreach_layer API

This adds a way to iterate the layer indices of the given material since
cogl_material_get_layers has been deprecated. The user provides a
callback to be called once for each layer.

Because modification of layers in the callback may potentially
invalidate any number of the internal CoglMaterialLayer structures and
invalidate the material's layer cache this should be more robust than
cogl_material_get_layers() which used to return a const GList *
pointing directly to internal state.

14 years agomaterial: don't declare backend vtables in headers
Robert Bragg [Tue, 14 Sep 2010 11:13:30 +0000 (12:13 +0100)]
material: don't declare backend vtables in headers

This fixes the material backends to declare their constant vtable in the
c file with a corresponding extern declaration in the header. This
should fix complaints about duplicate symbols seen on OSX.

14 years agomaterial-arbfp: don't recompile for constant changes
Robert Bragg [Mon, 13 Sep 2010 23:33:46 +0000 (00:33 +0100)]
material-arbfp: don't recompile for constant changes

Instead of lazily incorporating combine constants as arbfp PARAM
constants in the source directly we now use program.local parameters
instead so we can avoid repeating codegen if a material's combine
constant is updated. This should be a big win for applications animating
a constant used for example in an animated interpolation, such as
gnome-shell.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2280

14 years agomaterial-arbfp: don't redo codegen for texture changes
Robert Bragg [Mon, 13 Sep 2010 23:27:57 +0000 (00:27 +0100)]
material-arbfp: don't redo codegen for texture changes

This makes it so we don't consider LAYER_STATE_TEXTURE changes to affect
the arbfp code. This should avoid a lot of unneeded passes of
code generation for applications modifying the texture for a layer.

14 years agomaterial: make layer/material_pre_changes mutually exclusive
Robert Bragg [Mon, 13 Sep 2010 23:18:02 +0000 (00:18 +0100)]
material: make layer/material_pre_changes mutually exclusive

This makes it so we only notify backends of either a single material
change or a single layer change. Previously all material STATE_LAYERS
changes would be followed by a more detailed layer change.

For backends that perform code generation for fragment processing they
typically need to understand the details of how layers get changed to
determine if they need to repeat codegen. It doesn't help them to report
a material STATE_LAYERS change for all layer changes since it's so
broad, they really need to wait for the layer change to be notified.

What does help though is to report a STATE_LAYERS change for a change in
material->n_layers because they typically do need to repeat codegen in
that case.

14 years agomaterial-arbfp: fixes for how we track private state
Robert Bragg [Mon, 13 Sep 2010 22:52:18 +0000 (23:52 +0100)]
material-arbfp: fixes for how we track private state

This fixes a number of issues relating to how we track the arbfp private
state associated with CoglMaterials. At the same time it adds much more
extensive code documentation to try and make it a bit more approachable.

14 years agomaterial: pass material owner for layer pre changes
Robert Bragg [Mon, 13 Sep 2010 22:05:49 +0000 (23:05 +0100)]
material: pass material owner for layer pre changes

When notifying a backend about a layer being modified we now pass the
layers current owner for reference. NB: Although a layer can indirectly
be referenced by multiple layers, a layer is considered immutable once
it has dependants, so there is only ever one material associated with a
layer being modified. Passing the material pointer to the backends
layer_pre_change callback can be useful for backends that associate
their private state with materials and may need to update that state in
response to layer changes.

14 years agoarbfp: rename get_arbfp_authority clarifying semantics
Robert Bragg [Mon, 13 Sep 2010 21:56:44 +0000 (22:56 +0100)]
arbfp: rename get_arbfp_authority clarifying semantics

This renames the get_arbfp_authority function to
get_arbfp_authority_no_check to clarify that the function doesn't
validate that the authority cache is still valid by looking at the age
of the referenced material. The function should only be used when we
*know* the cache has already been checked.

14 years agomaterial: track if material change is layer change
Robert Bragg [Mon, 13 Sep 2010 20:26:31 +0000 (21:26 +0100)]
material: track if material change is layer change

We now pass a boolean to _cogl_material_pre_change_notify to know when
a material change is as a result of a layer change. We plan to use this
information to avoid notifying the backends about material changes if
they are as a result of layer changes. This will simplify the handling
of state changes in the backends because they can assume that layer and
material changes are mutually exclusive.

14 years agomaterial: Adds _get_layer_combine_constant API
Robert Bragg [Mon, 13 Sep 2010 20:21:09 +0000 (21:21 +0100)]
material: Adds _get_layer_combine_constant API

This adds an internal _cogl_material_get_layer_combine_constant function
so we can query the current layer combine constant back. We should
probably make this a public property getter, but for now we just need
this so we can read the constant in the arbfp backend.

14 years agomaterial: track unit state with arbfp private state
Robert Bragg [Mon, 13 Sep 2010 20:17:30 +0000 (21:17 +0100)]
material: track unit state with arbfp private state

We are going to start tracking more per-texture unit state with arbfp
private state so this adds an internal UnitState type and we allocate an
array of these when setting up a new private state structure. The first
thing that has been moved into this is the sampled boolean to know when
a particular texture unit gets sampled from in the generated arbfp code.

14 years agoEnumeration value should be on the same line
Emmanuele Bassi [Wed, 15 Sep 2010 10:56:59 +0000 (11:56 +0100)]
Enumeration value should be on the same line

The glib-mkenums script is not clever enough to deal with

  FLAGS_VALUE = FLAGS_A |
                FLAGS_B

And since this breaks the enumeration GType and the introspection data,
we cannot really wait for it to be fixed.

See: https://bugzilla.gnome.org/show_bug.cgi?id=629741

http://bugzilla.clutter-project.org/show_bug.cgi?id=2238

14 years agocogl-framebuffer: fix leak when popping framebuffer
Robert Bragg [Wed, 1 Sep 2010 14:15:20 +0000 (15:15 +0100)]
cogl-framebuffer: fix leak when popping framebuffer

We were using g_slist_remove_link instead of g_slist_delete_link
resulting in a memory leak. Thanks to Simon Lanzmich for
reporting this bug.

14 years agoInitialize the cogl uprof state in cogl_create_context
Robert Bragg [Mon, 16 Aug 2010 20:11:42 +0000 (21:11 +0100)]
Initialize the cogl uprof state in cogl_create_context

This avoids the use of of gcc constructor and destructor attributes to
initialize the cogl uprof context and optionally print a cogl uprof
report at app exit. We now initialize the uprof context in
cogl_context_create instead.

14 years agojournal: provide more detailed uprof instrumentation
Robert Bragg [Mon, 26 Jul 2010 14:21:18 +0000 (15:21 +0100)]
journal: provide more detailed uprof instrumentation

This adds more timing around key stages of the journal flushing process.

14 years agoprofile: Update to uprof-0.3 dep for --enable-profile
Robert Bragg [Mon, 21 Jun 2010 14:36:46 +0000 (15:36 +0100)]
profile: Update to uprof-0.3 dep for --enable-profile

When building with --enable-profile we now depend on the uprof-0.3
developer release which brings a few improvements:

» It lets us "fix" how we initialize uprof so that instead of using a shared
object constructor/destructor (which was a hack used when first adding
uprof support to Clutter) we can now initialize as part of clutter's
normal initialization code. As a side note though, I found that the way
Clutter initializes has some quite serious problems whenever it
involves GOptionGroups. It is not able to guarantee the initialization
of dependencies like uprof and Cogl. For this reason we still use the
contructor/destructor approach to initialize uprof in Cogl.

» uprof-0.3 provides a better API for adding custom columns when reporting
timer and counter statistics which lets us remove quite a lot of manual
report generation code in clutter-profile.c.

» uprof-0.3 provides a shared context for tracking mainloop timer
statistics. This means any mainloop based library following the same
"Mainloop" timer naming convention can use the shared context and no
matter who ends up owning the final mainloop the statistics will always
be in the same place. This allows profiling of Clutter with an
external mainloop such as with the Mutter compositor.

» uprof-0.3 can export statistics over dbus and comes with an ncurses
based ui to vizualize timer and counter stats live.

The latest version of uprof can be cloned from:
git://github.com/rib/UProf.git

14 years agocogl-framebuffer: Clear the renderbuffer list on failure
Neil Roberts [Mon, 6 Sep 2010 16:18:11 +0000 (17:18 +0100)]
cogl-framebuffer: Clear the renderbuffer list on failure

When try_creating_fbo fails it deletes any intermediate render buffers
that were created. However it doesn't clear the list so I think if it
failed a second time it would try to delete the render buffers
again. This could potentially cause problems if a subsequent fbo is
created because the destructor for the original might delete the
renderbuffers of the new fbo.

14 years agoclone: scale src with apply_transform not cogl_scale
Robert Bragg [Sat, 11 Sep 2010 01:31:03 +0000 (02:31 +0100)]
clone: scale src with apply_transform not cogl_scale

Since a ClutterClone may be allocated a different size than its source
actor we need to apply a scale factor before painting the source actor.
We were manually using cogl_scale to do this in clutter_clone_paint but
really this kind of thing is best handled in an implementation of the
apply_transform virtual so Clutter can be aware of the transform for
other purposes, such as input transformations. Also we want to provide
an implementation of the get_paint_volume virtual where Clutter will
also expect to be able to use the apply_transform virtual to transform
the volume into its parent's coordinate space.

14 years agoglx: Mark stage clip initialized even for NULL user clip
Robert Bragg [Wed, 8 Sep 2010 00:41:01 +0000 (01:41 +0100)]
glx: Mark stage clip initialized even for NULL user clip

If a NULL clip is passed to clutter_stage_glx_add_redraw_clip then we
update the redraw clip to have width of 0, but we weren't setting
stage_glx->initialized_redraw_clip = TRUE. This could result in a full,
unclipped stage redraw being reduced to a clipped redraw.

14 years agoactor: warn if allocating actor not descending from stage
Robert Bragg [Tue, 7 Sep 2010 21:02:23 +0000 (22:02 +0100)]
actor: warn if allocating actor not descending from stage

This adds a verbose warning that will be displayed if
clutter_actor_allocate is passed an actor that isn't a descendent of a
ClutterStage. Layouting should always bubble up from a stage so this
condition is likely to indicate a buggy container that allocating a
child that it has already unparented.

14 years agogeometry: Adds a clutter_geometry_intersects API
Robert Bragg [Tue, 7 Sep 2010 16:00:49 +0000 (17:00 +0100)]
geometry: Adds a clutter_geometry_intersects API

This adds a public function named clutter_geometry_intersects which
determines if two geometries intersect or not returning TRUE if so else
FALSE.

14 years agoactor-box: Correctly clamp to pixel
Emmanuele Bassi [Mon, 16 Aug 2010 15:02:53 +0000 (16:02 +0100)]
actor-box: Correctly clamp to pixel

Use ceilf() to round up the right side of the actor's box, and floorf()
to round down the left side.

14 years agoTry to clean up how we handle actor transformations
Robert Bragg [Tue, 7 Sep 2010 12:10:55 +0000 (13:10 +0100)]
Try to clean up how we handle actor transformations

When building actor relative transforms, instead of using the matrix
stack to combine transformations and making assumptions about what is
currently on the stack we now just explicitly initialize an identity
matrix and apply transforms to that.

This removes the full_vertex_t typedef for internal transformation code
and we just use ClutterVertex.

ClutterStage now implements apply_transform like any other actor now
and the code we had in _cogl_setup_viewport has been moved to the
stage's apply_transform instead.

ClutterStage now tracks an explicit projection matrix and viewport
geometry. The projection matrix is derived from the perspective whenever
that changes, and the viewport is updated when the stage gets a new
allocation. The SYNC_MATRICES mechanism has been removed in favour of
_clutter_stage_dirty_viewport/projection() APIs that get used when
switching between multiple stages to ensure cogl has the latest
information about the onscreen framebuffer.

14 years agoactor: Add _get_stage_internal to clutter-private.h
Robert Bragg [Tue, 7 Sep 2010 10:55:23 +0000 (11:55 +0100)]
actor: Add _get_stage_internal to clutter-private.h

This adds _clutter_actor_get_stage_internal to clutter-private.h since
we plan to use it in clutter-offscreen-effect when preparing to
redirect an actor offscreen.

14 years agoMerge remote branch 'elliot/cookbook-animations-reuse'
Emmanuele Bassi [Mon, 13 Sep 2010 14:51:18 +0000 (15:51 +0100)]
Merge remote branch 'elliot/cookbook-animations-reuse'

* elliot/cookbook-animations-reuse:
  cookbook: Added a recipe for reusing a complex animation
  cookbook: Added id for section in "rotating an actor" recipe
  cookbook: Simplified and clarified example code
  cookbook: Cleaned up the "animations reuse" example
  cookbook: Refactored reusable animation example
  cookbook: Added example for animation reuse recipe

14 years agoRemove trailin enum commas, avoiding C++ warnings.
Murray Cumming [Mon, 13 Sep 2010 14:16:25 +0000 (16:16 +0200)]
Remove trailin enum commas, avoiding C++ warnings.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2314

14 years agocookbook: Added a recipe for reusing a complex animation
Elliot Smith [Mon, 13 Sep 2010 13:29:37 +0000 (14:29 +0100)]
cookbook: Added a recipe for reusing a complex animation

This recipe explains how to "reuse" the same animation
definition for different actors, by creating a new
instance of a "rig" (empty container) and animation
for the rig each time the animation is required.

An actor is then re-parented to the rig and animated
using it, rather than being animated directly.

JSON is used to define the rig + animator, to make
creating new instances of them simpler. The recipe
also discusses various caveats around using this
approach, rather than directly animating an actor.

14 years agocookbook: Added id for section in "rotating an actor" recipe
Elliot Smith [Mon, 13 Sep 2010 09:28:16 +0000 (10:28 +0100)]
cookbook: Added id for section in "rotating an actor" recipe

Added an id to a section in the "rotating an actor" recipe,
so that it can be referred to from the "reusing an animation"
recipe.

14 years agocookbook: Simplified and clarified example code
Elliot Smith [Thu, 9 Sep 2010 11:02:02 +0000 (12:02 +0100)]
cookbook: Simplified and clarified example code

Modified the "animation reuse" sample code to provide
a simpler example to explain in the recipe.

Also modified variable names to mirror the names used
for the previous "complex animation" example and added
some more comments, to further simplify and support the
recipe.

14 years agocookbook: Cleaned up the "animations reuse" example
Elliot Smith [Wed, 1 Sep 2010 10:18:15 +0000 (11:18 +0100)]
cookbook: Cleaned up the "animations reuse" example

With some help from pippin, moved variable declarations
into more sensible positions within their functions,
changed a function name, and found a better way
to unref a script once its associated actor has
been destroyed.

14 years agocookbook: Refactored reusable animation example
Elliot Smith [Wed, 1 Sep 2010 09:21:23 +0000 (10:21 +0100)]
cookbook: Refactored reusable animation example

Extracted the animation into its own JSON definition,
then create a new script and get the animation each
time a rectangle is clicked.

Removes the need to reparent onto the background and
copy property values to the rectangle after the animation,
and generally much cleaner.

14 years agocookbook: Added example for animation reuse recipe
Elliot Smith [Tue, 31 Aug 2010 16:42:37 +0000 (17:42 +0100)]
cookbook: Added example for animation reuse recipe

Added an example showing how to reuse a ClutterAnimator
instance to animate multiple actors at different times
using an animatable rig, combined with reparenting.

14 years agocally: fixing public headers
Alejandro Piñeiro [Thu, 22 Jul 2010 11:19:34 +0000 (13:19 +0200)]
cally: fixing public headers

Cally headers were including specific clutter object headers.

This caused a problem including cally.h on a external program.

http://bugzilla.clutter-project.org/show_bug.cgi?id=2234

14 years agobuild: Add tesselator to the include paths
Emmanuele Bassi [Mon, 13 Sep 2010 10:31:46 +0000 (11:31 +0100)]
build: Add tesselator to the include paths

14 years agobuild: Generate ClutterJson in the top directory
Emmanuele Bassi [Mon, 13 Sep 2010 01:15:44 +0000 (02:15 +0100)]
build: Generate ClutterJson in the top directory

Instead of doing the shlib trick, build ClutterJson (if needed) inside
the top-level clutter/ directory - similar to a non-recursive layout.

Hopefully, one day, we'll be able to do this with a real non-recursive
layout.

14 years agobuild: List the units inside a file
Emmanuele Bassi [Mon, 13 Sep 2010 01:01:50 +0000 (02:01 +0100)]
build: List the units inside a file

This avoids trying to run test-conformance -l multiple times.

14 years agobuild: Remove gir files from dist
Emmanuele Bassi [Mon, 13 Sep 2010 01:01:21 +0000 (02:01 +0100)]
build: Remove gir files from dist

The introspection data is going to be generated, so it should never be
in the dist.

14 years agobuild: Do not build a noinst library for the tesselator
Emmanuele Bassi [Sun, 12 Sep 2010 18:25:47 +0000 (19:25 +0100)]
build: Do not build a noinst library for the tesselator

Let's try to keep Cogl's build as non-recursive as possible, in the hope
that one day we'll be able to make it fully non-recursive along with the
rest of Clutter.

14 years agocookbook: Use the new CLUTTER_KEY_* symbol constants
Emmanuele Bassi [Fri, 10 Sep 2010 17:21:45 +0000 (18:21 +0100)]
cookbook: Use the new CLUTTER_KEY_* symbol constants

Instead of the deprecated CLUTTER_* ones.

14 years agobuild: Disable deprecated API when building the cookbook examples
Emmanuele Bassi [Fri, 10 Sep 2010 17:21:19 +0000 (18:21 +0100)]
build: Disable deprecated API when building the cookbook examples

Make sure we always use the latest API.

14 years agokeysyms: Update the macros to CLUTTER_KEY_*
Emmanuele Bassi [Fri, 10 Sep 2010 12:41:49 +0000 (13:41 +0100)]
keysyms: Update the macros to CLUTTER_KEY_*

The keysyms defines in clutter-keysyms.h are generated from the X11 key
symbols headers by doing the equivalent of a pass of sed from XK_* to
CLUTTER_*. This might lead to namespace collisions, down the road.
Instead, we should use the CLUTTER_KEY_* namespace.

This commit includes the script, taken from GDK, that parses the X11
key symbols and generates two headers:

  - clutter-keysyms.h: the default included header, with CLUTTER_KEY_*
  - clutter-keysyms-compat.h: the compatibility header, with CLUTTER_*

The compat.h header file is included if CLUTTER_DISABLE_DEPRECATED is
not defined - essentially deprecating all the old key symbols.

This does not change any ABI and, assuming that an application or
library is not compiling with CLUTTER_DISABLE_DEPRECATED, the source
compatibility is still guaranteed.

14 years agointrospection: Add full path for the Clutter-1.0 inclusion
Lucas Rocha [Fri, 10 Sep 2010 16:48:12 +0000 (17:48 +0100)]
introspection: Add full path for the Clutter-1.0 inclusion

Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>
14 years agobuild: Disable deprecated API when building Clutter
Emmanuele Bassi [Fri, 10 Sep 2010 13:27:56 +0000 (14:27 +0100)]
build: Disable deprecated API when building Clutter

Make sure we don't use deprecated API internally by adding
CLUTTER_DISABLE_DEPRECATED to the AM_CPPFLAGS.

This requires adding -UCLUTTER_DISABLE_DEPRECATED to the introspection
scanner CFLAGS, otherwise the deprecated API will never be introspected
and the data generated will not be compatible.

14 years agoDo not use deprecated API internally
Emmanuele Bassi [Fri, 10 Sep 2010 13:27:37 +0000 (14:27 +0100)]
Do not use deprecated API internally

14 years agoUpdate the NEWS
Emmanuele Bassi [Fri, 10 Sep 2010 11:24:37 +0000 (12:24 +0100)]
Update the NEWS

14 years agoactor: Add a method for querying key focus
Emmanuele Bassi [Fri, 10 Sep 2010 10:42:11 +0000 (11:42 +0100)]
actor: Add a method for querying key focus

A simple convenience method on Clutter.Actor for checking whether
it has key focus assigned on the Stage to which it belongs.