Hi Marciej, I also think the easiest way to trigger/check this bug is applying my patch and adding a breakpoint after the following line to inspect the internal state of the zone:
if( aSourceItem.Type() == PCB_ZONE_AREA_T ) But this can be done locally for testing purpose, and my patch shouldn't be merged into the official repo until a bugfix for the zone behavior is made. The rendering issues are created by the capture program, so nothing to worry about. I need to find a better program in the future. Regards, Thomas Am 2017-02-27 um 18:13 schrieb Maciej Sumiński: > Hi Thomas, > > I see your point, but before we merge your patch, we need to address the > other problem first. I would like to see a bug report, but I guess this > behavior is hard to trigger without applying your patch first. > > BTW. are the rendering glitches visible in the linked video occurring in > KiCad as well? > > Regards, > Orson > > On 02/24/2017 12:28 AM, Thomas Pointhuber wrote: >> Hy, >> >> I think the "select Copper" and "select Net" feature should also select >> zones, and not only track's and via's. >> >> When thinking about a user which doesn't know the behavior of the >> "select Copper" feature, it should be the expected assumption that this >> feature is really selecting the interconnected copper area, and not only >> interconnected track's and via's. >> >> This patch adds this behavior. >> >> Unfortunately the patch is not working in all cases at the moment, due >> to a regression which triggers an assert, when someone want's to "select >> Copper" on a zone which is not filled but the internal state say's >> other. It seems IsFilled return's true in some cases where the zone is >> actually not filled. >> >> I created a video which I illustrates such a case where the variable is >> not set as expected: https://owncloud.oe5tpo.com/index.php/s/YxqNpq4NxebmJGW >> >> Regards, Thomas >> >> >> >> _______________________________________________ >> Mailing list: https://launchpad.net/~kicad-developers >> Post to : [email protected] >> Unsubscribe : https://launchpad.net/~kicad-developers >> More help : https://help.launchpad.net/ListHelp >> > > > > _______________________________________________ > Mailing list: https://launchpad.net/~kicad-developers > Post to : [email protected] > Unsubscribe : https://launchpad.net/~kicad-developers > More help : https://help.launchpad.net/ListHelp
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
