kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #15870
Re: [PATCH] Crash with zero size pads in module editor
Your patch has been applied in the product branch r5291. Thank you for
your contribution to KiCad.
Regards,
Wayne
On 11/18/2014 10:52 PM, Blair Bonnett wrote:
> Currently, the module editor crashes if the pad size is set to zero
> (e.g., while backspacing the current value to replace it). Note that
> this also happens if the entry is cleared as that is also treated as
> zero. Basically, the previewer tries to set the drawing context scale to
> zero and the drawing library doesn't like this.
>
> Also, the editor accepts zero as a pad size for surface mount pads.
>
> The attached patch adds checks for the zero entry case and sets the
> scale to (what I think is) a sensible default -- it uses the drill size
> rather than the pad size, or if the drill size is also zero, it
> arbitrarily uses a size of 1mm. It also adds an error message for
> zero-size pads when checking if the dialog is valid before accepting it.
>
> This closes lp:1378917.
>
> Regards,
> Blair
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
Follow ups
References