From 37a8e7e37012039759e13248508b2c758f11b8a8 Mon Sep 17 00:00:00 2001
From: Wouter Wijngaards <wouter@nlnetlabs.nl>
Date: Tue, 12 Jun 2007 14:51:49 +0000
Subject: [PATCH] fixup comments for doxygen 1.5

git-svn-id: file:///svn/unbound/trunk@383 be551aaa-1e26-0410-a405-d3ace91eadb9
---
 daemon/daemon.c            |  5 ++++-
 daemon/unbound.c           | 10 ++++++++--
 daemon/worker.c            |  7 ++++++-
 doc/Changelog              |  1 +
 services/cache/dns.c       |  6 +++++-
 services/outside_network.c | 10 +++++-----
 testcode/checklocks.c      | 16 ++++++++--------
 testcode/fake_event.c      |  2 +-
 testcode/ldns-testpkts.c   |  2 +-
 testcode/lock_verify.c     |  2 +-
 testcode/pktview.c         |  4 ++--
 testcode/replay.c          |  2 +-
 testcode/testbound.c       |  2 +-
 testcode/unitlruhash.c     |  6 +++---
 testcode/unitmsgparse.c    |  8 ++++----
 testcode/unitslabhash.c    |  6 +++---
 util/config_file.c         |  2 +-
 util/config_file.h         |  4 ++--
 util/data/dname.c          |  2 +-
 util/data/msgparse.c       |  6 +++---
 util/data/msgreply.c       |  6 +++---
 util/data/msgreply.h       |  4 ++--
 util/locks.c               |  2 +-
 util/net_help.c            |  2 +-
 util/netevent.c            |  6 +++---
 util/rbtree.c              | 10 +++++-----
 util/region-allocator.h    |  4 ++--
 util/storage/lruhash.h     |  2 +-
 28 files changed, 79 insertions(+), 60 deletions(-)

diff --git a/daemon/daemon.c b/daemon/daemon.c
index 26b42a53f..66ebb7181 100644
--- a/daemon/daemon.c
+++ b/daemon/daemon.c
@@ -62,7 +62,10 @@ static int sig_record_quit = 0;
 static int sig_record_reload = 0;
 
 /** used when no other sighandling happens, so we don't die
-  * when multiple signals in quick succession are sent to us. */
+  * when multiple signals in quick succession are sent to us. 
+  * @param sig: signal number.
+  * @return signal handler return type (void or int).
+  */
 static RETSIGTYPE record_sigh(int sig)
 {
 #ifdef LIBEVENT_SIGNAL_PROBLEM
diff --git a/daemon/unbound.c b/daemon/unbound.c
index 48780640e..8f94acf19 100644
--- a/daemon/unbound.c
+++ b/daemon/unbound.c
@@ -142,7 +142,10 @@ apply_dir(struct daemon* daemon, struct config_file* cfg, int cmdline_verbose)
 	checkrlimits(cfg);
 }
 
-/** Read existing pid from pidfile. */
+/** Read existing pid from pidfile. 
+ * @param file: file name of pid file.
+ * @return: the pid from the file or -1 if none.
+ */
 static pid_t
 readpid (const char* file)
 {
@@ -182,7 +185,10 @@ readpid (const char* file)
 	return pid;
 }
 
-/** write pid to file. */
+/** write pid to file. 
+ * @param pidfile: file name of pid file.
+ * @param pid: pid to write to file.
+ */
 static void
 writepid (const char* pidfile, pid_t pid)
 {
diff --git a/daemon/worker.c b/daemon/worker.c
index b414db09c..b039888d2 100644
--- a/daemon/worker.c
+++ b/daemon/worker.c
@@ -447,7 +447,12 @@ worker_check_request(ldns_buffer* pkt)
 	return 0;
 }
 
-/** process control messages from the main thread. */
+/** process control messages from the main thread. 
+ * @param c: comm point to read from.
+ * @param arg: worker.
+ * @param error: error status of comm point.
+ * @param reply_info: not used.
+ */
 static int 
 worker_handle_control_cmd(struct comm_point* c, void* arg, int error, 
 	struct comm_reply* ATTR_UNUSED(reply_info))
