summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2022-05-03 13:38:17 +0200
committerThomas Haller <thaller@redhat.com>2022-05-09 19:20:18 +0200
commit80a19958dca72aa08670c8d04e26135ab17575df (patch)
tree8538336263fe2504ce30286f0217e9cf2c9ab0d8
parentc6e41b2df35f6d2ad9c8b9d50799dbef3f7e1747 (diff)
downloadNetworkManager-80a19958dca72aa08670c8d04e26135ab17575df.tar.gz
code-style: fix wrong indentation for code comments
-rw-r--r--src/libnm-glib-aux/nm-default-glib.h42
-rw-r--r--src/libnm-std-aux/nm-default-std.h92
2 files changed, 67 insertions, 67 deletions
diff --git a/src/libnm-glib-aux/nm-default-glib.h b/src/libnm-glib-aux/nm-default-glib.h
index 9d04ddc407..40b22ee130 100644
--- a/src/libnm-glib-aux/nm-default-glib.h
+++ b/src/libnm-glib-aux/nm-default-glib.h
@@ -33,27 +33,27 @@
#if NM_MORE_ASSERTS == 0
#ifndef G_DISABLE_CAST_CHECKS
/* Unless compiling with G_DISABLE_CAST_CHECKS, glib performs type checking
- * during G_VARIANT_TYPE() via g_variant_type_checked_(). This is not necessary
- * because commonly this cast is needed during something like
- *
- * g_variant_builder_init (&props, G_VARIANT_TYPE ("a{sv}"));
- *
- * Note that in if the variant type would be invalid, the check still
- * wouldn't make the buggy code magically work. Instead of passing a
- * bogus type string (bad), it would pass %NULL to g_variant_builder_init()
- * (also bad).
- *
- * Also, a function like g_variant_builder_init() already validates
- * the input type via something like
- *
- * g_return_if_fail (g_variant_type_is_container (type));
- *
- * So, by having G_VARIANT_TYPE() also validate the type, we validate
- * twice, whereas the first validation is rather pointless because it
- * doesn't prevent the function to be called with invalid arguments.
- *
- * Just patch G_VARIANT_TYPE() to perform no check.
- */
+ * during G_VARIANT_TYPE() via g_variant_type_checked_(). This is not necessary
+ * because commonly this cast is needed during something like
+ *
+ * g_variant_builder_init (&props, G_VARIANT_TYPE ("a{sv}"));
+ *
+ * Note that in if the variant type would be invalid, the check still
+ * wouldn't make the buggy code magically work. Instead of passing a
+ * bogus type string (bad), it would pass %NULL to g_variant_builder_init()
+ * (also bad).
+ *
+ * Also, a function like g_variant_builder_init() already validates
+ * the input type via something like
+ *
+ * g_return_if_fail (g_variant_type_is_container (type));
+ *
+ * So, by having G_VARIANT_TYPE() also validate the type, we validate
+ * twice, whereas the first validation is rather pointless because it
+ * doesn't prevent the function to be called with invalid arguments.
+ *
+ * Just patch G_VARIANT_TYPE() to perform no check.
+ */
#undef G_VARIANT_TYPE
#define G_VARIANT_TYPE(type_string) ((const GVariantType *) (type_string))
#endif
diff --git a/src/libnm-std-aux/nm-default-std.h b/src/libnm-std-aux/nm-default-std.h
index 23f5fc604b..b4f88059cf 100644
--- a/src/libnm-std-aux/nm-default-std.h
+++ b/src/libnm-std-aux/nm-default-std.h
@@ -42,52 +42,52 @@
#if NM_MORE_ASSERTS == 0
/* The cast macros like NM_TYPE() are implemented via G_TYPE_CHECK_INSTANCE_CAST()
- * and _G_TYPE_CIC(). The latter, by default performs runtime checks of the type
- * by calling g_type_check_instance_cast().
- * This check has a certain overhead without being helpful.
- *
- * Example 1:
- * static void foo (NMType *obj)
- * {
- * access_obj_without_check (obj);
- * }
- * foo ((NMType *) obj);
- * // There is no runtime check and passing an invalid pointer
- * // leads to a crash.
- *
- * Example 2:
- * static void foo (NMType *obj)
- * {
- * access_obj_without_check (obj);
- * }
- * foo (NM_TYPE (obj));
- * // There is a runtime check which prints a g_warning(), but that doesn't
- * // avoid the crash as NM_TYPE() cannot do anything then passing on the
- * // invalid pointer.
- *
- * Example 3:
- * static void foo (NMType *obj)
- * {
- * g_return_if_fail (NM_IS_TYPE (obj));
- * access_obj_without_check (obj);
- * }
- * foo ((NMType *) obj);
- * // There is a runtime check which prints a g_critical() which also avoids
- * // the crash. That is actually helpful to catch bugs and avoid crashes.
- *
- * Example 4:
- * static void foo (NMType *obj)
- * {
- * g_return_if_fail (NM_IS_TYPE (obj));
- * access_obj_without_check (obj);
- * }
- * foo (NM_TYPE (obj));
- * // The runtime check is performed twice, with printing a g_warning() and
- * // a g_critical() and avoiding the crash.
- *
- * Example 3 is how it should be done. Type checks in NM_TYPE() are pointless.
- * Disable them for our production builds.
- */
+ * and _G_TYPE_CIC(). The latter, by default performs runtime checks of the type
+ * by calling g_type_check_instance_cast().
+ * This check has a certain overhead without being helpful.
+ *
+ * Example 1:
+ * static void foo (NMType *obj)
+ * {
+ * access_obj_without_check (obj);
+ * }
+ * foo ((NMType *) obj);
+ * // There is no runtime check and passing an invalid pointer
+ * // leads to a crash.
+ *
+ * Example 2:
+ * static void foo (NMType *obj)
+ * {
+ * access_obj_without_check (obj);
+ * }
+ * foo (NM_TYPE (obj));
+ * // There is a runtime check which prints a g_warning(), but that doesn't
+ * // avoid the crash as NM_TYPE() cannot do anything then passing on the
+ * // invalid pointer.
+ *
+ * Example 3:
+ * static void foo (NMType *obj)
+ * {
+ * g_return_if_fail (NM_IS_TYPE (obj));
+ * access_obj_without_check (obj);
+ * }
+ * foo ((NMType *) obj);
+ * // There is a runtime check which prints a g_critical() which also avoids
+ * // the crash. That is actually helpful to catch bugs and avoid crashes.
+ *
+ * Example 4:
+ * static void foo (NMType *obj)
+ * {
+ * g_return_if_fail (NM_IS_TYPE (obj));
+ * access_obj_without_check (obj);
+ * }
+ * foo (NM_TYPE (obj));
+ * // The runtime check is performed twice, with printing a g_warning() and
+ * // a g_critical() and avoiding the crash.
+ *
+ * Example 3 is how it should be done. Type checks in NM_TYPE() are pointless.
+ * Disable them for our production builds.
+ */
#ifndef G_DISABLE_CAST_CHECKS
#define G_DISABLE_CAST_CHECKS
#endif