kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #37477
Re: [PATCH] pcbnew: add option to set custom drill mark size during print
On Donnerstag, 13. September 2018 05:01:30 CEST Seth Hillbrand wrote:
> Hi Zoltan-
>
> Thank you for your contribution. As a quick note on our status right now,
> we are focusing on the 5.1 branch that is moving eeschema to our underlying
> GAL framework along with dialog revisions. After that is released, we'll
> begin accepting new features such as your patch.
>
> The patch looks like a nice feature. To be accepted, we'll need it to
> follow KiCad coding policy (
> https://kicad-source-mirror.readthedocs.io/en/stable/Documentation/developme
> nt/coding-style-policy/) consistently. There are a few places where you are
> missing spacing around commas or parentheses. Also, please ensure that you
> do not use "magic" numbers, e.g. 10000000. Instead use constexpr with
> well-named variable and comment for units.
>
> I would suggest making these revisions and attaching the patch to the
> associated bug report. This will ensure that we don't lose track between
> now and when v6 development opens.
>
> Best-
> Seth
>
> Am So., 9. Sep. 2018 um 11:11 Uhr schrieb Zoltan Gyarmati <
>
> zgyarmati@xxxxxxxxxxxx>:
> > To help manaul drilling, the print dialog provides option to
> > print small drill marks.
> > This patch adds one more option to set custom drill mark size,
> > while keeping backward compability with previously saved
> > configuration values.
> >
> > Fixes: lp:1287362
> > https://bugs.launchpad.net/kicad/+bug/1287362
> >
> > Signed-off-by: Zoltan Gyarmati <zgyarmati@xxxxxxxxxxxx>
> > ---
> >
> > pcbnew/dialogs/dialog_print_using_printer.cpp | 22 +-
> > .../dialog_print_using_printer_base.cpp | 36 +-
> > .../dialog_print_using_printer_base.fbp | 541 ++++++++++++++++--
> > .../dialogs/dialog_print_using_printer_base.h | 13 +-
> > pcbnew/pcbplot.h | 1 +
> > pcbnew/print_board_functions.cpp | 21 +-
> > pcbnew/printout_controler.h | 4 +-
> > 7 files changed, 584 insertions(+), 54 deletions(-)
> >
> > _______________________________________________
> > 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
Dear Seth&All,
thanks for the review! I'll give this one more iteration, and attach the patch
to the bug report in the next days.
BR,
--
Zoltan Gyarmati
https://zgyarmati.de
Attachment:
signature.asc
Description: This is a digitally signed message part.
Follow ups
References