[Inkscape-devel] My second patch and a general question about inkscape development

My second bug fix is available now at https://bugs.launchpad.net/inkscape/+bug/180912 it builds and works just fine but I have one question. I had to include "event-context.h" in "verbs.cpp" and I'm not sure if this coupling is wise to do. Any comments on that?

I didn't want to move the special toggling feature for the dropper tool to verbs.cpp mainly due to the space toggling to "select tool" and back would still be in event-context.cpp.

During compilation there are a few "unused variable" warnings and other kinds of warnings as well. Within the Inkscape project do you usually file a bug report against this (for keeping track of stuff) and then fix the warning(s) or just fix it?

Regards