[Inkscape-devel] Bugs, ,bugs, and more bugs: PVS Studio

Dear Inkscape Developers,

I would be careful with the this!=NULL statement for

bool SPLPEItem::performPathEffect

as far as I can tell, this is not a virtual function, and there is no reason why the this pointer of a non virtual function should not be NULL. One can call a non virtual member function on a NULL pointer. Not very nice, but some people put such checks into the code as extra safety check on purpose and some even use tricks like calling a function on NULL pointers on purpose, e.g. for saving a lot of checks when the function is called.

Best regards,

Michael