diff --git a/doc/Changelog b/doc/Changelog
index ae4d1e665..c0523a649 100644
--- a/doc/Changelog
+++ b/doc/Changelog
@@ -4,6 +4,7 @@
 	- synthesize CNAME and DNAME responses from the cache.
 	- Updated doxygen config for doxygen 1.5.
 	- aclocal newer version.
+	- doxygen 1.5 fixes for comments (for the strict check on docs).
 
 11 June 2007: Wouter
 	- replies on TCP queries have the address field set in replyinfo,
diff --git a/services/cache/dns.c b/services/cache/dns.c
index 7dc9b08d6..14cd72263 100644
--- a/services/cache/dns.c
+++ b/services/cache/dns.c
@@ -49,7 +49,11 @@
 #include "util/net_help.h"
 #include "util/region-allocator.h"
 
-/** store rrsets in the rrset cache. */
+/** store rrsets in the rrset cache. 
+ * @param env: module environment with caches.
+ * @param rep: contains list of rrsets to store.
+ * @param now: current time(NULL).
+ */
 static void
 store_rrsets(struct module_env* env, struct reply_info* rep, uint32_t now)
 {
diff --git a/services/outside_network.c b/services/outside_network.c
index b1a2c5a88..c0f2c1e74 100644
--- a/services/outside_network.c
+++ b/services/outside_network.c
@@ -62,7 +62,7 @@
 /** number of retries on outgoing UDP queries */
 #define OUTBOUND_UDP_RETRY 4
 
-/** callback for serviced query UDP answers. */
+/** callback for serviced query UDP answers */
 static int serviced_udp_callback(struct comm_point* c, void* arg, int error,
         struct comm_reply* rep);
 /** initiate TCP transaction for serviced query */
@@ -205,7 +205,7 @@ outnet_tcp_take_into_use(struct waiting_tcp* w, uint8_t* pkt)
 	return 1;
 }
 
-/** see if buffers can be used to service TCP queries. */
+/** see if buffers can be used to service TCP queries */
 static void
 use_free_buffer(struct outside_network* outnet)
 {
@@ -260,7 +260,7 @@ outnet_tcp_cb(struct comm_point* c, void* arg, int error,
 	return 0;
 }
 
-/** callback for incoming udp answers from the network. */
+/** callback for incoming udp answers from the network */
 static int 
 outnet_udp_cb(struct comm_point* c, void* arg, int error,
 	struct comm_reply *reply_info)
@@ -387,7 +387,7 @@ make_udp_range(struct comm_point** coms, const char* ifname,
 	return done;
 }
 
-/** calculate number of ip4 and ip6 interfaces, times multiplier. */
+/** calculate number of ip4 and ip6 interfaces, times multiplier */
 static void 
 calc_num46(char** ifs, int num_ifs, int do_ip4, int do_ip6, 
 	size_t multiplier, size_t* num_ip4, size_t* num_ip6)
@@ -822,7 +822,7 @@ pending_tcp_query(struct outside_network* outnet, ldns_buffer* packet,
 	return w;
 }
 
-/** create query for serviced queries. */
+/** create query for serviced queries */
 static void
 serviced_gen_query(ldns_buffer* buff, uint8_t* qname, size_t qnamelen, 
 	uint16_t qtype, uint16_t qclass, uint16_t flags)
diff --git a/testcode/checklocks.c b/testcode/checklocks.c
index b85cda088..6d1ac9c83 100644
--- a/testcode/checklocks.c
+++ b/testcode/checklocks.c
@@ -194,7 +194,7 @@ prot_check(struct checked_lock* lock,
 	}
 }
 
-/** Copy protected memory region. */
+/** Copy protected memory region */
 static void 
 prot_store(struct checked_lock* lock)
 {
@@ -205,7 +205,7 @@ prot_store(struct checked_lock* lock)
 	}
 }
 
-/** write lock trace info to file, while you hold those locks. */
+/** write lock trace info to file, while you hold those locks */
 static void
 ordercheck_locklock(struct thr_check* thr, struct checked_lock* lock)
 {
@@ -225,7 +225,7 @@ ordercheck_locklock(struct thr_check* thr, struct checked_lock* lock)
 		log_err("fwrite: %s", strerror(errno));
 }
 
