[Fluxus] ArgCheck "i" parameter

David Griffiths dave at pawfal.org
Fri Jan 7 01:23:51 PST 2011


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




More information about the Fluxus mailing list