diff --git a/src/env.h b/src/env.h
index 5d2334efc..f5d240b84 100644
--- a/src/env.h
+++ b/src/env.h
@@ -95,7 +95,7 @@ class env_var_t {
 
     env_var_t() = default;
 
-    bool empty() const { return vals.empty() || (vals.size() == 1 && vals[0].empty()); };
+    bool empty() const { return vals.empty() || (vals.size() == 1 && vals[0].empty()); }
     bool read_only() const { return flags & flag_read_only; }
     bool exports() const { return flags & flag_export; }
     bool is_pathvar() const { return flags & flag_pathvar; }
diff --git a/src/lru.h b/src/lru.h
index 31e192ba0..f91c63f74 100644
--- a/src/lru.h
+++ b/src/lru.h
@@ -291,8 +291,8 @@ class lru_cache_t {
         }
     };
 
-    iterator begin() const { return iterator(mouth.prev); };
-    iterator end() const { return iterator(&mouth); };
+    iterator begin() const { return iterator(mouth.prev); }
+    iterator end() const { return iterator(&mouth); }
 
     void check_sanity() const {
         // Check linked list sanity
diff --git a/src/parse_grammar.h b/src/parse_grammar.h
index ce2344fdf..b6b1cb2ee 100644
--- a/src/parse_grammar.h
+++ b/src/parse_grammar.h
@@ -215,7 +215,7 @@ DEF_ALT(job_decorator) {
 
 // A job_conjunction is a job followed by a continuation.
 DEF(job_conjunction) produces_sequence<job, job_conjunction_continuation> {
-    BODY(job_conjunction);
+    BODY(job_conjunction)
 };
 
 DEF_ALT(job_conjunction_continuation) {
@@ -280,7 +280,7 @@ DEF_ALT(case_item_list) {
 };
 
 DEF(case_item) produces_sequence<keyword<parse_keyword_case>, argument_list, tok_end, job_list> {
-    BODY(case_item);
+    BODY(case_item)
 };
 
 DEF(block_statement)
@@ -298,7 +298,7 @@ DEF_ALT(block_header) {
 DEF(for_header)
 produces_sequence<keyword<parse_keyword_for>, tok_string, keyword<parse_keyword_in>, argument_list,
                   tok_end> {
-    BODY(for_header);
+    BODY(for_header)
 };
 
 DEF(while_header)
diff --git a/src/proc.h b/src/proc.h
index c588cf766..30ed6e921 100644
--- a/src/proc.h
+++ b/src/proc.h
@@ -304,9 +304,9 @@ class job_t {
 
     // Helper functions to check presence of flags on instances of jobs
     /// The job has been fully constructed, i.e. all its member processes have been launched
-    bool is_constructed() const { return get_flag(job_flag_t::CONSTRUCTED); };
+    bool is_constructed() const { return get_flag(job_flag_t::CONSTRUCTED); }
     /// The job was launched in the foreground and has control of the terminal
-    bool is_foreground() const { return get_flag(job_flag_t::FOREGROUND); };
+    bool is_foreground() const { return get_flag(job_flag_t::FOREGROUND); }
     /// The job is complete, i.e. all its member processes have been reaped
     bool is_completed() const;
     /// The job is in a stopped state