On 2011-06-25 8:05, Campbell Barton wrote:
Hi, I was trying to compile inkscape with clang, and for many reasons this failed, however one looks like it should probably be fixed in the code.
src/ui/widget/registered-widget.h
operator const Gtk::Widget () { return dynamic_castGtk::Widget*(this); }
Should either be: operator const Gtk::Widget () { return *dynamic_castGtk::Widget*(this); } Or: operator const Gtk::Widget () { return dynamic_castGtk::Widget(this);
I'm actually a little surprised that it returns a COPY of a Gtk::Widget. It might be more appropriate to return a Gtk::Widget pointer. But in any case I'm a little surprised that this isn't caught using gcc, which makes we wonder whether this is used at all.
BTW, insanely cool that you're trying to use clang to build Inkscape, I'd love to know if and when you succeed.