-/** write ordercheck lock creation details to file. */
+/** write ordercheck lock creation details to file */
 static void 
 ordercheck_lockcreate(struct thr_check* thr, struct checked_lock* lock)
 {
@@ -365,7 +365,7 @@ checklock_destroy(enum check_lock_type type, struct checked_lock** lock,
 	free(e);
 }
 
-/** finish acquiring lock, shared between _(rd|wr||)lock() routines. */
+/** finish acquiring lock, shared between _(rd|wr||)lock() routines */
 static void 
 finish_acquire_lock(struct thr_check* thr, struct checked_lock* lock,
         const char* func, const char* file, int line)
@@ -611,7 +611,7 @@ checklock_unlock(enum check_lock_type type, struct checked_lock* lock,
 	}
 }
 
-/** open order info debug file, thr->num must be valid. */
+/** open order info debug file, thr->num must be valid */
 static void 
 open_lockorder(struct thr_check* thr)
 {
@@ -631,7 +631,7 @@ open_lockorder(struct thr_check* thr)
 		log_err("fwrite: %s", strerror(errno));
 }
 
-/** checklock thread main, Inits thread structure. */
+/** checklock thread main, Inits thread structure */
 static void* checklock_main(void* arg)
 {
 	struct thr_check* thr = (struct thr_check*)arg; 
@@ -791,7 +791,7 @@ total_debug_info()
 	}
 }
 
-/** signal handler for join timeout, Exits. */
+/** signal handler for join timeout, Exits */
 static RETSIGTYPE joinalarm(int ATTR_UNUSED(sig))
 {
 	log_err("join thread timeout. hangup or deadlock. Info follows.");
@@ -799,7 +799,7 @@ static RETSIGTYPE joinalarm(int ATTR_UNUSED(sig))
 	fatal_exit("join thread timeout. hangup or deadlock.");
 }
 
-/** wait for thread with a timeout. */
+/** wait for thread with a timeout */
 void 
 checklock_thrjoin(pthread_t thread)
 {
diff --git a/testcode/fake_event.c b/testcode/fake_event.c
index a30640c47..b336deaba 100644
--- a/testcode/fake_event.c
+++ b/testcode/fake_event.c
@@ -73,7 +73,7 @@ fake_event_cleanup()
 	saved_scenario = NULL;
 }
 
-/** helper function that logs a ldns_pkt packet to logfile. */
+/** helper function that logs a ldns_pkt packet to logfile */
 static void
 log_pkt(const char* desc, ldns_pkt* pkt)
 {
diff --git a/testcode/ldns-testpkts.c b/testcode/ldns-testpkts.c
index d090a7222..39a183769 100644
--- a/testcode/ldns-testpkts.c
+++ b/testcode/ldns-testpkts.c
@@ -31,7 +31,7 @@ struct sockaddr_storage;
 /** string to show in warnings and errors */
 static const char* prog_name = "ldns-testpkts";
 
-/** logging routine, provided by caller. */
+/** logging routine, provided by caller */
 void verbose(int lvl, const char* msg, ...);
 
 /** print error and exit */
diff --git a/testcode/lock_verify.c b/testcode/lock_verify.c
index 18e5a667b..b1801aaff 100644
--- a/testcode/lock_verify.c
+++ b/testcode/lock_verify.c
@@ -309,7 +309,7 @@ static void found_cycle(struct lock_ref* visit, int level)
 	}
 }
 
-/** Detect cycle by comparing visited now with all (unvisited) bigger nodes. */
+/** Detect cycle by comparing visited now with all (unvisited) bigger nodes */
 static int detect_cycle(struct lock_ref* visit, struct lock_ref* from)
 {
 	struct lock_ref* p = from;
diff --git a/testcode/pktview.c b/testcode/pktview.c
index 4c66ff2e3..b059f5266 100644
--- a/testcode/pktview.c
+++ b/testcode/pktview.c
@@ -67,7 +67,7 @@ void read_input(ldns_buffer* pkt, FILE* in)
 	hex_to_buf(pkt, buf);
 }
 
