← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] pcbnew: add option to set custom drill mark size during print

 

Hi Zoltan,

I should have mentioned this earlier, but I am working on a large
refactor of the printing system to take advantage of GAL. I think most
of your patch will stay valid, and I will handle the conversion to the
new code. Just to let you know - do not focus too much on the drawing
code part. Sincere apologies for late response.

Regarsd,
Orson

On 9/13/18 8:28 PM, Zoltan Gyarmati wrote:
> 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,
> 
> 
> _______________________________________________
> 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
> 


Attachment: signature.asc
Description: OpenPGP digital signature


Follow ups

References