Suggested Performance Improvement

[Eric Robishaw]

If a line style.width = 0, you should not draw the border.

So for every DrawBorder method, do:
if (LineStyle.BorderWidth >0)
{

}

Same for the Node's DrawInterior() method regarding FillStyles:

if (FillStyle.Type == Solid && (FillStyle.BackColor == Color.Transparent || FillStyle.BackColor == Color.None))
{
//do not DrawInterior
}

Otherwise, for empty backgrounds you're creating a brush and doing a FillPath for NO reason.

**** IDEALLY *****
You should really add a "Visible" attribute to both FillStyle and LineStyle classes. Then, check first if Visible before doint DrawBorder or DrawInterior actions.
*************************************
In many cases, this simple, very easy to implement change would produce significant performance improvements.
*************************************

ALSO: you need to make DrawShading a protected method. Otherwise, if I create a derived class of say Ellipse Node, and want to implement my own Render method, or my own DrawInterior method, I have to implement my own shading, b/c your DrawShading method is private!


Thanks
Eric

[Ajeet M]

Hi Eric,

Thanks for this important update. I have logged this as a defect report and we shall start to work on this shortly.

Let me know if you shall need any further clarifications in this regard.

Warm Regards,
Ajeet

[Administrator]

Hi Eric,

The defect has been fixed in our latest codebase, and if you would like to receive a solution, then please contact Syncfusion Support (email: [e-mail]) as we cannot provide defect fixes in a public forum.

Regards,
Dinesh