Merge pull request #5714 from alexischr/update_bockbuild
[mono.git] / mono / metadata / exception.c
index d8bdf35dfa53913f53b3eca1724b804ae856bfc5..beece6709d74432af8e3a942362a6dcc9305986d 100644 (file)
@@ -1,5 +1,6 @@
-/*
- * exception.c: Exception handling
+/**
+ * \file
+ * Exception handling
  *
  * Authors:
  *     Paolo Molaro    (lupus@ximian.com)
@@ -35,14 +36,14 @@ static gpointer unhandled_exception_hook_data = NULL;
 
 /**
  * mono_exception_from_name:
- * @image: the Mono image where to look for the class
- * @name_space: the namespace for the class
- * @name: class name
+ * \param image the Mono image where to look for the class
+ * \param name_space the namespace for the class
+ * \param name class name
  *
  * Creates an exception of the given namespace/name class in the
  * current domain.
  *
- * Returns: the initialized exception instance.
+ * \returns the initialized exception instance.
  */
 MonoException *
 mono_exception_from_name (MonoImage *image, const char *name_space,
@@ -53,15 +54,15 @@ mono_exception_from_name (MonoImage *image, const char *name_space,
 
 /**
  * mono_exception_from_name_domain:
- * @domain: Domain where the return object will be created.
- * @image: the Mono image where to look for the class
- * @name_space: the namespace for the class
- * @name: class name
+ * \param domain Domain where the return object will be created.
+ * \param image the Mono image where to look for the class
+ * \param name_space the namespace for the class
+ * \param name class name
  *
  * Creates an exception object of the given namespace/name class on
  * the given domain.
  *
- * Returns: the initialized exception instance.
+ * \returns the initialized exception instance.
  */
 MonoException *
 mono_exception_from_name_domain (MonoDomain *domain, MonoImage *image, 
@@ -91,12 +92,12 @@ mono_exception_from_name_domain (MonoDomain *domain, MonoImage *image,
 
 /**
  * mono_exception_from_token:
- * @image: the Mono image where to look for the class
- * @token: The type token of the class
+ * \param image the Mono image where to look for the class
+ * \param token The type token of the class
  *
- * Creates an exception of the type given by @token.
+ * Creates an exception of the type given by \p token.
  *
- * Returns: the initialized exception instance.
+ * \returns the initialized exception instance.
  */
 MonoException *
 mono_exception_from_token (MonoImage *image, guint32 token)
@@ -163,16 +164,16 @@ create_exception_two_strings (MonoClass *klass, MonoString *a1, MonoString *a2,
 
 /**
  * mono_exception_from_name_two_strings:
- * @image: the Mono image where to look for the class
- * @name_space: the namespace for the class
- * @name: class name
- * @a1: first string argument to pass
- * @a2: second string argument to pass
+ * \param image the Mono image where to look for the class
+ * \param name_space the namespace for the class
+ * \param name class name
+ * \param a1 first string argument to pass
+ * \param a2 second string argument to pass
  *
  * Creates an exception from a constructor that takes two string
  * arguments.
  *
- * Returns: the initialized exception instance.
+ * \returns the initialized exception instance.
  */
 MonoException *
 mono_exception_from_name_two_strings (MonoImage *image, const char *name_space,
@@ -188,18 +189,18 @@ mono_exception_from_name_two_strings (MonoImage *image, const char *name_space,
 
 /**
  * mono_exception_from_name_two_strings_checked:
- * @image: the Mono image where to look for the class
- * @name_space: the namespace for the class
- * @name: class name
- * @a1: first string argument to pass
- * @a2: second string argument to pass
- * @error: set on error
+ * \param image the Mono image where to look for the class
+ * \param name_space the namespace for the class
+ * \param name class name
+ * \param a1 first string argument to pass
+ * \param a2 second string argument to pass
+ * \param error set on error
  *
  * Creates an exception from a constructor that takes two string
  * arguments.
  *
- * Returns: the initialized exception instance. On failure returns
- * NULL and sets @error.
+ * \returns the initialized exception instance. On failure returns
+ * NULL and sets \p error.
  */
 MonoException *
 mono_exception_from_name_two_strings_checked (MonoImage *image, const char *name_space,
@@ -208,7 +209,7 @@ mono_exception_from_name_two_strings_checked (MonoImage *image, const char *name
 {
        MonoClass *klass;
 
-       mono_error_init (error);
+       error_init (error);
        klass = mono_class_load_from_name (image, name_space, name);
 
        return create_exception_two_strings (klass, a1, a2, error);
@@ -216,25 +217,29 @@ mono_exception_from_name_two_strings_checked (MonoImage *image, const char *name
 
 /**
  * mono_exception_from_name_msg:
- * @image: the Mono image where to look for the class
- * @name_space: the namespace for the class
- * @name: class name
- * @msg: the message to embed inside the exception
+ * \param image the Mono image where to look for the class
+ * \param name_space the namespace for the class
+ * \param name class name
+ * \param msg the message to embed inside the exception
  *
  * Creates an exception and initializes its message field.
  *
- * Returns: the initialized exception instance.
+ * \returns the initialized exception instance.
  */
 MonoException *
 mono_exception_from_name_msg (MonoImage *image, const char *name_space,
                              const char *name, const char *msg)
 {
+       MonoError error;
        MonoException *ex;
 
        ex = mono_exception_from_name (image, name_space, name);
 
-       if (msg)
-               MONO_OBJECT_SETREF (ex, message, mono_string_new (mono_object_get_domain ((MonoObject*)ex), msg));
+       if (msg) {
+               MonoString  *msg_str = mono_string_new_checked (mono_object_get_domain ((MonoObject*)ex), msg, &error);
+               mono_error_assert_ok (&error);
+               MONO_OBJECT_SETREF (ex, message, msg_str);
+       }
 
        return ex;
 }
@@ -269,7 +274,7 @@ mono_exception_from_token_two_strings_checked (MonoImage *image, guint32 token,
 {
        MonoClass *klass;
 
-       mono_error_init (error);
+       error_init (error);
 
        klass = mono_class_get_checked (image, token, error);
        mono_error_assert_ok (error); /* FIXME handle the error. */
@@ -279,8 +284,7 @@ mono_exception_from_token_two_strings_checked (MonoImage *image, guint32 token,
 
 /**
  * mono_get_exception_divide_by_zero:
- *
- * Returns: a new instance of the `System.DivideByZeroException`
+ * \returns a new instance of the \c System.DivideByZeroException
  */
 MonoException *
 mono_get_exception_divide_by_zero ()
@@ -291,8 +295,7 @@ mono_get_exception_divide_by_zero ()
 
 /**
  * mono_get_exception_security:
- *
- * Returns: a new instance of the `System.Security.SecurityException`
+ * \returns a new instance of the \c System.Security.SecurityException
  */
 MonoException *
 mono_get_exception_security ()
@@ -303,8 +306,7 @@ mono_get_exception_security ()
 
 /**
  * mono_get_exception_thread_abort:
- *
- * Returns: a new instance of the `System.Threading.ThreadAbortException`
+ * \returns a new instance of the \c System.Threading.ThreadAbortException
  */
 MonoException *
 mono_get_exception_thread_abort ()
@@ -315,8 +317,7 @@ mono_get_exception_thread_abort ()
 
 /**
  * mono_get_exception_thread_interrupted:
- *
- * Returns: a new instance of the `System.Threading.ThreadInterruptedException`
+ * \returns a new instance of the \c System.Threading.ThreadInterruptedException
  */
 MonoException *
 mono_get_exception_thread_interrupted ()
@@ -327,8 +328,7 @@ mono_get_exception_thread_interrupted ()
 
 /**
  * mono_get_exception_arithmetic:
- *
- * Returns: a new instance of the `System.ArithmeticException`
+ * \returns a new instance of the \c System.ArithmeticException
  */
 MonoException *
 mono_get_exception_arithmetic ()
@@ -339,8 +339,7 @@ mono_get_exception_arithmetic ()
 
 /**
  * mono_get_exception_overflow:
- *
- * Returns: a new instance of the `System.OverflowException`
+ * \returns a new instance of the \c System.OverflowException
  */
 MonoException *
 mono_get_exception_overflow ()
@@ -351,8 +350,7 @@ mono_get_exception_overflow ()
 
 /**
  * mono_get_exception_null_reference:
- *
- * Returns: a new instance of the `System.NullReferenceException`
+ * \returns a new instance of the \c System.NullReferenceException
  */
 MonoException *
 mono_get_exception_null_reference ()
@@ -363,9 +361,8 @@ mono_get_exception_null_reference ()
 
 /**
  * mono_get_exception_execution_engine:
- * @msg: the message to pass to the user
- *
- * Returns: a new instance of the `System.ExecutionEngineException`
+ * \param msg the message to pass to the user
+ * \returns a new instance of the \c System.ExecutionEngineException
  */
 MonoException *
 mono_get_exception_execution_engine (const char *msg)
@@ -375,9 +372,8 @@ mono_get_exception_execution_engine (const char *msg)
 
 /**
  * mono_get_exception_serialization:
- * @msg: the message to pass to the user
- *
- * Returns: a new instance of the `System.Runtime.Serialization.SerializationException`
+ * \param msg the message to pass to the user
+ * \returns a new instance of the \c System.Runtime.Serialization.SerializationException
  */
 MonoException *
 mono_get_exception_serialization (const char *msg)
@@ -387,8 +383,7 @@ mono_get_exception_serialization (const char *msg)
 
 /**
  * mono_get_exception_invalid_cast:
- *
- * Returns: a new instance of the `System.InvalidCastException`
+ * \returns a new instance of the \c System.InvalidCastException
  */
 MonoException *
 mono_get_exception_invalid_cast ()
@@ -398,9 +393,8 @@ mono_get_exception_invalid_cast ()
 
 /**
  * mono_get_exception_invalid_operation:
- * @msg: the message to pass to the user
- *
- * Returns: a new instance of the `System.InvalidOperationException`
+ * \param msg the message to pass to the user
+ * \returns a new instance of the \c System.InvalidOperationException
  */
 MonoException *
 mono_get_exception_invalid_operation (const char *msg)
@@ -411,8 +405,7 @@ mono_get_exception_invalid_operation (const char *msg)
 
 /**
  * mono_get_exception_index_out_of_range:
- *
- * Returns: a new instance of the `System.IndexOutOfRangeException`
+ * \returns a new instance of the \c System.IndexOutOfRangeException
  */
 MonoException *
 mono_get_exception_index_out_of_range ()
@@ -423,8 +416,7 @@ mono_get_exception_index_out_of_range ()
 
 /**
  * mono_get_exception_array_type_mismatch:
- *
- * Returns: a new instance of the `System.ArrayTypeMismatchException`
+ * \returns a new instance of the \c System.ArrayTypeMismatchException
  */
 MonoException *
 mono_get_exception_array_type_mismatch ()
@@ -435,17 +427,21 @@ mono_get_exception_array_type_mismatch ()
 
 /**
  * mono_get_exception_type_load:
- * @class_name: the name of the class that could not be loaded
- * @assembly_name: the assembly where the class was looked up.
- *
- * Returns: a new instance of the `System.TypeLoadException`
+ * \param class_name the name of the class that could not be loaded
+ * \param assembly_name the assembly where the class was looked up.
+ * \returns a new instance of the \c System.TypeLoadException
  */
 MonoException *
 mono_get_exception_type_load (MonoString *class_name, char *assembly_name)
 {
-       MonoString *s = assembly_name ? mono_string_new (mono_domain_get (), assembly_name) : mono_string_new (mono_domain_get (), "");
-
        MonoError error;
+       MonoString *s = NULL;
+       if (assembly_name) {
+               s = mono_string_new_checked (mono_domain_get (), assembly_name, &error);
+               mono_error_assert_ok (&error);
+       } else
+               s = mono_string_empty (mono_domain_get ());
+
        MonoException *ret = mono_exception_from_name_two_strings_checked (mono_get_corlib (), "System",
                                                                   "TypeLoadException", class_name, s, &error);
        mono_error_assert_ok (&error);
@@ -454,9 +450,8 @@ mono_get_exception_type_load (MonoString *class_name, char *assembly_name)
 
 /**
  * mono_get_exception_not_implemented:
- * @msg: the message to pass to the user
- *
- * Returns: a new instance of the `System.NotImplementedException`
+ * \param msg the message to pass to the user
+ * \returns a new instance of the \c System.NotImplementedException
  */
 MonoException *
 mono_get_exception_not_implemented (const char *msg)
@@ -466,9 +461,8 @@ mono_get_exception_not_implemented (const char *msg)
 
 /**
  * mono_get_exception_not_supported:
- * @msg: the message to pass to the user
- *
- * Returns: a new instance of the `System.NotSupportedException`
+ * \param msg the message to pass to the user
+ * \returns a new instance of the \c System.NotSupportedException
  */
 MonoException *
 mono_get_exception_not_supported (const char *msg)
@@ -478,18 +472,19 @@ mono_get_exception_not_supported (const char *msg)
 
 /**
  * mono_get_exception_missing_method:
- * @class_name: the class where the lookup was performed.
- * @member_name: the name of the missing method.
- *
- * Returns: a new instance of the `System.MissingMethodException`
+ * \param class_name the class where the lookup was performed.
+ * \param member_name the name of the missing method.
+ * \returns a new instance of the \c System.MissingMethodException
  */
 MonoException *
 mono_get_exception_missing_method (const char *class_name, const char *member_name)
 {
-       MonoString *s1 = mono_string_new (mono_domain_get (), class_name);
-       MonoString *s2 = mono_string_new (mono_domain_get (), member_name);
-
        MonoError error;
+       MonoString *s1 = mono_string_new_checked (mono_domain_get (), class_name, &error);
+       mono_error_assert_ok (&error);
+       MonoString *s2 = mono_string_new_checked (mono_domain_get (), member_name, &error);
+       mono_error_assert_ok (&error);
+
        MonoException *ret = mono_exception_from_name_two_strings_checked (mono_get_corlib (), "System",
                                                                           "MissingMethodException", s1, s2, &error);
        mono_error_assert_ok (&error);
@@ -498,18 +493,19 @@ mono_get_exception_missing_method (const char *class_name, const char *member_na
 
 /**
  * mono_get_exception_missing_field:
- * @class_name: the class where the lookup was performed
- * @member_name: the name of the missing method.
- *
- * Returns: a new instance of the `System.MissingFieldException`
+ * \param class_name the class where the lookup was performed
+ * \param member_name the name of the missing method.
+ * \returns a new instance of the \c System.MissingFieldException
  */
 MonoException *
 mono_get_exception_missing_field (const char *class_name, const char *member_name)
 {
-       MonoString *s1 = mono_string_new (mono_domain_get (), class_name);
-       MonoString *s2 = mono_string_new (mono_domain_get (), member_name);
-
        MonoError error;
+       MonoString *s1 = mono_string_new_checked (mono_domain_get (), class_name, &error);
+       mono_error_assert_ok (&error);
+       MonoString *s2 = mono_string_new_checked (mono_domain_get (), member_name, &error);
+       mono_error_assert_ok (&error);
+
        MonoException *ret = mono_exception_from_name_two_strings_checked (mono_get_corlib (), "System",
                                                                   "MissingFieldException", s1, s2, &error);
        mono_error_assert_ok (&error);
@@ -518,9 +514,8 @@ mono_get_exception_missing_field (const char *class_name, const char *member_nam
 
 /**
  * mono_get_exception_argument_null:
- * @arg: the name of the argument that is null
- *
- * Returns: a new instance of the `System.ArgumentNullException`
+ * \param arg the name of the argument that is null
+ * \returns a new instance of the \c System.ArgumentNullException
  */
 MonoException*
 mono_get_exception_argument_null (const char *arg)
@@ -531,8 +526,11 @@ mono_get_exception_argument_null (const char *arg)
                mono_get_corlib (), "System", "ArgumentNullException");
 
        if (arg) {
+               MonoError error;
                MonoArgumentException *argex = (MonoArgumentException *)ex;
-               MONO_OBJECT_SETREF (argex, param_name, mono_string_new (mono_object_get_domain ((MonoObject*)ex), arg));
+               MonoString *arg_str = mono_string_new_checked (mono_object_get_domain ((MonoObject*)ex), arg, &error);
+               mono_error_assert_ok (&error);
+               MONO_OBJECT_SETREF (argex, param_name, arg_str);
        }
        
        return ex;
@@ -540,9 +538,8 @@ mono_get_exception_argument_null (const char *arg)
 
 /**
  * mono_get_exception_argument:
- * @arg: the name of the invalid argument.
- *
- * Returns: a new instance of the `System.ArgumentException`
+ * \param arg the name of the invalid argument.
+ * \returns a new instance of the \c System.ArgumentException
  */
 MonoException *
 mono_get_exception_argument (const char *arg, const char *msg)
@@ -553,8 +550,11 @@ mono_get_exception_argument (const char *arg, const char *msg)
                mono_get_corlib (), "System", "ArgumentException", msg);
 
        if (arg) {
+               MonoError error;
                MonoArgumentException *argex = (MonoArgumentException *)ex;
-               MONO_OBJECT_SETREF (argex, param_name, mono_string_new (mono_object_get_domain ((MonoObject*)ex), arg));
+               MonoString *arg_str = mono_string_new_checked (mono_object_get_domain ((MonoObject*)ex), arg, &error);
+               mono_error_assert_ok (&error);
+               MONO_OBJECT_SETREF (argex, param_name, arg_str);
        }
        
        return ex;
@@ -562,9 +562,8 @@ mono_get_exception_argument (const char *arg, const char *msg)
 
 /**
  * mono_get_exception_argument_out_of_range:
- * @arg: the name of the out of range argument.
- *
- * Returns: a new instance of the `System.ArgumentOutOfRangeException`
+ * \param arg the name of the out of range argument.
+ * \returns a new instance of the \c System.ArgumentOutOfRangeException
  */
 MonoException *
 mono_get_exception_argument_out_of_range (const char *arg)
@@ -575,8 +574,11 @@ mono_get_exception_argument_out_of_range (const char *arg)
                mono_get_corlib (), "System", "ArgumentOutOfRangeException");
 
        if (arg) {
+               MonoError error;
                MonoArgumentException *argex = (MonoArgumentException *)ex;
-               MONO_OBJECT_SETREF (argex, param_name, mono_string_new (mono_object_get_domain ((MonoObject*)ex), arg));
+               MonoString *arg_str = mono_string_new_checked (mono_object_get_domain ((MonoObject*)ex), arg, &error);
+               mono_error_assert_ok (&error);
+               MONO_OBJECT_SETREF (argex, param_name, arg_str);
        }
        
        return ex;
@@ -584,9 +586,8 @@ mono_get_exception_argument_out_of_range (const char *arg)
 
 /**
  * mono_get_exception_thread_state:
- * @msg: the message to present to the user
- *
- * Returns: a new instance of the `System.Threading.ThreadStateException`
+ * \param msg the message to present to the user
+ * \returns a new instance of the \c System.Threading.ThreadStateException
  */
 MonoException *
 mono_get_exception_thread_state (const char *msg)
@@ -597,9 +598,8 @@ mono_get_exception_thread_state (const char *msg)
 
 /**
  * mono_get_exception_io:
- * @msg: the message to present to the user
- *
- * Returns: a new instance of the `System.IO.IOException`
+ * \param msg the message to present to the user
+ * \returns a new instance of the \c System.IO.IOException
  */
 MonoException *
 mono_get_exception_io (const char *msg)
@@ -610,9 +610,8 @@ mono_get_exception_io (const char *msg)
 
 /**
  * mono_get_exception_file_not_found:
- * @fname: the name of the file not found.
- *
- * Returns: a new instance of the `System.IO.FileNotFoundException`
+ * \param fname the name of the file not found.
+ * \returns a new instance of the \c System.IO.FileNotFoundException
  */
 MonoException *
 mono_get_exception_file_not_found (MonoString *fname)
@@ -626,17 +625,20 @@ mono_get_exception_file_not_found (MonoString *fname)
 
 /**
  * mono_get_exception_file_not_found2:
- * @msg: an informative message for the user.
- * @fname: the name of the file not found.
- *
- * Returns: a new instance of the `System.IO.FileNotFoundException`
+ * \param msg an informative message for the user.
+ * \param fname the name of the file not found.
+ * \returns a new instance of the \c System.IO.FileNotFoundException
  */
 MonoException *
 mono_get_exception_file_not_found2 (const char *msg, MonoString *fname)
 {
-       MonoString *s = msg ? mono_string_new (mono_domain_get (), msg) : NULL;
-
        MonoError error;
+       MonoString *s = NULL;
+       if (msg) {
+               s = mono_string_new_checked (mono_domain_get (), msg, &error);
+               mono_error_assert_ok (&error);
+       }
+
        MonoException *ret = mono_exception_from_name_two_strings_checked (
                mono_get_corlib (), "System.IO", "FileNotFoundException", s, fname, &error);
        mono_error_assert_ok (&error);
@@ -645,10 +647,9 @@ mono_get_exception_file_not_found2 (const char *msg, MonoString *fname)
 
 /**
  * mono_get_exception_type_initialization:
- * @type_name: the name of the type that failed to initialize.
- * @inner: the inner exception.
- *
- * Returns: a new instance of the `System.TypeInitializationException`
+ * \param type_name the name of the type that failed to initialize.
+ * \param inner the inner exception.
+ * \returns a new instance of the \c System.TypeInitializationException
  */
 MonoException *
 mono_get_exception_type_initialization (const gchar *type_name, MonoException *inner)
@@ -672,6 +673,8 @@ mono_get_exception_type_initialization_checked (const gchar *type_name, MonoExce
        MonoMethod *method;
        gpointer iter;
 
+       error_init (error);
+
        klass = mono_class_load_from_name (mono_get_corlib (), "System", "TypeInitializationException");
 
        mono_class_init (klass);
@@ -688,7 +691,9 @@ mono_get_exception_type_initialization_checked (const gchar *type_name, MonoExce
        }
        g_assert (method);
 
-       args [0] = mono_string_new (mono_domain_get (), type_name);
+       MonoString *type_name_str = mono_string_new_checked (mono_domain_get (), type_name, error);
+       mono_error_assert_ok (error);
+       args [0] = type_name_str;
        args [1] = inner;
 
        exc = mono_object_new_checked (mono_domain_get (), klass, error);
@@ -702,9 +707,8 @@ mono_get_exception_type_initialization_checked (const gchar *type_name, MonoExce
 
 /**
  * mono_get_exception_synchronization_lock:
- * @inner: the inner exception.
- *
- * Returns: a new instance of the `System.SynchronizationLockException`
+ * \param inner the inner exception.
+ * \returns a new instance of the \c System.SynchronizationLockException
  */
 MonoException *
 mono_get_exception_synchronization_lock (const char *msg)
@@ -714,9 +718,8 @@ mono_get_exception_synchronization_lock (const char *msg)
 
 /**
  * mono_get_exception_cannot_unload_appdomain:
- * @inner: the inner exception.
- *
- * Returns: a new instance of the `System.CannotUnloadAppDomainException`
+ * \param inner the inner exception.
+ * \returns a new instance of the \c System.CannotUnloadAppDomainException
  */
 MonoException *
 mono_get_exception_cannot_unload_appdomain (const char *msg)
@@ -726,8 +729,7 @@ mono_get_exception_cannot_unload_appdomain (const char *msg)
 
 /**
  * mono_get_exception_appdomain_unloaded
- *
- * Returns: a new instance of the `System.AppDomainUnloadedException`
+ * \returns a new instance of the \c System.AppDomainUnloadedException
  */
 MonoException *
 mono_get_exception_appdomain_unloaded (void)
@@ -737,9 +739,8 @@ mono_get_exception_appdomain_unloaded (void)
 
 /**
  * mono_get_exception_bad_image_format:
- * @msg: an informative message for the user.
- *
- * Returns: a new instance of the `System.BadImageFormatException`
+ * \param msg an informative message for the user.
+ * \returns a new instance of the \c System.BadImageFormatException
  */
 MonoException *
 mono_get_exception_bad_image_format (const char *msg)
@@ -749,17 +750,21 @@ mono_get_exception_bad_image_format (const char *msg)
 
 /**
  * mono_get_exception_bad_image_format2:
- * @msg: an informative message for the user.
- * @fname: The full name of the file with the invalid image.
- *
- * Returns: a new instance of the `System.BadImageFormatException`
+ * \param msg an informative message for the user.
+ * \param fname The full name of the file with the invalid image.
+ * \returns a new instance of the \c System.BadImageFormatException
  */
 MonoException *
 mono_get_exception_bad_image_format2 (const char *msg, MonoString *fname)
 {
-       MonoString *s = msg ? mono_string_new (mono_domain_get (), msg) : NULL;
-
        MonoError error;
+       MonoString *s = NULL;
+
+       if (msg) {
+               s = mono_string_new_checked (mono_domain_get (), msg, &error);
+               mono_error_assert_ok (&error);
+       }
+
        MonoException *ret = mono_exception_from_name_two_strings_checked (
                mono_get_corlib (), "System", "BadImageFormatException", s, fname, &error);
        mono_error_assert_ok (&error);
@@ -768,8 +773,7 @@ mono_get_exception_bad_image_format2 (const char *msg, MonoString *fname)
 
 /**
  * mono_get_exception_stack_overflow:
- *
- * Returns: a new instance of the `System.StackOverflowException`
+ * \returns a new instance of the \c System.StackOverflowException
  */
 MonoException *
 mono_get_exception_stack_overflow (void)
@@ -779,8 +783,7 @@ mono_get_exception_stack_overflow (void)
 
 /**
  * mono_get_exception_out_of_memory:
- *
- * Returns: a new instance of the `System.OutOfMemoryException`
+ * \returns a new instance of the \c System.OutOfMemoryException
  */
 MonoException *
 mono_get_exception_out_of_memory (void)
@@ -790,8 +793,7 @@ mono_get_exception_out_of_memory (void)
 
 /**
  * mono_get_exception_field_access:
- *
- * Returns: a new instance of the `System.FieldAccessException`
+ * \returns a new instance of the \c System.FieldAccessException
  */
 MonoException *
 mono_get_exception_field_access (void)
@@ -801,9 +803,8 @@ mono_get_exception_field_access (void)
 
 /**
  * mono_get_exception_field_access2:
- * @msg: an informative message for the user.
- *
- * Returns: a new instance of the `System.FieldAccessException`
+ * \param msg an informative message for the user.
+ * \returns a new instance of the \c System.FieldAccessException
  */
 MonoException *
 mono_get_exception_field_access_msg (const char *msg)
@@ -813,8 +814,7 @@ mono_get_exception_field_access_msg (const char *msg)
 
 /**
  * mono_get_exception_method_access:
- *
- * Returns: a new instance of the `System.MethodAccessException`
+ * \returns a new instance of the \c System.MethodAccessException
  */
 MonoException *
 mono_get_exception_method_access (void)
@@ -824,9 +824,8 @@ mono_get_exception_method_access (void)
 
 /**
  * mono_get_exception_method_access2:
- * @msg: an informative message for the user.
- *
- * Returns: a new instance of the `System.MethodAccessException`
+ * \param msg an informative message for the user.
+ * \returns a new instance of the \c System.MethodAccessException
  */
 MonoException *
 mono_get_exception_method_access_msg (const char *msg)
@@ -836,10 +835,9 @@ mono_get_exception_method_access_msg (const char *msg)
 
 /**
  * mono_get_exception_reflection_type_load:
- * @types: an array of types that were defined in the moduled loaded.
- * @exceptions: an array of exceptions that were thrown during the type loading.
- *
- * Returns: a new instance of the `System.Reflection.ReflectionTypeLoadException`
+ * \param types an array of types that were defined in the moduled loaded.
+ * \param exceptions an array of exceptions that were thrown during the type loading.
+ * \returns a new instance of the \c System.Reflection.ReflectionTypeLoadException
  */
 MonoException *
 mono_get_exception_reflection_type_load (MonoArray *types_raw, MonoArray *exceptions_raw)
@@ -867,7 +865,7 @@ mono_get_exception_reflection_type_load_checked (MonoArrayHandle types, MonoArra
        MonoMethod *method;
        gpointer iter;
 
-       mono_error_init (error);
+       error_init (error);
 
        klass = mono_class_load_from_name (mono_get_corlib (), "System.Reflection", "ReflectionTypeLoadException");
 
@@ -899,6 +897,9 @@ mono_get_exception_reflection_type_load_checked (MonoArrayHandle types, MonoArra
        return exc;
 }
 
+/**
+ * mono_get_exception_runtime_wrapped:
+ */
 MonoException *
 mono_get_exception_runtime_wrapped (MonoObject *wrapped_exception)
 {
@@ -1017,7 +1018,7 @@ ves_icall_Mono_Runtime_GetNativeStackTrace (MonoExceptionHandle exc, MonoError *
 {
        char *trace;
        MonoStringHandle res;
-       mono_error_init (error);
+       error_init (error);
 
        if (!exc) {
                mono_error_set_argument_null (error, "exception", "");
@@ -1032,13 +1033,13 @@ ves_icall_Mono_Runtime_GetNativeStackTrace (MonoExceptionHandle exc, MonoError *
 
 /**
  * mono_error_raise_exception:
- * @target_error: the exception to raise
+ * \param target_error the exception to raise
  *
- * Raises the exception of @target_error.
- * Does nothing if @target_error has a success error code.
+ * Raises the exception of \p target_error.
+ * Does nothing if \p target_error has a success error code.
  * Aborts in case of a double fault. This happens when it can't recover from an error caused by trying
  * to construct the first exception object.
- * The error object @target_error is cleaned up.
+ * The error object \p target_error is cleaned up.
 */
 void
 mono_error_raise_exception (MonoError *target_error)
@@ -1050,11 +1051,9 @@ mono_error_raise_exception (MonoError *target_error)
 
 /**
  * mono_error_set_pending_exception:
- * @error: The error
- *
- *
- * If @error is set, convert it to an exception and set the pending exception for the current icall.
- * Returns TRUE if @error was set, or FALSE otherwise, so that you can write:
+ * \param error The error
+ * If \p error is set, convert it to an exception and set the pending exception for the current icall.
+ * \returns TRUE if \p error was set, or FALSE otherwise, so that you can write:
  *    if (mono_error_set_pending_exception (error)) {
  *      { ... cleanup code ... }
  *      return;
@@ -1073,7 +1072,7 @@ mono_error_set_pending_exception (MonoError *error)
 }
 
 void
-mono_install_unhandled_exception_hook (MonoUnhandledExceptionFunc func, gpointer user_data)
+mono_install_unhandled_exception_hook (MonoUnhandledExceptionFunc func, void *user_data)
 {
        unhandled_exception_hook = func;
        unhandled_exception_hook_data = user_data;