Do not modify parameters in place
authorEmmanuele Bassi <ebassi@gnome.org>
Wed, 10 Dec 2008 23:00:48 +0000 (23:00 +0000)
committerEmmanuele Bassi <ebassi@gnome.org>
Wed, 10 Dec 2008 23:00:48 +0000 (23:00 +0000)
commitbc6a107ff6b3b7a6cd5c314c25894327b68dc955
treedf9c2c7484c0dd1fcbde3c01c8e3069cf1eb0ba0
parentdc6545ccd1dcf63ae9981b4c2171eb74d704183f
Do not modify parameters in place

When the apply_transform_to_point() and its relative variant
landed in Clutter 0.3, the initial approach was to modify the
passed vertex as an in-out parameter. This was later dropped
in favour of a more consistent out parameter.

Unfortunately, the implementation never changed: both methods
where modifying the passed vertex with the partial results of
the computations.

This commit copies the contents of the "point" ClutterVertex
argument inside a stack variable and, for good measure, constifies
the argument.

Thanks to Thomas Steinacher for catching this in Python.
clutter/clutter-actor.c
clutter/clutter-actor.h