I'm sorry, yes, x is supposed to be s. I have a question regarding "i". Can I declare it as an internal "data" type? I have plenty of on-board RAM. So for example:

long value(mystring xdata *s)
{
// int xdata i; // old way
unsigned char i; // new way
long result = 0;

for (i=0; i<(s->size); i++)
{
result <<= 8;
result += s->val[i];
}

return result;
}