New test.
[mono.git] / mono / metadata / string-icalls.c
index efee8f95637c22d10ffae7332ef85a5435a7d3e9..71bd1f7ebe0f84a29f8570e354c3669b487aec17 100644 (file)
@@ -3,6 +3,7 @@
  *
  * Author:
  *   Patrik Torstensson (patrik.torstensson@labs2.com)
+ *   Duncan Mak  (duncan@ximian.com)
  *
  * (C) 2001 Ximian, Inc.
  */
 #include <signal.h>
 #include <string.h>
 #include <mono/metadata/string-icalls.h>
+#include <mono/metadata/class-internals.h>
 #include <mono/metadata/appdomain.h>
 #include <mono/metadata/tabledefs.h>
 #include <mono/metadata/loader.h>
 #include <mono/metadata/object.h>
-#include <mono/metadata/unicode.h>
 #include <mono/metadata/exception.h>
+#include <mono/metadata/debug-helpers.h>
+
+/* Internal helper methods */
+
+static gboolean
+string_icall_is_in_array (MonoArray *chars, gint32 arraylength, gunichar2 chr);
+
+static MonoString*
+empty_string (MonoDomain *domain)
+{
+       MonoVTable *vtable = mono_class_vtable (domain, mono_defaults.string_class);
+       MonoObject *o;
+       static MonoClassField *empty_field = NULL;
+
+       if (!empty_field) {
+               MonoClassField *field;
+               gpointer iter;
+
+               iter = NULL;
+               while ((field = mono_class_get_fields (mono_defaults.string_class, &iter))) {
+                       if (!strcmp (field->name, "Empty"))
+                               break;
+               }
+
+               g_assert (field);
+               empty_field = field;
+       }
+
+       mono_field_static_get_value (vtable, empty_field, &o);
+       g_assert (o);
+       return (MonoString*)o;
+}
 
 MonoString *
-mono_string_Internal_ctor_charp (gpointer dummy, gunichar2 *value)
+ves_icall_System_String_ctor_charp (gpointer dummy, gunichar2 *value)
 {
        gint32 i, length;
        MonoDomain *domain;
 
+       MONO_ARCH_SAVE_REGS;
+
        domain = mono_domain_get ();
 
        if (value == NULL)
-               length = 0;
-       else {
-               for (i = 0; *(value + i) != '\0'; i++);
-               length = i;
-       }
+               return empty_string (domain);
+
+       for (i = 0; *(value + i) != '\0'; i++);
+       length = i;
 
        return mono_string_new_utf16 (domain, value, length);
 }
 
 MonoString *
