isa/mutter
1
0
Fork 0
mirror of https://github.com/brl/mutter.git synced 2024-09-20 06:25:51 -04:00
Code Issues Packages Projects Releases Wiki Activity
d6d208da7d
mutter/clutter
History
Emmanuele Bassi d6d208da7d Remove Units from the public API 
With the recent change to internal floating point values, ClutterUnit
has become a redundant type, defined to be a float. All integer entry
points are being internally converted to floating point values to be
passed to the GL pipeline with the least amount of conversion.

ClutterUnit is thus exposed as just a "pixel with fractionary bits",
and not -- as users might think -- as generic, resolution and device
independent units. not that it was the case, but a definitive amount
of people was convinced it did provide this "feature", and was flummoxed
about the mere existence of this type.

So, having ClutterUnit exposed in the public API doubles the entry
points and has the following disadvantages:

  - we have to maintain twice the amount of entry points in ClutterActor
  - we still do an integer-to-float implicit conversion
  - we introduce a weird impedance between pixels and "pixels with
    fractionary bits"
  - language bindings will have to choose what to bind, and resort
    to manually overriding the API
    + *except* for language bindings based on GObject-Introspection, as
      they cannot do manual overrides, thus will replicate the entire
      set of entry points

For these reason, we should coalesces every Actor entry point for
pixels and for ClutterUnit into a single entry point taking a float,
like:

  void clutter_actor_set_x (ClutterActor *self,
                            gfloat        x);
  void clutter_actor_get_size (ClutterActor *self,
                               gfloat       *width,
                               gfloat       *height);
  gfloat clutter_actor_get_height (ClutterActor *self);

etc.

The issues I have identified are:

  - we'll have a two cases of compiler warnings:
    - printf() format of the return values from %d to %f
    - clutter_actor_get_size() taking floats instead of unsigned ints
  - we'll have a problem with varargs when passing an integer instead
    of a floating point value, except on 64bit platforms where the
    size of a float is the same as the size of an int

To be clear: the *intent* of the API should not change -- we still use
pixels everywhere -- but:

  - we remove ambiguity in the API with regard to pixels and units
  - we remove entry points we get to maintain for the whole 1.0
    version of the API
  - we make things simpler to bind for both manual language bindings
    and automatic (gobject-introspection based) ones
  - we have the simplest API possible while still exposing the
    capabilities of the underlying GL implementation
