>x=atc(“;”,m_from)
>replace all m_from with substr(m_from,x+1)
>replace all m_from with alltrim(m_from)


m_from is a variable storing the fieldname right? Dont you need...
replace all (m_from) with substr(...)

If m_from is the actual name of the field, then try prefacing it in these couple of lines with the 'tablename.' and see if that makes any difference.