-mono_string_Internal_ctor_char_int (gpointer dummy, gunichar2 value, gint32 count)
+ves_icall_System_String_ctor_char_int (gpointer dummy, gunichar2 value, gint32 count)
 {
        MonoDomain *domain;
        MonoString *res;
        gunichar2 *chars;
        gint32 i;
 
+       MONO_ARCH_SAVE_REGS;
+
+       if (count < 0)
+               mono_raise_exception (mono_get_exception_argument_out_of_range ("count"));
+
        domain = mono_domain_get ();
        res = mono_string_new_size (domain, count);
 
@@ -56,98 +95,161 @@ mono_string_Internal_ctor_char_int (gpointer dummy, gunichar2 value, gint32 coun
 }
 
 MonoString *
-mono_string_Internal_ctor_charp_int_int (gpointer dummy, gunichar2 *value, gint32 sindex, gint32 length)
+ves_icall_System_String_ctor_charp_int_int (gpointer dummy, gunichar2 *value, gint32 sindex, gint32 length)
 {
        gunichar2 *begin;
        MonoDomain * domain;
        
+       MONO_ARCH_SAVE_REGS;
+
        domain = mono_domain_get ();
 
-       if ((value == NULL) && (sindex != 0) && (length != 0))
-               mono_raise_exception (mono_get_exception_argument_null ("Argument null"));
+       if ((value == NULL) && (length != 0))
+               mono_raise_exception (mono_get_exception_argument_out_of_range ("Out of range"));
 
        if ((sindex < 0) || (length < 0))
                mono_raise_exception (mono_get_exception_argument_out_of_range ("Out of range"));
        
+       if (length == 0)
+               return empty_string (domain);
+       
        begin = (gunichar2 *) (value + sindex);
 
        return mono_string_new_utf16 (domain, begin, length);
 }
 
 MonoString *
-mono_string_Internal_ctor_sbytep (gpointer dummy, gint8 *value)
+ves_icall_System_String_ctor_sbytep (gpointer dummy, gint8 *value)
 {
-       int i, length;
        MonoDomain *domain;
        
+       MONO_ARCH_SAVE_REGS;
+
        domain = mono_domain_get ();
 
-       if (value == NULL)
-               length = 0;
-       else {
-               for (i = 0; *(value + i) != '\0'; i++);
-               length = i;
-       }
+       if (NULL == value)
+               return empty_string (domain);
 
-       return mono_string_new_utf16 (domain, (gunichar2 *) value, length);
+       return mono_string_new (domain, (const char *) value);
 }
 
 MonoString *
-mono_string_Internal_ctor_sbytep_int_int (gpointer dummy, gint8 *value, gint32 sindex, gint32 length)
+ves_icall_System_String_ctor_sbytep_int_int (gpointer dummy, gint8 *value, gint32 sindex, gint32 length)
 {
-       gunichar2 *begin;
+       guchar *begin;
        MonoDomain *domain;
+       MonoString *res;
+       gunichar2 *chars;
+       int i;
        
+       MONO_ARCH_SAVE_REGS;
+
        domain = mono_domain_get ();
 
-       if ((value == NULL) && (sindex != 0) && (length != 0))
-               mono_raise_exception (mono_get_exception_argument_null ("Argument null"));
+       if ((value == NULL) && (length != 0))
+               mono_raise_exception (mono_get_exception_argument_out_of_range ("Out of range"));
 
-       if ((sindex > 0) || (length < 0))
+       if ((sindex < 0) || (length < 0))
                mono_raise_exception (mono_get_exception_argument_out_of_range ("Out of range"));
 
-       begin = (gunichar2 *) (value + sindex);
+       begin = (guchar *) (value + sindex);
+       res = mono_string_new_size (domain, length);
+       chars = mono_string_chars (res);
+       for (i = 0; i < length; ++i)
+               chars [i] = begin [i];
 
-       return mono_string_new_utf16 (domain, begin, length);
+       return res;
 }
 
 MonoString *
-mono_string_Internal_ctor_chara (gpointer dummy, MonoArray *value)
+ves_icall_System_String_ctor_chara (gpointer dummy, MonoArray *value)
 {
        MonoDomain *domain;
 
-       MONO_CHECK_ARG_NULL (value);
-       
+       MONO_ARCH_SAVE_REGS;
+
        domain = mono_domain_get ();
-       
-       return mono_string_new_utf16 (domain, (gunichar2 *) mono_array_addr(value, gunichar2, 0),  value->max_length);
+
+       if (value == NULL)
+               return mono_string_new_utf16 (domain, NULL, 0);
+       else
+               return mono_string_new_utf16 (domain, (gunichar2 *) mono_array_addr(value, gunichar2, 0),  value->max_length);
 }
 
 MonoString *
-mono_string_Internal_ctor_chara_int_int (gpointer dummy, MonoArray *value, 
+ves_icall_System_String_ctor_chara_int_int (gpointer dummy, MonoArray *value, 
                                         gint32 sindex, gint32 length)
 {
        MonoDomain *domain;
 
-       MONO_CHECK_ARG_NULL (value);
+       MONO_ARCH_SAVE_REGS;
+
+       if (value == NULL)
+               mono_raise_exception (mono_get_exception_argument_null ("value"));
+       if (sindex < 0)
+               mono_raise_exception (mono_get_exception_argument_out_of_range ("startIndex"));         
+       if (length < 0)
+               mono_raise_exception (mono_get_exception_argument_out_of_range ("length"));
+       if (sindex + length > mono_array_length (value))
+               mono_raise_exception (mono_get_exception_argument_out_of_range ("Out of range"));
 
-       domain = ((MonoObject *) value)->vtable->domain;
+       domain = mono_domain_get ();
        
        return mono_string_new_utf16 (domain, (gunichar2 *) mono_array_addr(value, gunichar2, sindex), length);
 }
 
 MonoString *
-mono_string_Internal_ctor_encoding (gpointer dummy, gint8 *value, gint32 sindex, 
-                                   gint32 length, MonoObject *enc)
+ves_icall_System_String_ctor_encoding (gpointer dummy, gint8 *value, gint32 sindex, 
+                                      gint32 length, MonoObject *enc)
+{
+       MonoArray *arr;
+       MonoString *s;
+       MonoObject *exc;
+       MonoDomain *domain = mono_domain_get ();
+       MonoMethod *get_string;
+       gpointer args [1];
+       MonoClass *klass;
+
+       MONO_ARCH_SAVE_REGS;
+
+       if ((value == NULL) || (length == 0))
+               return mono_string_new_size (mono_domain_get (), 0);
+       if (enc == NULL)
+               mono_raise_exception (mono_get_exception_argument_null ("enc"));
+       if (sindex < 0)
+               mono_raise_exception (mono_get_exception_argument_out_of_range ("startIndex"));         
+       if (length < 0)
+               mono_raise_exception (mono_get_exception_argument_out_of_range ("length"));
+
+       arr = mono_array_new (domain, mono_defaults.byte_class, length);
+       memcpy (mono_array_addr (arr, guint8, 0), value + sindex, length);
+
+       /* Find the System.Text.Encoding class */
+       for (klass = enc->vtable->klass; klass->parent->parent != NULL; klass = klass->parent)
+               ;
+       
+       get_string = mono_class_get_method_from_name (klass, "GetString", 1);
+       args [0] = arr;
+       s = (MonoString*)mono_runtime_invoke (get_string, enc, args, &exc);
+       if (!s || exc)
+               mono_raise_exception (mono_get_exception_argument ("", "Unable to decode the array into a valid string."));
+
+       return s;
+}
+
+/* This function is redirected to String.CreateString ()
+   by mono_marshal_get_native_wrapper () */
+void
+ves_icall_System_String_ctor_RedirectToCreateString (void)
 {
        g_assert_not_reached ();
-       return NULL;
 }
 
 MonoString * 
-mono_string_InternalJoin (MonoString *separator, MonoArray * value, gint32 sindex, gint32 count)
+ves_icall_System_String_InternalJoin (MonoString *separator, MonoArray * value, gint32 sindex, gint32 count)
 {
        MonoString * ret;
+       MonoString *current;
        gint32 length;
        gint32 pos;
        gint32 insertlen;
@@ -157,26 +259,34 @@ mono_string_InternalJoin (MonoString *separator, MonoArray * value, gint32 sinde
        gunichar2 *dest;
        gunichar2 *src;
 
+       MONO_ARCH_SAVE_REGS;
+
        insert = mono_string_chars(separator);
        insertlen = mono_string_length(separator);
 
        length = 0;
        for (pos = sindex; pos != sindex + count; pos++) {
-               length += mono_string_length(mono_array_get(value, MonoString *, pos));
+               current = mono_array_get (value, MonoString *, pos);
+               if (current != NULL)
+                       length += mono_string_length (current);
+
                if (pos < sindex + count - 1)
                        length += insertlen;
        }
 
-       ret = mono_string_InternalAllocateStr(length);
+       ret = mono_string_new_size( mono_domain_get (), length);
        dest = mono_string_chars(ret);
        destpos = 0;
 
        for (pos = sindex; pos != sindex + count; pos++) {
-               src = mono_string_chars(mono_array_get(value, MonoString *, pos));
-               srclen = mono_string_length(mono_array_get(value, MonoString *, pos));
+               current = mono_array_get (value, MonoString *, pos);
+               if (current != NULL) {
+                       src = mono_string_chars (current);
+                       srclen = mono_string_length (current);
 
-               memcpy(dest + destpos, src, srclen * sizeof(gunichar2));
-               destpos += srclen;
+                       memcpy (dest + destpos, src, srclen * sizeof(gunichar2));
+                       destpos += srclen;
+               }
 
                if (pos < sindex + count - 1) {
                        memcpy(dest + destpos, insert, insertlen * sizeof(gunichar2));
@@ -188,75 +298,94 @@ mono_string_InternalJoin (MonoString *separator, MonoArray * value, gint32 sinde
 }
 
 MonoString * 
-mono_string_InternalInsert (MonoString *me, gint32 sindex, MonoString *value)
+ves_icall_System_String_InternalInsert (MonoString *me, gint32 sindex, MonoString *value)
 {
        MonoString * ret;
        gunichar2 *src;
        gunichar2 *insertsrc;
        gunichar2 *dest;
        gint32 srclen;
+       gint32 insertlen;
+
+       MONO_ARCH_SAVE_REGS;
 
        src = mono_string_chars(me);
-       srclen = mono_string_length(value);
+       srclen = mono_string_length(me);
+
+       insertsrc = mono_string_chars(value);
+       insertlen = mono_string_length(value);
 
-       ret = mono_string_InternalAllocateStr(mono_string_length(me) + srclen - sindex);
+       ret = mono_string_new_size( mono_domain_get (), srclen + insertlen);
        dest = mono_string_chars(ret);
 
        memcpy(dest, src, sindex * sizeof(gunichar2));
-       memcpy(dest + sindex, insertsrc, srclen * sizeof(gunichar2));
+       memcpy(dest + sindex, insertsrc, insertlen * sizeof(gunichar2));
+       memcpy(dest + sindex + insertlen, src + sindex, (srclen - sindex) * sizeof(gunichar2));
 
        return ret;
 }
 
 MonoString * 
-mono_string_InternalReplaceChar (MonoString *me, gunichar2 oldChar, gunichar2 newChar)
+ves_icall_System_String_InternalReplace_Char (MonoString *me, gunichar2 oldChar, gunichar2 newChar)
 {
-       g_warning("mono_string_InternalReplaceChar not impl");
-       return mono_string_new_utf16(mono_domain_get(), mono_string_chars(me), mono_string_length(me));
-}
+       MonoString *ret;
+       gunichar2 *src;
+       gunichar2 *dest;
+       gint32 i, srclen;
 
-MonoString * 
-mono_string_InternalReplaceStr (MonoString *me, MonoString *oldValue, MonoString *newValue)
-{
-       g_warning("mono_string_InternalReplaceStr not impl");
-       return mono_string_new_utf16(mono_domain_get(), mono_string_chars(me), mono_string_length(me));
+       MONO_ARCH_SAVE_REGS;
+
+       src = mono_string_chars(me);
+       srclen = mono_string_length(me);
+
+       ret = mono_string_new_size( mono_domain_get (), srclen);
+       dest = mono_string_chars(ret);
+
+       for (i = 0; i != srclen; i++) {
+               if (src[i] == oldChar)
+                       dest[i] = newChar;
+               else
+                       dest[i] = src[i];
+       }
+
+       return ret;
 }
 
 MonoString * 
-mono_string_InternalRemove (MonoString *me, gint32 sindex, gint32 count)
+ves_icall_System_String_InternalRemove (MonoString *me, gint32 sindex, gint32 count)
 {
        MonoString * ret;
-       gint32 count_bytes;
-       gint32 index_bytes;
-       gint32 me_bytes;
+       gint32 srclen;
        gunichar2 *dest;
        gunichar2 *src;
 
-       me_bytes = mono_string_length(me);
-       ret = mono_string_InternalAllocateStr(me_bytes - count);
-       index_bytes = sindex * sizeof(gunichar2);
-       count_bytes = count * sizeof(gunichar2);
+       MONO_ARCH_SAVE_REGS;
+
+       srclen = mono_string_length(me);
+       ret = mono_string_new_size( mono_domain_get (), srclen - count);
 
        src = mono_string_chars(me);
        dest = mono_string_chars(ret);
 
-       memcpy(dest, src, index_bytes);
-       memcpy(dest + sindex, src + sindex + count, me_bytes - count_bytes);
+       memcpy(dest, src, sindex * sizeof(gunichar2));
+       memcpy(dest + sindex, src + sindex + count, (srclen - count - sindex) * sizeof(gunichar2));
 
        return ret;
 }
 
 void
-mono_string_InternalCopyTo (MonoString *me, gint32 sindex, MonoArray *dest, gint32 dindex, gint32 count)
+ves_icall_System_String_InternalCopyTo (MonoString *me, gint32 sindex, MonoArray *dest, gint32 dindex, gint32 count)
 {
        gunichar2 *destptr = (gunichar2 *) mono_array_addr(dest, gunichar2, dindex);
        gunichar2 *src =  mono_string_chars(me);
 
+       MONO_ARCH_SAVE_REGS;
+
        memcpy(destptr, src + sindex, sizeof(gunichar2) * count);
 }
 
 MonoArray * 
-mono_string_InternalSplit (MonoString *me, MonoArray *separator, gint32 count)
+ves_icall_System_String_InternalSplit (MonoString *me, MonoArray *separator, gint32 count)
 {
        MonoString * tmpstr;
        MonoArray * retarr;
@@ -266,13 +395,19 @@ mono_string_InternalSplit (MonoString *me, MonoArray *separator, gint32 count)
        gint32 tmpstrsize;
        gunichar2 *tmpstrptr;
 
+       gunichar2 cmpchar;
+
+       MONO_ARCH_SAVE_REGS;
+
        src = mono_string_chars(me);
        srcsize = mono_string_length(me);
        arrsize = mono_array_length(separator);
 
+       cmpchar = mono_array_get(separator, gunichar2, 0);
+
        splitsize = 0;
        for (i = 0; i != srcsize && splitsize < count; i++) {
-               if (mono_string_isinarray(separator, arrsize, src[i]))
+               if (string_icall_is_in_array(separator, arrsize, src[i]))
                        splitsize++;
        }
 
@@ -281,32 +416,28 @@ mono_string_InternalSplit (MonoString *me, MonoArray *separator, gint32 count)
 
        /* if no split chars found return the string */
        if (splitsize == 0) {
-               retarr = mono_array_new(mono_domain_get(), mono_defaults.string_class, 1);
-               tmpstr = mono_string_InternalAllocateStr(srcsize);
-               tmpstrptr = mono_string_chars(tmpstr);
-
-               memcpy(tmpstrptr, src, srcsize * sizeof(gunichar2));
-               mono_array_set(retarr, MonoString *, 0, tmpstr);
+               retarr = mono_array_new(mono_domain_get(), mono_get_string_class (), 1);
+               mono_array_setref (retarr, 0, me);
 
                return retarr;
        }
 
-       if (splitsize + 1 < count)
+       if (splitsize != count)
                splitsize++;
 
-       retarr = mono_array_new(mono_domain_get(), mono_defaults.string_class, splitsize);
+       retarr = mono_array_new(mono_domain_get(), mono_get_string_class (), splitsize);
        for (i = 0; i != srcsize && arrpos != count; i++) {
-               if (mono_string_isinarray(separator, arrsize, src[i])) {
+               if (string_icall_is_in_array(separator, arrsize, src[i])) {
                        if (arrpos == count - 1)
                                tmpstrsize = srcsize - lastpos;
                        else
                                tmpstrsize = i - lastpos;
 
-                       tmpstr = mono_string_InternalAllocateStr(tmpstrsize);
+                       tmpstr = mono_string_new_size( mono_domain_get (), tmpstrsize);
                        tmpstrptr = mono_string_chars(tmpstr);
 
                        memcpy(tmpstrptr, src + lastpos, tmpstrsize * sizeof(gunichar2));
-                       mono_array_set(retarr, MonoString *, arrpos, tmpstr);
+                       mono_array_setref (retarr, arrpos, tmpstr);
                        arrpos++;
                        lastpos = i + 1;
                }
@@ -314,25 +445,25 @@ mono_string_InternalSplit (MonoString *me, MonoArray *separator, gint32 count)
 
        if (arrpos < count) {
                tmpstrsize = srcsize - lastpos;
-               tmpstr = mono_string_InternalAllocateStr(tmpstrsize);
+               tmpstr = mono_string_new_size( mono_domain_get (), tmpstrsize);
                tmpstrptr = mono_string_chars(tmpstr);
 
                memcpy(tmpstrptr, src + lastpos, tmpstrsize * sizeof(gunichar2));
-               mono_array_set(retarr, MonoString *, arrpos, tmpstr);
+               mono_array_setref (retarr, arrpos, tmpstr);
        }
-       
+
        return retarr;
 }
 
-gboolean
-mono_string_isinarray (MonoArray *chars, gint32 arraylength, gunichar2 chr)
+static gboolean
+string_icall_is_in_array (MonoArray *chars, gint32 arraylength, gunichar2 chr)
 {
        gunichar2 cmpchar;
        gint32 arrpos;
 
        for (arrpos = 0; arrpos != arraylength; arrpos++) {
                cmpchar = mono_array_get(chars, gunichar2, arrpos);
-               if (mono_string_cmp_char(cmpchar, chr, 1) == 0)
+               if (cmpchar == chr)
                        return TRUE;
        }
        
@@ -340,13 +471,15 @@ mono_string_isinarray (MonoArray *chars, gint32 arraylength, gunichar2 chr)
 }
 
 MonoString * 
-mono_string_InternalTrim (MonoString *me, MonoArray *chars, gint32 typ)
+ves_icall_System_String_InternalTrim (MonoString *me, MonoArray *chars, gint32 typ)
 {
        MonoString * ret;
        gunichar2 *src, *dest;
        gint32 srclen, newlen, arrlen;
        gint32 i, lenfirst, lenlast;
 
+       MONO_ARCH_SAVE_REGS;
+
        srclen = mono_string_length(me);
        src = mono_string_chars(me);
        arrlen = mono_array_length(chars);
@@ -356,7 +489,7 @@ mono_string_InternalTrim (MonoString *me, MonoArray *chars, gint32 typ)
 
        if (0 == typ || 1 == typ) {
                for (i = 0; i != srclen; i++) {
-                       if (mono_string_isinarray(chars, arrlen, src[i]))
+                       if (string_icall_is_in_array(chars, arrlen, src[i]))
                                lenfirst++;
                        else 
                                break;
@@ -365,7 +498,7 @@ mono_string_InternalTrim (MonoString *me, MonoArray *chars, gint32 typ)
 
        if (0 == typ || 2 == typ) {
                for (i = srclen - 1; i > lenfirst - 1; i--) {
-                       if (mono_string_isinarray(chars, arrlen, src[i]))
+                       if (string_icall_is_in_array(chars, arrlen, src[i]))
                                lenlast++;
                        else 
                                break;
@@ -373,8 +506,10 @@ mono_string_InternalTrim (MonoString *me, MonoArray *chars, gint32 typ)
        }
 
        newlen = srclen - lenfirst - lenlast;
+       if (newlen == srclen)
+               return me;
 
-       ret = mono_string_InternalAllocateStr(newlen);
+       ret = mono_string_new_size( mono_domain_get (), newlen);
        dest = mono_string_chars(ret);
 
        memcpy(dest, src + lenfirst, newlen *sizeof(gunichar2));
@@ -383,52 +518,15 @@ mono_string_InternalTrim (MonoString *me, MonoArray *chars, gint32 typ)
 }
 
 gint32 
-mono_string_InternalIndexOfChar (MonoString *me, gunichar2 value, gint32 sindex, gint32 count)
-{
-       gint32 pos;
-       gunichar2 *src;
-
-       src = mono_string_chars(me);
-       for (pos = sindex; pos != count + sindex; pos++) {
-               if ( src [pos] == value)
-                       return pos;
-       }
-
-       return -1;
-}
-
-gint32 
-mono_string_InternalIndexOfStr (MonoString *me, MonoString *value, gint32 sindex, gint32 count)
-{
-       gint32 lencmpstr;
-       gint32 pos;
-       gunichar2 *src;
-       gunichar2 *cmpstr;
-
-       lencmpstr = mono_string_length(value);
-
-       src = mono_string_chars(me);
-       cmpstr = mono_string_chars(value);
-
-       for (pos = sindex; pos != count + sindex; pos++) {
-               if (pos + lencmpstr > count + sindex)
-                       return -1;
-
-               if (0 == memcmp(src + pos, cmpstr, lencmpstr * sizeof(gunichar2)))
-                       return pos;
-       }
-
-       return -1;
-}
-
-gint32 
-mono_string_InternalIndexOfAny (MonoString *me, MonoArray *arr, gint32 sindex, gint32 count)
+ves_icall_System_String_InternalIndexOfAny (MonoString *me, MonoArray *arr, gint32 sindex, gint32 count)
 {
        gint32 pos;
        gint32 loop;
        gint32 arraysize;
        gunichar2 *src;
 
+       MONO_ARCH_SAVE_REGS;
+
        arraysize = mono_array_length(arr);
        src = mono_string_chars(me);
 
@@ -442,11 +540,13 @@ mono_string_InternalIndexOfAny (MonoString *me, MonoArray *arr, gint32 sindex, g
 }
 
 gint32 
-mono_string_InternalLastIndexOfChar (MonoString *me, gunichar2 value, gint32 sindex, gint32 count)
+ves_icall_System_String_InternalLastIndexOf_Char (MonoString *me, gunichar2 value, gint32 sindex, gint32 count)
 {
        gint32 pos;
        gunichar2 *src;
 
+       MONO_ARCH_SAVE_REGS;
+
        src = mono_string_chars(me);
        for (pos = sindex; pos > sindex - count; pos--) {
                if (src [pos] == value)
@@ -457,19 +557,21 @@ mono_string_InternalLastIndexOfChar (MonoString *me, gunichar2 value, gint32 sin
 }
 
 gint32 
-mono_string_InternalLastIndexOfStr (MonoString *me, MonoString *value, gint32 sindex, gint32 count)
+ves_icall_System_String_InternalLastIndexOf_Str (MonoString *me, MonoString *value, gint32 sindex, gint32 count)
 {
        gint32 lencmpstr;
        gint32 pos;
        gunichar2 *src;
        gunichar2 *cmpstr;
 
+       MONO_ARCH_SAVE_REGS;
+
        lencmpstr = mono_string_length(value);
 
        src = mono_string_chars(me);
        cmpstr = mono_string_chars(value);
 
-       for (pos = sindex; pos > sindex - count; pos--) {
+       for (pos = sindex - lencmpstr + 1; pos > sindex - count; pos--) {
                if (0 == memcmp(src + pos, cmpstr, lencmpstr * sizeof(gunichar2)))
                        return pos;
        }
@@ -478,13 +580,15 @@ mono_string_InternalLastIndexOfStr (MonoString *me, MonoString *value, gint32 si
 }
 
 gint32 
-mono_string_InternalLastIndexOfAny (MonoString *me, MonoArray *anyOf, gint32 sindex, gint32 count)
+ves_icall_System_String_InternalLastIndexOfAny (MonoString *me, MonoArray *anyOf, gint32 sindex, gint32 count)
 {
        gint32 pos;
        gint32 loop;
        gint32 arraysize;
        gunichar2 *src;
 
+       MONO_ARCH_SAVE_REGS;
+
        arraysize = mono_array_length(anyOf);
        src = mono_string_chars(me);
 
@@ -498,7 +602,7 @@ mono_string_InternalLastIndexOfAny (MonoString *me, MonoArray *anyOf, gint32 sin
 }
 
 MonoString *
-mono_string_InternalPad (MonoString *me, gint32 width, gunichar2 chr, MonoBoolean right)
+ves_icall_System_String_InternalPad (MonoString *me, gint32 width, gunichar2 chr, MonoBoolean right)
 {
        MonoString * ret;
        gunichar2 *src;
@@ -507,10 +611,12 @@ mono_string_InternalPad (MonoString *me, gint32 width, gunichar2 chr, MonoBoolea
        gint32 srclen;
        gint32 i;
 
+       MONO_ARCH_SAVE_REGS;
+
        srclen = mono_string_length(me);
        src = mono_string_chars(me);
 
-       ret = mono_string_InternalAllocateStr(width);
+       ret = mono_string_new_size( mono_domain_get (), width);
        dest = mono_string_chars(ret);
        fillcount = width - srclen;
 
@@ -532,188 +638,98 @@ mono_string_InternalPad (MonoString *me, gint32 width, gunichar2 chr, MonoBoolea
 }
 
 MonoString *
-mono_string_InternalToLower (MonoString *me)
+ves_icall_System_String_InternalAllocateStr (gint32 length)
 {
-       MonoString * ret;
-       gunichar2 *src; 
-       gunichar2 *dest;
-       gint32 i;
-
-       ret = mono_string_new_size(mono_domain_get (), mono_string_length(me));
-
-       src = mono_string_chars (me);
-       dest = mono_string_chars (ret);
-
-       for (i = 0; i < mono_string_length (me); ++i)
-               dest[i] = g_unichar_tolower(src[i]);
-
-       return ret;
-}
-
-MonoString *
-mono_string_InternalToUpper (MonoString *me)
-{
-       int i;
-       MonoString * ret;
-       gunichar2 *src; 
-       gunichar2 *dest;
-
-       ret = mono_string_new_size(mono_domain_get (), mono_string_length(me));
-
-       src = mono_string_chars (me);
-       dest = mono_string_chars (ret);
+       MONO_ARCH_SAVE_REGS;
 
-       for (i = 0; i < mono_string_length (me); ++i)
-               dest[i] = g_unichar_toupper(src[i]);
-
-       return ret;
-}
-
-MonoString *
-mono_string_InternalAllocateStr(gint32 length)
-{
        return mono_string_new_size(mono_domain_get (), length);
 }
 
 void 
-mono_string_InternalStrcpyStr (MonoString *dest, gint32 destPos, MonoString *src)
+ves_icall_System_String_InternalStrcpy_Str (MonoString *dest, gint32 destPos, MonoString *src)
 {
        gunichar2 *srcptr;
        gunichar2 *destptr;
 
+       MONO_ARCH_SAVE_REGS;
+
        srcptr = mono_string_chars (src);
        destptr = mono_string_chars (dest);
 
-       memcpy(destptr + destPos, srcptr, mono_string_length(src) * sizeof(gunichar2));
+       g_memmove (destptr + destPos, srcptr, mono_string_length(src) * sizeof(gunichar2));
 }
 
 void 
-mono_string_InternalStrcpyStrN (MonoString *dest, gint32 destPos, MonoString *src, gint32 startPos, gint32 count)
+ves_icall_System_String_InternalStrcpy_StrN (MonoString *dest, gint32 destPos, MonoString *src, gint32 startPos, gint32 count)
 {
        gunichar2 *srcptr;
        gunichar2 *destptr;
 
+       MONO_ARCH_SAVE_REGS;
+
        srcptr = mono_string_chars (src);
        destptr = mono_string_chars (dest);
-       memcpy(destptr + destPos, srcptr + startPos, count * sizeof(gunichar2));
-}
-
-MonoString  *
-mono_string_InternalIntern (MonoString *str)
-{
-       return mono_string_intern(str);
-}
-
-MonoString * 
-mono_string_InternalIsInterned (MonoString *str)
-{
-       return mono_string_is_interned(str);
+       g_memmove (destptr + destPos, srcptr + startPos, count * sizeof(gunichar2));
 }
 
-gint32
-mono_string_InternalCompareStrN (MonoString *s1, gint32 i1, MonoString *s2, gint32 i2, gint32 length, MonoBoolean inCase)
+void 
+ves_icall_System_String_InternalStrcpy_Chars (MonoString *dest, gint32 destPos, MonoArray *src)
 {
-       /* c translation of C# code.. :)
-       */
-       gint32 lenstr1;
-       gint32 lenstr2;
-       gint32 charcmp;
-       gunichar2 *str1;
-       gunichar2 *str2;
+       gunichar2 *srcptr;
+       gunichar2 *destptr;
 
-       gint32 pos;
-       gint16 mode;
-       
-       if (inCase)
-               mode = 1;
-       else
-               mode = 0;
+       MONO_ARCH_SAVE_REGS;
 
-       lenstr1 = mono_string_length(s1);
-       lenstr2 = mono_string_length(s2);
+       srcptr = mono_array_addr (src, gunichar2, 0);
+       destptr = mono_string_chars (dest);
 
-       str1 = mono_string_chars(s1);
-       str2 = mono_string_chars(s2);
+       g_memmove (destptr + destPos, srcptr, mono_array_length (src) * sizeof(gunichar2));
+}
 
-       pos = 0;
+void 
+ves_icall_System_String_InternalStrcpy_CharsN (MonoString *dest, gint32 destPos, MonoArray *src, gint32 startPos, gint32 count)
+{
+       gunichar2 *srcptr;
+       gunichar2 *destptr;
 
-       for (pos = 0; pos != length; pos++) {
-               if (i1 + pos >= lenstr1 || i2 + pos >= lenstr2)
-                       break;
+       MONO_ARCH_SAVE_REGS;
 
-               charcmp = mono_string_cmp_char(str1[i1 + pos], str2[i2 + pos], mode);
-               if (charcmp != 0)
-                       return charcmp;
-       }
+       srcptr = mono_array_addr (src, gunichar2, 0);
+       destptr = mono_string_chars (dest);
 
-       /* the lesser wins, so if we have looped until length we just need to check the last char */
-       if (pos == length) {
-               return mono_string_cmp_char(str1[i1 + pos - 1], str2[i2 + pos - 1], mode);
-       }
+       g_memmove (destptr + destPos, srcptr + startPos, count * sizeof(gunichar2));
+}
 
-       /* Test if one the strings has been compared to the end */
-       if (i1 + pos >= lenstr1) {
-               if (i2 + pos >= lenstr2)
-                       return 0;
-               else
-                       return -1;
-       } else if (i2 + pos >= lenstr2)
-               return 1;
+MonoString  *
+ves_icall_System_String_InternalIntern (MonoString *str)
+{
+       MONO_ARCH_SAVE_REGS;
 
-       /* if not, check our last char only.. (can this happen?) */
-       return mono_string_cmp_char(str1[i1 + pos], str2[i2 + pos], mode);
+       return mono_string_intern(str);
 }
 
-gint32
-mono_string_GetHashCode (MonoString *me)
+MonoString * 
+ves_icall_System_String_InternalIsInterned (MonoString *str)
 {
-       int i, h = 0;
-       gunichar2 *data = mono_string_chars (me);
+       MONO_ARCH_SAVE_REGS;
 
-       for (i = 0; i < mono_string_length (me); ++i)
-               h = (h << 5) - h + data [i];
-
-       return h;
+       return mono_string_is_interned(str);
 }
 
 gunichar2 
-mono_string_get_Chars (MonoString *me, gint32 idx)
+ves_icall_System_String_get_Chars (MonoString *me, gint32 idx)
 {
+       MONO_ARCH_SAVE_REGS;
+
+       if ((idx < 0) || (idx >= mono_string_length (me)))
+               mono_raise_exception (mono_get_exception_index_out_of_range ());
        return mono_string_chars(me)[idx];
 }
 
-/* @mode :     0 = StringCompareModeDirect
-                       1 = StringCompareModeCaseInsensitive
-                       2 = StringCompareModeOrdinal
-*/
-gint32 
-mono_string_cmp_char (gunichar2 c1, gunichar2 c2, gint16 mode)
-{
-       gint32 result;
-       GUnicodeType c1type, c2type;
-
-       c1type = g_unichar_type (c1);
-       c2type = g_unichar_type (c2);
-       switch (mode) {
-       case 0: 
-               /* TODO: compare with culture info */
-               if (c1type == G_UNICODE_UPPERCASE_LETTER && c2type == G_UNICODE_LOWERCASE_LETTER)
-                       return 1;
-                                       
-               if (c1type == G_UNICODE_LOWERCASE_LETTER && c2type == G_UNICODE_UPPERCASE_LETTER)
-                       return -1;
-       
-               result = (gint32) c1 - c2;
-               break;
-       case 1: 
-               result = (gint32) (c1type != G_UNICODE_LOWERCASE_LETTER ? g_unichar_tolower(c1) : c1) - 
-                                 (c2type != G_UNICODE_LOWERCASE_LETTER ? g_unichar_tolower(c2) : c2);
-               break;
-               /* fix: compare ordinal */
-       case 2: 
-               result = (gint32) g_unichar_tolower(c1) - g_unichar_tolower(c2);
-               break;
-       }
+void
+ves_icall_System_String_InternalCharCopy (gunichar2 *src, gunichar2 *dest, gint32 count)
+{
+       MONO_ARCH_SAVE_REGS;
 
-       return ((result < 0) ? -1 : (result > 0) ? 1 : 0);
+       memcpy (dest, src, sizeof (gunichar2) * count);
 }