2009-05-06 16:44:47 +01:00
..
cogl Unnecessary glColorMask on alpha drops performance 2009-04-24 15:26:32 +01:00
eglnative Remove Units from the public API 2009-05-06 16:44:47 +01:00
eglx Enforce invariants on mapped, realized, visibility states 2009-04-24 15:27:19 +01:00
fruity Remove Units from the public API 2009-05-06 16:44:47 +01:00
glx Enforce invariants on mapped, realized, visibility states 2009-04-24 15:27:19 +01:00
json [json] Avoid leaks on error code paths 2009-04-15 12:18:55 +01:00
osx Remove Units from the public API 2009-05-06 16:44:47 +01:00
pango [cogl-pango-render] Fix returning to default color after a color attribute 2009-04-06 11:14:43 +01:00
sdl Remove Units from the public API 2009-05-06 16:44:47 +01:00
win32 Remove Units from the public API 2009-05-06 16:44:47 +01:00
x11 Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-actor.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-actor.h Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-alpha.c [alpha] Sanitize easing equations 2009-04-23 11:35:10 +01:00
clutter-alpha.h Merge branch 'cogl-float' 2009-01-20 18:47:50 +00:00
clutter-animatable.c Add license notice to ClutterAnimation files 2009-01-08 13:18:00 +00:00
clutter-animatable.h Add license notice to ClutterAnimation files 2009-01-08 13:18:00 +00:00
clutter-animation.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-animation.h [animation] Add the ::started signal 2009-04-03 14:19:20 +01:00
clutter-backend.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-backend.h [backend] Constify font options 2009-02-26 15:32:48 +00:00
clutter-behaviour-depth.c Fix compiler warnings 2009-01-23 13:08:46 +00:00
clutter-behaviour-depth.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-behaviour-ellipse.c [docs] Remove ClutterFixed from the documentation 2009-03-10 12:38:04 +00:00
clutter-behaviour-ellipse.h Move ClutterBehaviourEllipse to CoglFixed 2009-03-10 12:38:03 +00:00
clutter-behaviour-opacity.c [animation] Move the alpha value to floating point 2009-01-20 16:42:49 +00:00
clutter-behaviour-opacity.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-behaviour-path.c Add gobject-introspection annotations 2009-02-20 12:09:07 +00:00
clutter-behaviour-path.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-behaviour-rotate.c Move ClutterBehaviourRotate to CoglFixed 2009-03-10 12:38:03 +00:00
clutter-behaviour-rotate.h Move ClutterBehaviourRotate to CoglFixed 2009-03-10 12:38:03 +00:00
clutter-behaviour-scale.c Move BehaviourScale to CoglFixed 2009-03-10 12:38:03 +00:00
clutter-behaviour-scale.h Move BehaviourScale to CoglFixed 2009-03-10 12:38:03 +00:00
clutter-behaviour.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-behaviour.h [animation] Move the alpha value to floating point 2009-01-20 16:42:49 +00:00
clutter-bezier.c [Automatic fixed-to-float.sh change] Applies a number fixed to float patches 2009-01-20 16:20:54 +00:00
clutter-bezier.h Bug 1252 - Merge ClutterBehaviourPath and ClutterBehaviourBspline 2008-12-05 13:13:37 +00:00
clutter-binding-pool.c Add gobject-introspection annotations 2009-02-20 12:09:07 +00:00
clutter-binding-pool.h Add declaration of clutter_binding_pool_get_type() 2009-01-14 15:03:30 +00:00
clutter-cairo-texture.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-cairo-texture.h [docs] Documentation warnings 2009-01-09 14:26:35 +00:00
clutter-child-meta.c Add gobject-introspection annotations 2009-02-20 12:09:07 +00:00
clutter-child-meta.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-clone.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-clone.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-color.c Remove usage of the grave accent as quotation mark 2009-03-17 14:13:31 +00:00
clutter-color.h Clean up of the ClutterColor API 2009-03-10 12:38:02 +00:00
clutter-container.c Add ClutterContainer::foreach_with_internals() 2009-04-24 15:25:08 +01:00
clutter-container.h Add ClutterContainer::foreach_with_internals() 2009-04-24 15:25:08 +01:00
clutter-debug.h Add debug annotations for the layout process 2009-04-15 17:25:50 +01:00
clutter-deprecated.h Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-enum-types.c.in 2007-12-25 Emmanuele Bassi <ebassi@openedhand.com> 2007-12-25 11:41:10 +00:00
clutter-enum-types.h.in 2007-08-20 Emmanuele Bassi <ebassi@openedhand.com> 2007-08-20 20:16:15 +00:00
clutter-event.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-event.h Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-feature.c 2008-06-06 Matthew Allum <mallum@openedhand.com> 2008-06-06 13:44:22 +00:00
clutter-feature.h 2008-10-30 Emmanuele Bassi <ebassi@linux.intel.com> 2008-10-30 17:04:34 +00:00
clutter-fixed.c [docs] Remove ClutterFixed from the documentation 2009-03-10 12:38:04 +00:00
clutter-fixed.h [fixed] Remove ClutterFixed API 2009-03-10 12:38:04 +00:00
clutter-frame-source.c Count timeline frames using the FPS instead of an integer interval 2009-03-16 15:40:18 +00:00
clutter-frame-source.h Count timeline frames using the FPS instead of an integer interval 2009-03-16 15:40:18 +00:00
clutter-group.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-group.h [group] avoid double evaluation of arguments in clutter_group_add 2009-04-08 18:19:46 +01:00
clutter-id-pool.c 2008-03-09 Matthew Allum <mallum@openedhand.com> 2008-03-09 19:07:17 +00:00
clutter-id-pool.h 2008-10-30 Emmanuele Bassi <ebassi@linux.intel.com> 2008-10-30 17:04:34 +00:00
clutter-interval.c [interval] Update fixed-point handling 2009-03-10 12:38:03 +00:00
clutter-interval.h [animation] Allow registering custom progress function 2009-01-08 11:15:09 +00:00
clutter-json.h.in 2008-04-23 Emmanuele Bassi <ebassi@openedhand.com> 2008-04-23 13:58:58 +00:00
clutter-keysyms-table.h 7 small miscellaneous fixes 2008-06-10 06:37:46 +00:00
clutter-keysyms.h 2008-08-04 Emmanuele Bassi <ebassi@openedhand.com> 2008-08-05 10:54:40 +00:00
clutter-list-model.c [list-model] Make ClutterListModel subclassable 2009-02-14 11:31:00 +00:00
clutter-list-model.h [list-model] Make ClutterListModel subclassable 2009-02-14 11:31:00 +00:00
clutter-main.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-main.h Count timeline frames using the FPS instead of an integer interval 2009-03-16 15:40:18 +00:00
clutter-marshal.list [animation] Move the alpha value to floating point 2009-01-20 16:42:49 +00:00
clutter-master-clock.c [timeline] Expose the msec advancement 2009-05-01 15:05:51 +01:00
clutter-master-clock.h Use a single master "clock" to drive timelines 2009-04-24 15:28:15 +01:00
clutter-media.c [media] Add sub-second resolution for the duration 2009-02-24 16:22:02 +00:00
clutter-media.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-model-private.h 2008-01-09 Emmanuele Bassi <ebassi@openedhand.com> 2008-01-09 14:18:53 +00:00
clutter-model.c Remove usage of the grave accent as quotation mark 2009-03-17 14:13:31 +00:00
clutter-model.h 2008-10-30 Emmanuele Bassi <ebassi@linux.intel.com> 2008-10-30 17:04:34 +00:00
clutter-path.c Prevent a possible zero division 2009-04-16 17:16:31 +01:00
clutter-path.h [build] Replaces use of #include <cairo/cairo.h> with #include <cairo.h> 2008-12-21 21:52:50 +00:00
clutter-private.h Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-rectangle.c Changes cogl_rectangle to take x1, y1, x2, y2 args not x1, y1, width, height 2009-01-28 17:20:15 +00:00
clutter-rectangle.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-score.c Remove usage of the grave accent as quotation mark 2009-03-17 14:13:31 +00:00
clutter-score.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-script-parser.c 2008-08-04 Emmanuele Bassi <ebassi@openedhand.com> 2008-08-04 16:21:27 +00:00
clutter-script-private.h 2008-08-04 Emmanuele Bassi <ebassi@openedhand.com> 2008-08-04 16:21:27 +00:00
clutter-script.c Remove usage of the grave accent as quotation mark 2009-03-17 14:13:31 +00:00
clutter-script.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-scriptable.c Intern the ClutterScriptable type name 2009-02-26 14:57:58 +00:00
clutter-scriptable.h 2008-10-30 Emmanuele Bassi <ebassi@linux.intel.com> 2008-10-30 17:04:34 +00:00
clutter-shader-types.c Remove usage of the grave accent as quotation mark 2009-03-17 14:13:31 +00:00
clutter-shader-types.h Do not use GL types in Clutter headers 2009-02-20 11:47:14 +00:00
clutter-shader.c [shader] Remove deprecated set_uniform_1f() 2009-04-03 12:47:21 +01:00
clutter-shader.h [shader] Remove deprecated set_uniform_1f() 2009-04-03 12:47:21 +01:00
clutter-stage-manager.c Add gobject-introspection annotations 2009-02-20 12:09:07 +00:00
clutter-stage-manager.h 2008-10-30 Emmanuele Bassi <ebassi@linux.intel.com> 2008-10-30 17:04:34 +00:00
clutter-stage-window.c 2008-04-04 Emmanuele Bassi <ebassi@openedhand.com> 2008-04-04 15:02:11 +00:00
clutter-stage-window.h 2008-10-30 Emmanuele Bassi <ebassi@linux.intel.com> 2008-10-30 17:04:34 +00:00
clutter-stage.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-stage.h Allow passing the pick mode to get_actor_at_pos() 2009-04-24 15:25:58 +01:00
clutter-text.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-text.h [clutter-text] Add a separate property for the selection color 2009-04-06 15:22:31 +01:00
clutter-texture.c Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-texture.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-timeline.c [docs] Document the destructor for Timeline 2009-05-06 16:42:57 +01:00
clutter-timeline.h [timeline] Expose the msec advancement 2009-05-01 15:05:51 +01:00
clutter-timeout-interval.c Count timeline frames using the FPS instead of an integer interval 2009-03-16 15:40:18 +00:00
clutter-timeout-interval.h Count timeline frames using the FPS instead of an integer interval 2009-03-16 15:40:18 +00:00
clutter-timeout-pool.c [docs] Fix naming of the interval parameter 2009-03-16 16:56:51 +00:00
clutter-timeout-pool.h Count timeline frames using the FPS instead of an integer interval 2009-03-16 15:40:18 +00:00
clutter-types.h Remove Units from the public API 2009-05-06 16:44:47 +01:00
clutter-units.c Remove usage of the grave accent as quotation mark 2009-03-17 14:13:31 +00:00
clutter-units.h [docs] Bring down the undocumented symbols to 2% 2009-03-16 15:00:55 +00:00
clutter-util.c * clutter/clutter-texture.c: (texture_get_tile_pixbuf), 2008-02-12 17:17:52 +00:00
clutter-util.h 2008-10-30 Emmanuele Bassi <ebassi@linux.intel.com> 2008-10-30 17:04:34 +00:00
clutter-version.h.in 2008-03-31 Matthew Allum <mallum@openedhand.com> 2008-03-31 17:15:02 +00:00
clutter.h Add more deprecation notices 2009-03-13 14:29:27 +00:00
Makefile.am Use a single master "clock" to drive timelines 2009-04-24 15:28:15 +01:00