kicad-developers team mailing list archive

[Wayne Stambaugh <stambaughw@...>]

Lorenzo wrote:
> --- In kicad-devel@xxxxxxxxxxxxxxx, Werner Almesberger <werner@...> wrote:
> 
>> ;-))) Seems that they hate you back: the inductor is now good, but
> 
> No, it was Charras that misapplied my patch:
> diffing 1846:1847 gives:
> 
> @@ -88,12 +89,12 @@
> if( draw_bgfill && Arc->m_Fill == FILLED_WITH_BG_BODYCOLOR )
> {
> plotter->set_color( ReturnLayerColor( LAYER_DEVICE_BACKGROUND ) );
> - plotter->arc( pos, t1, t2, Arc->m_Rayon, FILLED_SHAPE, 0 );
> + plotter->arc( pos, -t1, -t2, Arc->m_Rayon, FILLED_SHAPE, 0 );
> }
> plotter->set_color( ReturnLayerColor( LAYER_DEVICE ) );
> plotter->arc( pos,
> - t1,
> - t2,
> + -t1,
> + -t2,
> Arc->m_Rayon,
> Arc->m_Fill,
> Arc->m_Width );
> 
> Mine instead said -t2, -t1. The angles need to be negated AND swapped (the strange properties of arcs under a coordinate flip).
> 
> In fact my test case for this was eight FILTER and four 7400 in all the orientations (mirrored too!).
> 
>> By the way, is it just me, or does one have to back out a lot of
>> DXF code before r1847 even passes cmake, and some more for
>> compiling ?
> 
> No cmake problems here... it's compiling right now

Commit R1848 fixed the missing eeschema/plotdxf.cpp CMake error. Now
I'm getting a CMake error for a missing pcbnew/plotdxf.cpp.

Wayne