Having a quick peek at the code, probably
SPDocument *doc = SP_ACTIVE_DOCUMENT;
would fix it. I do not understand how the code works at this moment :S sp_document_repr_doc(doc) is a macro and expands to "doc->rdoc"...
Regards, Johan
________________________________
From: inkscape-devel-bounces@lists.sourceforge.net [mailto:inkscape-devel-bounces@lists.sourceforge.net] On Behalf Of Jon A. Cruz Sent: woensdag 4 april 2007 9:52 To: Inkscape Devel List Subject: Re: [Inkscape-devel] Warning in gradient-selector.cpp
On Apr 3, 2007, at 11:47 PM, Bob Jamison wrote:
src/widgets/gradient-selector.cpp, line 302:
static void
sp_gradient_selector_add_vector_clicked (GtkWidget *w,
SPGradientSelector *sel)
{
SPDocument *doc;
SPGradient *gr;
Inkscape::XML::Node *repr;
All other issues aside, this is one of the points where I *really* push for not leaving uninitialized variables.
sp_gradient_selector_add_vector_clicked (GtkWidget *w, SPGradientSelector *sel) { SPDocument *doc = 0; SPGradient *gr = 0; Inkscape::XML::Node *repr = 0;
Of course, that would cause things to crash each and every time that code is hit, but that's better than getting random effects and crashes later on. Pushes us to get it fixed right, too.
This case it doesn't quite address Bob's immediate question, but it is a good chance to ask people to keep an eye out and fix uninitialized things as they go near them.