gEDA-dev: PATCH: gerbv StepRepeat and two minor bugs

["Joost Witteveen" <joosteto-at-NOSPAMgmail_dot_com>]

Hi,

I've just written a patch for gerbv to make it interpret the %SR% step
repeat extention. I used the already present code to parse the
%SFX3Y3I0.2J0.3% input data, and just went on to actually use the data
in draw.c etc; I hope I did it in the rest of the gerbv spirit.

While working, I noticed two minor bugs:

1) When gerbv encounters an unknown % code (in my case then the not
yet implemented %SR% code), it goes in an endless loop in the case '%'
in parse_gerb(), as the while loop didn't check for EOF.

2) Setting the initial scale (image->info->min_x, min_y) in the case
'*' code in parse_gerb() went wrong for %MOMM% gerber files, as the
if() statements compared to the unscaled curr_net->stop_x.

The atached patch fixes the two mentioned problems (as well as adding
support for %SR%).

Thanks,
joost witteveen

(A previous post to this list hasn't apeard yet (though I sent it more
than 14 hours ago, maybe because I wasn't subscribed to the list yet.
Anyway, this email has a somewhat improved version of the patch)

gerbv-sr-patch

_______________________________________________
geda-dev mailing list
geda-dev@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev