drawpixels icon indicating copy to clipboard operation
drawpixels copied to clipboard

Published 20 hours ago • verified publisher icon AGulev

Duplicated structure

Open krasnikov-andrew opened this issue 6 years ago • 2 comments

https://github.com/koorva/drawpixels/blob/ec2fcd29e30502f48a23a16d18f21619631c4a4f/drawpixels/src/drawpixels.cpp#L26-L36 https://github.com/koorva/drawpixels/blob/ec2fcd29e30502f48a23a16d18f21619631c4a4f/drawpixels/src/drawpixels.cpp#L467-L475 #7 I am not sure that its right. Maybe we should replace Point by FloatPoint?

krasnikov-andrew avatar Sep 11 '18 14:09 krasnikov-andrew

Need to look into the code, I'll try to find time on the week. Let's try to ping @britzl, maybe he wants to look into it too.

AGulev avatar Sep 19 '18 09:09 AGulev

I looked into it, and I think it looks good. If you need Float Point, why not?

AGulev avatar Sep 19 '18 10:09 AGulev