← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Remove dangling ends when switching sheets

 

Seth,

I'm still not convinced that this was necessary but since it only gets
called on find when entering a different sheet, it's low overhead so I
committed your patch.

Thanks,

Wayne

On 12/10/2017 10:02 PM, Seth Hillbrand wrote:
> Find
> ​can
>  warp you to a new sheet
> ​ if the string matches a component that isn't on the current sheet​
> . If that sheet hasn't
> ​yet ​
> been shown
> ​ during the session
> , the dangling ends
> ​have
>  not
> ​ been​
> calculated
> ​, so we force it here.​
> ​
> ​Best-
> Seth​
> 
> 
> On Dec 10, 2017 5:44 PM, "Wayne Stambaugh" <stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>> wrote:
> 
>     Seth,
> 
>     Why is it necessary to test for dangling end on a find?  Find only
>     searches for string matches so I'm having a hard time understanding why
>     the dangling ends need to be cleaned up.
> 
>     Cheers,
> 
>     Wayne
> 
>     On 12/09/2017 11:45 PM, Seth Hillbrand wrote:
>     > ​This patch fixes https://bugs.launchpad.net/kicad/+bug/1672381
>     <https://bugs.launchpad.net/kicad/+bug/1672381>
>     > <https://bugs.launchpad.net/kicad/+bug/1672381
>     <https://bugs.launchpad.net/kicad/+bug/1672381>>
>     >
>     > When warping to a new sheet from either a "Find" command or
>     receiving a
>     > cross-probe request, Eeschema needs to check for dangling ends on
>     the sheet.
>     >
>     > -S​
>     >
>     >
>     >
>     > _______________________________________________
>     > Mailing list: https://launchpad.net/~kicad-developers
>     <https://launchpad.net/~kicad-developers>
>     > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>     > Unsubscribe : https://launchpad.net/~kicad-developers
>     <https://launchpad.net/~kicad-developers>
>     > More help   : https://help.launchpad.net/ListHelp
>     <https://help.launchpad.net/ListHelp>
>     >
> 
>     _______________________________________________
>     Mailing list: https://launchpad.net/~kicad-developers
>     <https://launchpad.net/~kicad-developers>
>     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>     Unsubscribe : https://launchpad.net/~kicad-developers
>     <https://launchpad.net/~kicad-developers>
>     More help   : https://help.launchpad.net/ListHelp
>     <https://help.launchpad.net/ListHelp>
> 
> 


References