In oprofile, I was seeing _cairo_fixed_from_double show up in the list. The
functions in cairo-fixed.c are all too short to deserve being functions
themselves. Given that cairo's fixed point will always be 16.16 or things will
break anyway, I think they should either be just inlined everywhere they are
used, or converted to macros.
The attached patch takes the macro approach.
Created attachment 3159 [details]
Working log with i810_drv.o and 6.8.2
Move bugs against "cvs" version to "0.9.3" so we can remove the "cvs" version.
First, the attachement seems to point to a X server log in some other bug (I guess this is due some mishap the bugzilla went over?).
Secondly, Cairo is moving to 24.8 internal representation and according to http://lists.freedesktop.org/archives/cairo/2007-June/010803.html, the fixed-point functions should be inlined along that change.
The changes are targeted for 1.6 I think, and should be merged to git soonish. This bug can be closed then.
(In reply to comment #3)
> First, the attachement seems to point to a X server log in some other bug (I
> guess this is due some mishap the bugzilla went over?).
Yeah, there was some machine disaster at some point, (a compromise followed by a rebuild or something?), that left all of the bugzilla attachments corrupted.
-Carl (who'd love to have a nice git-integrated bug tracking tool...)
Moved to todo.