Changeset d1582b50 in mainline for kernel/generic/src

Timestamp:

2020-12-14T20:41:53Z (5 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

02fe42e

Parents:

1382446

git-author:

Jiri Svoboda <jiri@…> (2020-12-14 20:33:54)

git-committer:

Jiri Svoboda <jiri@…> (2020-12-14 20:41:53)

Message:

Fix spacing in single-line comments using latest ccheck

This found incorrectly formatted section comments (with blocks of
asterisks or dashes). I strongly believe against using section comments
but I am not simply removing them since that would probably be
controversial.

Location:

kernel/generic/src

Files:

• adt/hash_table.c (modified) (2 diffs)
• mm/as.c (modified) (1 diff)
• mm/frame.c (modified) (3 diffs)
• mm/slab.c (modified) (4 diffs)
• udebug/udebug_ops.c (modified) (1 diff)

kernel/generic/src/adt/hash_table.c

@@ -331 +331 @@
 }
 
-/** Removes an item already present in the table. The item must be in the table.*/
+/** Removes an item already present in the table. The item must be in the table. */
 void hash_table_remove_item(hash_table_t *h, ht_link_t *item)
 {
@@ -393 +393 @@
 }
 
-/** Allocates and initializes the desired number of buckets. True if successful.*/
+/** Allocates and initializes the desired number of buckets. True if successful. */
 static bool alloc_table(size_t bucket_cnt, list_t **pbuckets)
 {

kernel/generic/src/mm/as.c

@@ -2057 +2057 @@
         /* Check if A is adjacent to the new interval */
         adj_a = (a != NULL) && (a->page + P2SZ(a->count) == page);
-        /* Check if the new interval is adjacent to B*/
+        /* Check if the new interval is adjacent to B */
         adj_b = (b != NULL) && page + P2SZ(count) == b->page;
 

kernel/generic/src/mm/frame.c

@@ -85 +85 @@
 }
 
-/*******************/
-/* Zones functions */
-/*******************/
+/*
+ * Zones functions
+ */
 
 /** Insert-sort zone into zones list.
@@ -341 +341 @@
 }
 
-/******************/
-/* Zone functions */
-/******************/
+/*
+ * Zone functions
+ */
 
 /** Return frame from zone. */
@@ -781 +781 @@
 }
 
-/*******************/
-/* Frame functions */
-/*******************/
+/*
+ * Frame functions
+ */
 
 /** Set parent of frame. */

kernel/generic/src/mm/slab.c

@@ -151 +151 @@
 #endif
 
-/**************************************/
-/* Slab allocation functions          */
-/**************************************/
+/*
+ * Slab allocation functions
+ */
 
 /** Allocate frames for slab space and initialize
@@ -223 +223 @@
 }
 
-/******************/
-/* Slab functions */
-/******************/
+/*
+ * Slab functions
+ */
 
 /** Return object to slab and call a destructor
@@ -323 +323 @@
 }
 
-/****************************/
-/* CPU-Cache slab functions */
-/****************************/
+/*
+ * CPU-Cache slab functions
+ */
 
 /** Find a full magazine in cache, take it from list and return it
@@ -531 +531 @@
 }
 
-/************************/
-/* Slab cache functions */
-/************************/
+/*
+ * Slab cache functions
+ */
 
 /** Return number of objects that fit in certain cache size

kernel/generic/src/udebug/udebug_ops.c

@@ -123 +123 @@
         /* Now verify that the thread belongs to the current task. */
         if (thread->task != TASK) {
-                /* No such thread belonging this task*/
+                /* No such thread belonging this task */
                 mutex_unlock(&TASK->udebug.lock);
                 return ENOENT;