That also should work and you'll have one macro less and one loop less.
Just try it. There is no need to change anything in your function (except missing comma :-)))
>yup, I'm sure.
>
>I really don't want to change the function, as it works for every other carrier, just not this one.
>
>>>uh....yeah, it is. That was one of the first things I checked :-)
>>
>>Sure?
>>
>>SELECT carrier
>>BROWSE FIELDS carr_270 NORMAL
>>ExecMemo(carrier.carr_270, InputTableName)
>>
>>FUNCTION execMemo(memoString, p1, p2, p3, p4, p5)
>> LOCAL lcParams, lcParList, lnPcount
>> lcParams = [p1, p2, p3, p4, p5]
>> lnPcount = PCOUNT()-1
>> lcParList = LEFT(m.lcParams, ATC([,], m.lcParams,m.lnPcount)-1)
>> TRY
>> retVal = EXECSCRIPT(memoString, &lcParList)
>> CATCH TO oError
>> ENDTRY
>>RETURN
>>
Against Stupidity the Gods themselves Contend in Vain - Johann Christoph Friedrich von Schiller
The only thing normal about database guys is their tables.