[Pcb-bugs] [Bug 1654884] Re: rubberband

[Chad P. <par...@gm...>]

Confirmed on 4.1.0

** Changed in: pcb
   Importance: Undecided => Medium

** Changed in: pcb
       Status: New => Confirmed

-- 
You received this bug notification because you are a member of PCB Bug
Team, which is subscribed to pcb.
https://bugs.launchpad.net/bugs/1654884

Title:
  rubberband

Status in pcb:
  Confirmed

Bug description:
  On November 24th, 2016, Igor2 wrote to the geda-user mailing list:

  Hi all,

  I managed to separate the rubber band code and moved it all in an optional
  feature plugin in pcb-rnd. After that, we started to test whether I
  broke something in the cleanup and Evan found an interesting bug that
  affects mainline too.

  REMINDER:

  we are having a test sprint on Saturday, to hunt & fix bugs like the
  below one.

  Event summary: http://repo.hu/cgi-bin/pcb-rnd-
  people.cgi?event=tsprint3

  Join us on IRC (any time): http://repo.hu/projects/pcb-rnd/irc.html

  BUGREPORT:

  It is easy to reproduce with the attached minimal board. Grab the top
  45 degree short line and move it (rubber band is on). Make sure you
  grab the line, not its endpoints - this can be done by clicking near
  the middle of the 45 degree edge of the line (zoom in!). Also make
  sure the line is not selected.

  Once the line segment is grabbed, there's no xor-drawn attached-lines
  and when the operation ends, the endpoints of the attached lines are
  thrown 2x further than necessary.

  Repeat the same with the bottom 45 degree line segment and it won't
  happen there.

  The only difference is that the top line is short while the bottom
  line is long. Because of a bug in rubber_callback() this results in
  finding and adding the same objects twice; this cancels in the xor-
  draw and multiplies the endpoint moves.

  The fix is commited in pcb-rnd r5265.

  Regards,

  Igor2

To manage notifications about this bug go to:
https://bugs.launchpad.net/pcb/+bug/1654884/+subscriptions