U++ forum: Welcome to the forum

Re: Bug: Missing width in DrawArc on X11 [message #35819 is a reply to message #35763]

Fri, 23 March 2012 20:44

Tom1
Messages: 1303
Registered: March 2007

Ultimate Contributor

Hi,

I agree with you: There should definitely be a SetLineStyle() call in the beginning of DrawArcOp() in file DrawOpX11.cpp as follows:

void SystemDraw::DrawArcOp(const Rect& rc, Point start, Point end, int width, Color color)
{
	GuiLock __;
	SetLineStyle(width);     // This needs to be added
	XGCValues gcv, gcv_old;
	XGetGCValues(Xdisplay, GetGC(), GCForeground, &gcv_old);
	Point offset = GetOffset();
        ...

Hope someone in the core team commits this soon. (I guess the DrawArc is a very little used functionality -- at least on X11 platforms -- since nobody has discovered and fixed it before.)

The SetLineStyle() should not be needed in applications since all drawing primitives should call it as needed for desired results.

Best regards,

Tom

Report message to a moderator