On Fri, 2011-01-07 at 10:12 +0100, gabor papp wrote: > hi Dave, > > i saw you changed the ArgCheck "i" parameter check from > (!SCHEME_INTP(argv[n])) to (!SCHEME_INTP(argv[n]) && > !SCHEME_NUMBERP(argv[n])). > > i believe this is not right, since there are functions that require int > and not number parameters like (build-torus) for example. this version > accepts (build-torus 1 2 10.1 10.1) for example, which crashes fluxus. i > fixed a couple of functions lately where there should be an "f" instead > of an "i" parameter in their ArgCheck call. you might have run into this > also and changed ArgCheck instead?
I didn't mean to commit this, sorry. I was just trying to get the extrusion code working again, which uses real numbers to do pdata-set! I sort of think it would be nicer to implicitly truncate floats to ints coming in, but this is type coercion that I usually am dead set against. cheers, dave
