On Monday 23 January 2006 16:54, Frans Pop wrote: > On Monday 23 January 2006 16:00, Eddy Petrişor wrote: > > Fortunately, on the way home from Extremadura I have tested the > > modification which I done to the fontforge stripping script[1] in > > order to be able to reverse selections. > I have added the shift for now, but I think your patch is not very nice > > It searches the whole parameter string for the presence of the option, > but using a 'shift' if the parameter is present assumes that it is the > first parameter. This could lead to another parameter being removed > instead and maybe to the script failing. > > Is a better solution for checking the presence of the parameter > possible? I never saw a reply to this. I still don't like this solution in the script and as the original scripts _was_ clean, I plan to revert the the change if this is not corrected.
Attachment:
pgpRwyMZhjR8F.pgp
Description: PGP signature