← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Add empty eeschema page layout for Eagle schematic import. Bug #1729722

 

Be careful with zero length line segments.  They have been known to
cause issues in the past.  We recently fixed a print bug where a zero
diameter circle was causing pages not to print.

On 2/7/2018 7:45 AM, Russell Oliver wrote:
> Hi Orson, 
> 
> I'm completely fine with any simplifications and style changes. 
> 
> With regards to the zero length line, it appears on line 110 of your
> patch file. 
> 110: +    "(line (name segm1:Line) (start 0 0) (end 0 0))\n" 
> 
> JP mentions in a comment to the bug report that there is a legacy
> compatibility requirement to have at least one item in the page layout,
> otherwise the default layout it used. This was for old schematics that
> do not have a page layout specified. 
> 
> Kind Regards
> Russell
> 
> 
> On Wed, Feb 7, 2018 at 12:13 AM Maciej Sumiński <maciej.suminski@xxxxxxx
> <mailto:maciej.suminski@xxxxxxx>> wrote:
> 
>     Hi Russell,
> 
>     Thank you very much for the patch. It works as expected and I would like
>     to merge it, but there are two things.
> 
>     I have simplified the patch a bit (moved the empty layout to an existing
>     file, minor code formatting fixes), so please confirm you are ok with
>     committing it under your name.
> 
>     Another question is about "there is a 0 length line to fool something
>     somewhere." comment for const char emptyLayout[]. Could you say
>     something more about it? I could not spot a 0 length line in the layout
>     description, so perhaps we can remove it to avoid confusion.
> 
>     Regards,
>     Orson
> 
>     On 02/03/2018 01:27 AM, Russell Oliver wrote:
>     > Attached is a patch that adds an empty layout using the same
>     method as the
>     > SetDefaultLayout function, which is then called by the Eagle schematic
>     > plugin to leave only the imported frame visible.
>     >
>     > https://bugs.launchpad.net/kicad/+bug/1729722
>     >
>     > Kind Regards
>     > Russell
>     >
>     >
>     >
>     > _______________________________________________
>     > Mailing list: https://launchpad.net/~kicad-developers
>     > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>     > Unsubscribe : https://launchpad.net/~kicad-developers
>     > More help   : https://help.launchpad.net/ListHelp
>     >
> 
>     _______________________________________________
>     Mailing list: https://launchpad.net/~kicad-developers
>     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>     Unsubscribe : https://launchpad.net/~kicad-developers
>     More help   : https://help.launchpad.net/ListHelp
> 
> 
> 
> _______________________________________________
> 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