-/** analyze domain name in packet, possibly compressed. */
+/** analyze domain name in packet, possibly compressed */
 void analyze_dname(ldns_buffer* pkt)
 {
 	size_t oldpos = ldns_buffer_position(pkt);
@@ -117,7 +117,7 @@ void analyze_rdata(ldns_buffer*pkt, const ldns_rr_descriptor* desc,
 	ldns_buffer_skip(pkt, (ssize_t)rdlen);
 }
 
-/** analyze rr in packet. */
+/** analyze rr in packet */
 void analyze_rr(ldns_buffer* pkt, int q)
 {
 	uint16_t type, dclass, len;
diff --git a/testcode/replay.c b/testcode/replay.c
index 13be63d03..4ba20dd39 100644
--- a/testcode/replay.c
+++ b/testcode/replay.c
@@ -210,7 +210,7 @@ replay_moment_read(char* remain, FILE* in, const char* name, int* lineno,
 	return mom;
 }
 
-/** makes scenario with title on rest of line. */
+/** makes scenario with title on rest of line */
 static struct replay_scenario*
 make_scenario(char* line)
 {
diff --git a/testcode/testbound.c b/testcode/testbound.c
index 64cad1619..b1f8dafef 100644
--- a/testcode/testbound.c
+++ b/testcode/testbound.c
@@ -105,7 +105,7 @@ add_opts(char* optarg, int* pass_argc, char* pass_argv[])
 	}
 }
 
-/** pretty print commandline for unbound in this test. */
+/** pretty print commandline for unbound in this test */
 static void
 echo_cmdline(int argc, char* argv[])
 {
diff --git a/testcode/unitlruhash.c b/testcode/unitlruhash.c
index 489df9424..0b56339b9 100644
--- a/testcode/unitlruhash.c
+++ b/testcode/unitlruhash.c
@@ -67,9 +67,9 @@ static void test_delkey(void*, void*, int);
 static void test_deldata(void*, void*);
 /* --- end test representation --- */
 
-/** hash func, very bad to improve collisions. */
+/** hash func, very bad to improve collisions */
 static hashvalue_t myhash(int id) {return (hashvalue_t)id & 0x0f;}
-/** allocate new key, fill in hash. */
+/** allocate new key, fill in hash */
 static struct testkey* newkey(int id) {
 	struct testkey* k = (struct testkey*)calloc(1, sizeof(struct testkey));
 	if(!k) fatal_exit("out of memory");
@@ -469,7 +469,7 @@ test_thr_main(void* arg)
 	return NULL;
 }
 
-/** test hash table access by multiple threads. */
+/** test hash table access by multiple threads */
 static void
 test_threaded_table(struct lruhash* table)
 {
diff --git a/testcode/unitmsgparse.c b/testcode/unitmsgparse.c
index c26391cde..8f7aa2318 100644
--- a/testcode/unitmsgparse.c
+++ b/testcode/unitmsgparse.c
@@ -224,7 +224,7 @@ test_buffers(ldns_buffer* pkt, ldns_buffer* out)
 	return 0;
 }
 
-/** check if unbound formerr equals ldns formerr. */
+/** check if unbound formerr equals ldns formerr */
 static void
 checkformerr(ldns_buffer* pkt)
 {
@@ -303,7 +303,7 @@ testpkt(ldns_buffer* pkt, struct alloc_cache* alloc, ldns_buffer* out,
 	region_destroy(region);
 }
 
-/** simple test of parsing. */
+/** simple test of parsing */
 static void
 simpletest(ldns_buffer* pkt, struct alloc_cache* alloc, ldns_buffer* out)
 {
@@ -354,7 +354,7 @@ simpletest(ldns_buffer* pkt, struct alloc_cache* alloc, ldns_buffer* out)
 	" 64 b9 00 04 c0 3a 80 1e  ");
 }
 
-/** simple test of parsing, pcat file. */
+/** simple test of parsing, pcat file */
 static void
 testfromfile(ldns_buffer* pkt, struct alloc_cache* alloc, ldns_buffer* out,
 	const char* fname)
@@ -381,7 +381,7 @@ testfromfile(ldns_buffer* pkt, struct alloc_cache* alloc, ldns_buffer* out,
 	fclose(in);
 }
 
-/** simple test of parsing, drill file. */
+/** simple test of parsing, drill file */
 static void
 testfromdrillfile(ldns_buffer* pkt, struct alloc_cache* alloc, 
 	ldns_buffer* out, const char* fname)
diff --git a/testcode/unitslabhash.c b/testcode/unitslabhash.c
index 32d17838a..933b28bb0 100644
--- a/testcode/unitslabhash.c
+++ b/testcode/unitslabhash.c
@@ -67,14 +67,14 @@ static void test_delkey(void*, void*, int);
 static void test_deldata(void*, void*);
 /* --- end test representation --- */
 
-/** hash func, very bad to improve collisions, both high and low bits. */
+/** hash func, very bad to improve collisions, both high and low bits */
 static hashvalue_t myhash(int id) {
 	hashvalue_t h = (hashvalue_t)id & 0x0f;
 	h |= (h << 28);
 	return h;
 }
 
-/** allocate new key, fill in hash. */
+/** allocate new key, fill in hash */
 static struct slabtestkey* newkey(int id) {
 	struct slabtestkey* k = (struct slabtestkey*)calloc(1, sizeof(struct slabtestkey));
 	if(!k) fatal_exit("out of memory");
@@ -351,7 +351,7 @@ test_thr_main(void* arg)
 	return NULL;
 }
 
-/** test hash table access by multiple threads. */
+/** test hash table access by multiple threads */
 static void
 test_threaded_table(struct slabhash* table)
 {
diff --git a/util/config_file.c b/util/config_file.c
index cda0f8004..7e7aa1c69 100644
--- a/util/config_file.c
+++ b/util/config_file.c
@@ -105,7 +105,7 @@ error_exit:
 	return NULL;
 }
 
-/** initialize the global cfg_parser object. */
+/** initialize the global cfg_parser object */
 static void
 create_cfg_parser(struct config_file* cfg, char* filename)
 {
diff --git a/util/config_file.h b/util/config_file.h
index 4df0c88d4..ecb12dc19 100644
--- a/util/config_file.h
+++ b/util/config_file.h
@@ -184,9 +184,9 @@ struct config_parser_state {
 
 /** global config parser object used during config parsing */
 extern struct config_parser_state* cfg_parser;
-/** parsing helpers: print error with file and line numbers. */
+/** parsing helpers: print error with file and line numbers */
 void ub_c_error(const char* msg);
-/** parsing helpers: print error with file and line numbers. */
+/** parsing helpers: print error with file and line numbers */
 void ub_c_error_msg(const char* fmt, ...) ATTR_FORMAT(printf, 1, 2);
 
 #endif /* UTIL_CONFIG_FILE_H */
diff --git a/util/data/dname.c b/util/data/dname.c
index c3542cbbc..5709b475f 100644
--- a/util/data/dname.c
+++ b/util/data/dname.c
@@ -45,7 +45,7 @@
 #include "util/log.h"
 #include "util/storage/lookup3.h"
 
-/** determine length of a dname in buffer, no compression pointers allowed. */
+/** determine length of a dname in buffer, no compression pointers allowed */
 size_t
 query_dname_len(ldns_buffer* query)
 {
diff --git a/util/data/msgparse.c b/util/data/msgparse.c
index 61c320740..f390a7cf7 100644
--- a/util/data/msgparse.c
+++ b/util/data/msgparse.c
@@ -44,7 +44,7 @@
 #include "util/storage/lookup3.h"
 #include "util/region-allocator.h"
 
-/** smart comparison of (compressed, valid) dnames from packet. */
+/** smart comparison of (compressed, valid) dnames from packet */
 static int
 smart_compare(ldns_buffer* pkt, uint8_t* dnow, 
 	uint8_t* dprfirst, uint8_t* dprlast)
@@ -96,7 +96,7 @@ new_rrset(struct msg_parse* msg, uint8_t* dname, size_t dnamelen,
 	return p;
 }
 
-/** See if next rrset is nsec at zone apex. */
+/** See if next rrset is nsec at zone apex */
 static int
 nsec_at_apex(ldns_buffer* pkt)
 {
@@ -166,7 +166,7 @@ pkt_hash_rrset(ldns_buffer* pkt, uint8_t* dname, uint16_t type,
 	return h;
 }
 
-/** compare rrset_parse with data. */
+/** compare rrset_parse with data */
 static int
 rrset_parse_equals(struct rrset_parse* p, ldns_buffer* pkt, hashvalue_t h, 
 	uint32_t rrset_flags, uint8_t* dname, size_t dnamelen, 
diff --git a/util/data/msgreply.c b/util/data/msgreply.c
index 859788363..4b2c6e783 100644
--- a/util/data/msgreply.c
+++ b/util/data/msgreply.c
@@ -51,7 +51,7 @@
 #include "util/data/msgparse.h"
 #include "util/data/msgencode.h"
 
-/** allocate qinfo, return 0 on error. */
+/** allocate qinfo, return 0 on error */
 static int
 parse_create_qinfo(ldns_buffer* pkt, struct msg_parse* msg, 
 	struct query_info* qinf, struct region* region)
@@ -103,7 +103,7 @@ construct_reply_info_base(struct region* region, uint16_t flags, size_t qd,
 	return rep;
 }
 
-/** allocate replyinfo, return 0 on error. */
+/** allocate replyinfo, return 0 on error */
 static int
 parse_create_repinfo(struct msg_parse* msg, struct reply_info** rep,
 	struct region* region)
@@ -116,7 +116,7 @@ parse_create_repinfo(struct msg_parse* msg, struct reply_info** rep,
 	return 1;
 }
 
-/** allocate (special) rrset keys, return 0 on error. */
+/** allocate (special) rrset keys, return 0 on error */
 static int
 repinfo_alloc_rrset_keys(struct reply_info* rep, struct alloc_cache* alloc, 
 	struct region* region)
diff --git a/util/data/msgreply.h b/util/data/msgreply.h
index 4af25c2d3..d301fb27d 100644
--- a/util/data/msgreply.h
+++ b/util/data/msgreply.h
@@ -251,7 +251,7 @@ int query_info_allocqname(struct query_info* m);
  */
 int query_info_compare(void* m1, void* m2);
 
-/** clear out query info structure. */
+/** clear out query info structure */
 void query_info_clear(struct query_info* m);
 
 /** calculate size of struct query_info + reply_info */
@@ -263,7 +263,7 @@ void query_entry_delete(void *q, void* arg, int is_locked);
 /** delete reply_info data structure */
 void reply_info_delete(void* d, void* arg);
 
-/** calculate hash value of query_info, lowercases the qname. */
+/** calculate hash value of query_info, lowercases the qname */
 hashvalue_t query_info_hash(struct query_info *q);
 
 /**
diff --git a/util/locks.c b/util/locks.c
index 2ceae05d7..e78b22de6 100644
--- a/util/locks.c
+++ b/util/locks.c
@@ -66,7 +66,7 @@ ub_thread_blocksigs()
 #endif /* HAVE_PTHREAD */
 }
 
-/** unblock one signal, so we can catch it. */
+/** unblock one signal, so we can catch it */
 void ub_thread_sig_unblock(int sig)
 {
 	int err;
diff --git a/util/net_help.c b/util/net_help.c
index 3ef69ecbb..7ba79e00a 100644
--- a/util/net_help.c
+++ b/util/net_help.c
@@ -43,7 +43,7 @@
 #include "util/data/dname.h"
 #include <fcntl.h>
 
-/** returns true is string addr is an ip6 specced address. */
+/** returns true is string addr is an ip6 specced address */
 int
 str_is_ip6(const char* str)
 {
diff --git a/util/netevent.c b/util/netevent.c
index b07913056..41adeaadd 100644
--- a/util/netevent.c
+++ b/util/netevent.c
@@ -137,7 +137,7 @@ static void comm_timer_callback(int fd, short event, void* arg);
  */
 static void comm_signal_callback(int fd, short event, void* arg);
 
-/** libevent callback for AF_UNIX fds. */
+/** libevent callback for AF_UNIX fds */
 static void comm_point_local_handle_callback(int fd, short event, void* arg);
 
 /** create a tcp handler with a parent */
@@ -254,7 +254,7 @@ comm_point_udp_callback(int fd, short event, void* arg)
 	}
 }
 
-/** Use a new tcp handler for new query fd, set to read query. */
+/** Use a new tcp handler for new query fd, set to read query */
 static void
 setup_tcp_handler(struct comm_point* c, int fd) 
 {
@@ -310,7 +310,7 @@ comm_point_tcp_accept_callback(int fd, short event, void* arg)
 	setup_tcp_handler(c_hdl, new_fd);
 }
 
-/** Make tcp handler free for next assignment. */
+/** Make tcp handler free for next assignment */
 static void
 reclaim_tcp_handler(struct comm_point* c)
 {
diff --git a/util/rbtree.c b/util/rbtree.c
index 02d978d5e..e4f95df74 100644
--- a/util/rbtree.c
+++ b/util/rbtree.c
@@ -57,9 +57,9 @@ rbnode_t	rbtree_null_node = {
 	BLACK			/* Color.  */
 };
 
-/** rotate subtree left (to preserve redblack property). */
+/** rotate subtree left (to preserve redblack property) */
 static void rbtree_rotate_left(rbtree_t *rbtree, rbnode_t *node);
-/** rotate subtree right (to preserve redblack property). */
+/** rotate subtree right (to preserve redblack property) */
 static void rbtree_rotate_right(rbtree_t *rbtree, rbnode_t *node);
 /** Fixup node colours when insert happened */
 static void rbtree_insert_fixup(rbtree_t *rbtree, rbnode_t *node);
@@ -291,7 +291,7 @@ static void swap_np(rbnode_t** x, rbnode_t** y)
 	rbnode_t* t = *x; *x = *y; *y = t; 
 }
 
-/** Update parent pointers of child trees of 'parent'. */
+/** Update parent pointers of child trees of 'parent' */
 static void change_parent_ptr(rbtree_t* rbtree, rbnode_t* parent, rbnode_t* old, rbnode_t* new)
 {
 	if(parent == RBTREE_NULL)
@@ -305,7 +305,7 @@ static void change_parent_ptr(rbtree_t* rbtree, rbnode_t* parent, rbnode_t* old,
 	if(parent->left == old) parent->left = new;
 	if(parent->right == old) parent->right = new;
 }
-/** Update parent pointer of a node 'child'. */
+/** Update parent pointer of a node 'child' */
 static void change_child_ptr(rbnode_t* child, rbnode_t* old, rbnode_t* new)
 {
 	if(child == RBTREE_NULL) return;
@@ -590,7 +590,7 @@ rbtree_previous(rbnode_t *node)
 	return node;
 }
 
-/** recursive descent traverse. */
+/** recursive descent traverse */
 static void 
 traverse_post(void (*func)(rbnode_t*, void*), void* arg, rbnode_t* node)
 {
diff --git a/util/region-allocator.h b/util/region-allocator.h
index 8c83658b3..3c857cb0c 100644
--- a/util/region-allocator.h
+++ b/util/region-allocator.h
@@ -163,14 +163,14 @@ char *region_strdup(region_type *region, const char *string);
 void region_recycle(region_type *region, void *block, size_t size);
 
 /**
- * Print some REGION statistics to OUT.
+ * Print some REGION statistics to OUT
  */
 void region_dump_stats(region_type *region, FILE *out);
 
 /** get size of recyclebin */
 size_t region_get_recycle_size(region_type* region);
 
-/** Debug print REGION statistics to LOG. */
+/** Debug print REGION statistics to LOG */
 void region_log_stats(region_type *region);
 
 #endif /* _REGION_ALLOCATOR_H_ */
diff --git a/util/storage/lruhash.h b/util/storage/lruhash.h
index 61da21262..81c45de01 100644
--- a/util/storage/lruhash.h
+++ b/util/storage/lruhash.h
@@ -298,7 +298,7 @@ void lru_touch(struct lruhash* table, struct lruhash_entry* entry);
  */
 void lruhash_remove(struct lruhash* table, hashvalue_t hash, void* key);
 
-/** init the hash bins for the table. */
+/** init the hash bins for the table */
 void bin_init(struct lruhash_bin* array, size_t size);
 
 /** delete the hash bin and entries inside it */
-- 
2.47.2