aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2019-10-22 09:40:31 -0400
committerNick Mathewson <nickm@torproject.org>2019-10-22 09:40:31 -0400
commit931a5db2a7ded73ddd72cb6646b506d49782f3ec (patch)
tree2f7d2e4850f2e8c698fdc53bded13c23704034cf
parentb2d487ae2249a307cd23255dbb1a22241eefd6ef (diff)
downloadtor-931a5db2a7ded73ddd72cb6646b506d49782f3ec.tar.gz
tor-931a5db2a7ded73ddd72cb6646b506d49782f3ec.zip
Re-run make autostyle.
-rw-r--r--src/app/config/config.c2
-rw-r--r--src/core/mainloop/mainloop.c2
-rw-r--r--src/core/or/circuitpadding.c2
-rw-r--r--src/core/or/or_periodic.c2
-rw-r--r--src/feature/client/proxymode.h2
-rw-r--r--src/feature/control/control_cmd.c2
-rw-r--r--src/feature/control/control_events.c2
-rw-r--r--src/feature/dirauth/dirauth_periodic.c2
-rw-r--r--src/feature/dircommon/consdiff.c2
-rw-r--r--src/feature/dirparse/authcert_members.h2
-rw-r--r--src/feature/hs/hs_service.c2
-rw-r--r--src/feature/relay/relay_periodic.c2
-rw-r--r--src/feature/relay/router.c2
-rw-r--r--src/feature/relay/routermode.h4
-rw-r--r--src/lib/buf/buffers.c2
-rw-r--r--src/lib/conf/confmacros.h2
-rw-r--r--src/lib/conf/conftesting.h6
-rw-r--r--src/lib/container/smartlist.h2
-rw-r--r--src/lib/crypt_ops/crypto_openssl_mgt.c2
-rw-r--r--src/lib/dispatch/dispatch_naming.c2
-rw-r--r--src/lib/evloop/timers.c2
-rw-r--r--src/lib/log/log.h6
-rw-r--r--src/lib/log/util_bug.h2
-rw-r--r--src/lib/thread/threads.h2
-rw-r--r--src/lib/version/git_revision.c2
-rw-r--r--src/lib/wallclock/timeval.h2
-rw-r--r--src/test/test.h6
-rw-r--r--src/test/test_addr.c4
-rw-r--r--src/test/test_confparse.c2
-rw-r--r--src/test/test_connection.c2
-rw-r--r--src/test/test_controller.c2
-rw-r--r--src/test/test_crypto.c2
-rw-r--r--src/test/test_crypto_slow.c2
-rw-r--r--src/test/test_dir.c2
-rw-r--r--src/test/test_entrynodes.c2
-rw-r--r--src/test/test_link_handshake.c2
-rw-r--r--src/test/test_options.c2
-rw-r--r--src/test/test_util.c4
38 files changed, 47 insertions, 47 deletions
diff --git a/src/app/config/config.c b/src/app/config/config.c
index 32ae620b0e..2cf64d1b72 100644
--- a/src/app/config/config.c
+++ b/src/app/config/config.c
@@ -191,7 +191,7 @@ static const char unix_q_socket_prefix[] = "unix:\"";
{ #name "DownloadSchedule", #name "DownloadInitialDelay", 0, 1 }
#else
#define DOWNLOAD_SCHEDULE(name) { NULL, NULL, 0, 1 }
-#endif
+#endif /* !defined(COCCI) */
/** A list of abbreviations and aliases to map command-line options, obsolete
* option names, or alternative option names, to their current values. */
diff --git a/src/core/mainloop/mainloop.c b/src/core/mainloop/mainloop.c
index 0519935a19..33a53e4161 100644
--- a/src/core/mainloop/mainloop.c
+++ b/src/core/mainloop/mainloop.c
@@ -1379,7 +1379,7 @@ CALLBACK(second_elapsed);
#define CALLBACK(name, r, f) \
PERIODIC_EVENT(name, PERIODIC_EVENT_ROLE_ ## r, f)
#define FL(name) (PERIODIC_EVENT_FLAG_ ## name)
-#endif
+#endif /* !defined(COCCI) */
STATIC periodic_event_item_t mainloop_periodic_events[] = {
diff --git a/src/core/or/circuitpadding.c b/src/core/or/circuitpadding.c
index f5481bc241..e9b14144cc 100644
--- a/src/core/or/circuitpadding.c
+++ b/src/core/or/circuitpadding.c
@@ -131,7 +131,7 @@ STATIC smartlist_t *relay_padding_machines = NULL;
if (!(circ)->padding_info[loop_var]) \
continue;
#define FOR_EACH_ACTIVE_CIRCUIT_MACHINE_END } STMT_END ;
-#endif
+#endif /* !defined(COCCI) */
/**
* Free the machineinfo at an index
diff --git a/src/core/or/or_periodic.c b/src/core/or/or_periodic.c
index 879d142cb2..0e79008275 100644
--- a/src/core/or/or_periodic.c
+++ b/src/core/or/or_periodic.c
@@ -26,7 +26,7 @@
PERIODIC_EVENT(name, \
PERIODIC_EVENT_ROLE_##roles, \
flags)
-#endif
+#endif /* !defined(COCCI) */
#define FL(name) (PERIODIC_EVENT_FLAG_ ## name)
diff --git a/src/feature/client/proxymode.h b/src/feature/client/proxymode.h
index f8352922c7..770a519155 100644
--- a/src/feature/client/proxymode.h
+++ b/src/feature/client/proxymode.h
@@ -14,4 +14,4 @@
int proxy_mode(const or_options_t *options);
-#endif
+#endif /* !defined(TOR_PROXYMODE_H) */
diff --git a/src/feature/control/control_cmd.c b/src/feature/control/control_cmd.c
index 2ad7851b31..68179beef0 100644
--- a/src/feature/control/control_cmd.c
+++ b/src/feature/control/control_cmd.c
@@ -2281,7 +2281,7 @@ typedef struct control_cmd_def_t {
0, \
&obsolete_syntax, \
}
-#endif
+#endif /* !defined(COCCI) */
/**
* An array defining all the recognized controller commands.
diff --git a/src/feature/control/control_events.c b/src/feature/control/control_events.c
index b8a46c306f..12b73641b2 100644
--- a/src/feature/control/control_events.c
+++ b/src/feature/control/control_events.c
@@ -1675,7 +1675,7 @@ control_event_status(int type, int severity, const char *format, va_list args)
r = control_event_status((event), (sev), format, ap); \
va_end(ap); \
} while (0)
-#endif
+#endif /* !defined(COCCI) */
/** Format and send an EVENT_STATUS_GENERAL event whose main text is obtained
* by formatting the arguments using the printf-style <b>format</b>. */
diff --git a/src/feature/dirauth/dirauth_periodic.c b/src/feature/dirauth/dirauth_periodic.c
index c02b9ac862..fc26358290 100644
--- a/src/feature/dirauth/dirauth_periodic.c
+++ b/src/feature/dirauth/dirauth_periodic.c
@@ -24,7 +24,7 @@
PERIODIC_EVENT(name, \
PERIODIC_EVENT_ROLE_##roles, \
flags)
-#endif
+#endif /* !defined(COCCI) */
#define FL(name) (PERIODIC_EVENT_FLAG_##name)
diff --git a/src/feature/dircommon/consdiff.c b/src/feature/dircommon/consdiff.c
index 3b6bffbbbd..fbfa9e0c0a 100644
--- a/src/feature/dircommon/consdiff.c
+++ b/src/feature/dircommon/consdiff.c
@@ -535,7 +535,7 @@ typedef struct router_id_iterator_t {
* Initializer for a router_id_iterator_t.
*/
#define ROUTER_ID_ITERATOR_INIT { { NULL, 0 }, { NULL, 0 } }
-#endif
+#endif /* !defined(COCCI) */
/** Given an index *<b>idxp</b> into the consensus at <b>cons</b>, advance
* the index to the next router line ("r ...") in the consensus, or to
diff --git a/src/feature/dirparse/authcert_members.h b/src/feature/dirparse/authcert_members.h
index 8f6fdf17c5..3b44c3eb02 100644
--- a/src/feature/dirparse/authcert_members.h
+++ b/src/feature/dirparse/authcert_members.h
@@ -26,4 +26,4 @@
NO_ARGS, NEED_OBJ),\
T01("dir-address", K_DIR_ADDRESS, GE(1), NO_OBJ)
-#endif
+#endif /* !defined(TOR_AUTHCERT_MEMBERS_H) */
diff --git a/src/feature/hs/hs_service.c b/src/feature/hs/hs_service.c
index 964bf68a17..da772fa6d2 100644
--- a/src/feature/hs/hs_service.c
+++ b/src/feature/hs/hs_service.c
@@ -89,7 +89,7 @@
(var = service->desc_next); \
if (var == NULL) continue;
#define FOR_EACH_DESCRIPTOR_END } STMT_END ;
-#endif
+#endif /* !defined(COCCI) */
/* Onion service directory file names. */
static const char fname_keyfile_prefix[] = "hs_ed25519";
diff --git a/src/feature/relay/relay_periodic.c b/src/feature/relay/relay_periodic.c
index 7dc4cb4530..d1b3183953 100644
--- a/src/feature/relay/relay_periodic.c
+++ b/src/feature/relay/relay_periodic.c
@@ -37,7 +37,7 @@
PERIODIC_EVENT(name, \
PERIODIC_EVENT_ROLE_##roles, \
flags)
-#endif
+#endif /* !defined(COCCI) */
#define FL(name) (PERIODIC_EVENT_FLAG_##name)
diff --git a/src/feature/relay/router.c b/src/feature/relay/router.c
index 92803f88a9..5c79010934 100644
--- a/src/feature/relay/router.c
+++ b/src/feature/relay/router.c
@@ -386,7 +386,7 @@ get_server_identity_key,(void))
return server_identitykey;
}
-#endif
+#endif /* defined(HAVE_MODULE_RELAY) */
/** Return true iff we are a server and the server identity key
* has been set. */
diff --git a/src/feature/relay/routermode.h b/src/feature/relay/routermode.h
index f6195a8698..9891a038ad 100644
--- a/src/feature/relay/routermode.h
+++ b/src/feature/relay/routermode.h
@@ -25,7 +25,7 @@ void set_server_advertised(int s);
/** Is the relay module enabled? */
#define have_module_relay() (1)
-#else
+#else /* !defined(HAVE_MODULE_RELAY) */
#define dir_server_mode(options) (((void)(options)),0)
#define server_mode(options) (((void)(options)),0)
@@ -37,6 +37,6 @@ void set_server_advertised(int s);
#define have_module_relay() (0)
-#endif
+#endif /* defined(HAVE_MODULE_RELAY) */
#endif /* !defined(TOR_ROUTERMODE_H) */
diff --git a/src/lib/buf/buffers.c b/src/lib/buf/buffers.c
index 28710ebab3..4adc08fdbf 100644
--- a/src/lib/buf/buffers.c
+++ b/src/lib/buf/buffers.c
@@ -110,7 +110,7 @@
memset(a,0,SENTINEL_LEN); \
} while (0)
#endif /* defined(DISABLE_MEMORY_SENTINELS) */
-#endif
+#endif /* !defined(COCCI) */
/** Move all bytes stored in <b>chunk</b> to the front of <b>chunk</b>->mem,
* to free up space at the end. */
diff --git a/src/lib/conf/confmacros.h b/src/lib/conf/confmacros.h
index 6449458a10..67082454a2 100644
--- a/src/lib/conf/confmacros.h
+++ b/src/lib/conf/confmacros.h
@@ -21,7 +21,7 @@
**/
#define END_OF_CONFIG_VARS \
{ .member = { .name = NULL } DUMMY_CONF_TEST_MEMBERS }
-#endif
+#endif /* !defined(COCCI) */
/**
* Declare a config_var_t as a member named <b>membername</b> of the structure
diff --git a/src/lib/conf/conftesting.h b/src/lib/conf/conftesting.h
index 8f82b5bf18..dbfd564079 100644
--- a/src/lib/conf/conftesting.h
+++ b/src/lib/conf/conftesting.h
@@ -72,8 +72,8 @@ typedef union {
#define DUMMY_CONF_TEST_MEMBERS , .var_ptr_dummy={ .INT=NULL }
#define DUMMY_TYPECHECK_INSTANCE(tp) \
static tp tp ## _dummy
-#endif
-#endif
+#endif /* defined(TOR_UNIT_TESTS) */
+#endif /* !defined(COCCI) */
#ifndef USE_CONF_TESTING
#define CONF_TEST_MEMBERS(tp, conftype, member)
@@ -82,6 +82,6 @@ typedef union {
struct tor_semicolon_eater
#define DUMMY_CONF_TEST_MEMBERS
-#endif /* defined(TOR_UNIT_TESTS) */
+#endif /* !defined(USE_CONF_TESTING) */
#endif /* !defined(TOR_LIB_CONF_CONFTESTING_H) */
diff --git a/src/lib/container/smartlist.h b/src/lib/container/smartlist.h
index 1dadf4ae6b..984cd2d293 100644
--- a/src/lib/container/smartlist.h
+++ b/src/lib/container/smartlist.h
@@ -166,6 +166,6 @@ char *smartlist_join_strings2(smartlist_t *sl, const char *join,
#define SMARTLIST_FOREACH_JOIN_END(var1, var2) \
} \
STMT_END
-#endif
+#endif /* !defined(COCCI) */
#endif /* !defined(TOR_SMARTLIST_H) */
diff --git a/src/lib/crypt_ops/crypto_openssl_mgt.c b/src/lib/crypt_ops/crypto_openssl_mgt.c
index 617c0dcd76..2fbebd87e0 100644
--- a/src/lib/crypt_ops/crypto_openssl_mgt.c
+++ b/src/lib/crypt_ops/crypto_openssl_mgt.c
@@ -126,7 +126,7 @@ crypto_openssl_get_header_version_str(void)
#error "OpenSSL has been built without thread support. Tor requires an \
OpenSSL library with thread support enabled."
#endif
-#endif
+#endif /* !defined(COCCI) */
#ifndef NEW_THREAD_API
/** Helper: OpenSSL uses this callback to manipulate mutexes. */
diff --git a/src/lib/dispatch/dispatch_naming.c b/src/lib/dispatch/dispatch_naming.c
index 79411cb3a1..e8e28cc71e 100644
--- a/src/lib/dispatch/dispatch_naming.c
+++ b/src/lib/dispatch/dispatch_naming.c
@@ -57,7 +57,7 @@ dispatch_naming_init(void)
return namemap_get_size(&type##_id_map); \
} \
EAT_SEMICOLON
-#endif
+#endif /* !defined(COCCI) */
DECLARE_ID_MAP_FNS(message);
DECLARE_ID_MAP_FNS(channel);
diff --git a/src/lib/evloop/timers.c b/src/lib/evloop/timers.c
index 23562971ca..c5bb0f5958 100644
--- a/src/lib/evloop/timers.c
+++ b/src/lib/evloop/timers.c
@@ -64,7 +64,7 @@ struct timeout_cb {
#else
/* We're not exposing any of the functions outside this file. */
#define TIMEOUT_PUBLIC static
-#endif /* defined(__GNUC__) */
+#endif /* defined(COCCI) || ... */
/* We're not using periodic events. */
#define TIMEOUT_DISABLE_INTERVALS
/* We always know the global_timeouts object, so we don't need each timeout
diff --git a/src/lib/log/log.h b/src/lib/log/log.h
index 19cb9484d7..5cf8a36cf7 100644
--- a/src/lib/log/log.h
+++ b/src/lib/log/log.h
@@ -27,8 +27,8 @@
#error "Your syslog.h thinks high numbers are more important. " \
"We aren't prepared to deal with that."
#endif
-#endif
-#else /* !(defined(HAVE_SYSLOG_H)) */
+#endif /* LOG_DEBUG < LOG_ERR */
+#else /* !defined(HAVE_SYSLOG_H) */
/* Note: Syslog's logging code refers to priorities, with 0 being the most
* important. Thus, all our comparisons needed to be reversed when we added
* syslog support.
@@ -310,7 +310,7 @@ MOCK_DECL(STATIC void, logv, (int severity, log_domain_mask_t domain,
va_list ap) CHECK_PRINTF(5,0));
MOCK_DECL(STATIC void, add_stream_log_impl,(
const log_severity_list_t *severity, const char *name, int fd));
-#endif
+#endif /* defined(LOG_PRIVATE) */
#if defined(LOG_PRIVATE) || defined(TOR_UNIT_TESTS)
/** Given a severity, yields an index into log_severity_list_t.masks to use
diff --git a/src/lib/log/util_bug.h b/src/lib/log/util_bug.h
index dcff5fd9c3..d0a311ac61 100644
--- a/src/lib/log/util_bug.h
+++ b/src/lib/log/util_bug.h
@@ -225,7 +225,7 @@
1)) \
: 0)
#endif /* defined(__GNUC__) */
-#endif
+#endif /* !defined(COCCI) */
#define IF_BUG_ONCE_VARNAME_(a) \
warning_logged_on_ ## a ## __
diff --git a/src/lib/thread/threads.h b/src/lib/thread/threads.h
index 2e63dac5d9..ad9ad98c70 100644
--- a/src/lib/thread/threads.h
+++ b/src/lib/thread/threads.h
@@ -109,7 +109,7 @@ typedef struct atomic_counter_t {
#ifndef COCCI
#define ATOMIC_LINKAGE static
#endif
-#else /* !(defined(HAVE_WORKING_STDATOMIC)) */
+#else /* !defined(HAVE_WORKING_STDATOMIC) */
typedef struct atomic_counter_t {
tor_mutex_t mutex;
size_t val;
diff --git a/src/lib/version/git_revision.c b/src/lib/version/git_revision.c
index 338ba2bc3d..6cfb3b4ec0 100644
--- a/src/lib/version/git_revision.c
+++ b/src/lib/version/git_revision.c
@@ -24,5 +24,5 @@ const char tor_bug_suffix[] = " (on Tor " VERSION
" "
#include "micro-revision.i"
#endif
-#endif
+#endif /* !defined(COCCI) */
")";
diff --git a/src/lib/wallclock/timeval.h b/src/lib/wallclock/timeval.h
index 897a864416..4710a3dfd7 100644
--- a/src/lib/wallclock/timeval.h
+++ b/src/lib/wallclock/timeval.h
@@ -83,6 +83,6 @@
((tv1)->tv_usec op (tv2)->tv_usec) : \
((tv1)->tv_sec op (tv2)->tv_sec))
#endif /* !defined(timercmp) */
-#endif
+#endif /* !defined(COCCI) */
#endif /* !defined(TOR_TIMEVAL_H) */
diff --git a/src/test/test.h b/src/test/test.h
index e9caef3fec..feaa13a3a5 100644
--- a/src/test/test.h
+++ b/src/test/test.h
@@ -65,7 +65,7 @@
tt_assert_test_fmt_type(a,b,#a" "#op" "#b,int64_t,(val1_ op val2_), \
int64_t, "%"PRId64, \
{print_ = (int64_t) value_;}, {}, TT_EXIT_TEST_FUNCTION)
-#endif
+#endif /* !defined(COCCI) */
/**
* Declare that the test is done, even though no tt___op() calls were made.
@@ -164,7 +164,7 @@ void free_pregenerated_keys(void);
NULL, \
NULL, \
}
-#endif
+#endif /* !defined(COCCI) */
#define NS(name) US_CONCAT_3_(NS_MODULE, NS_SUBMODULE, name)
#define NS_FULL(module, submodule, name) US_CONCAT_3_(module, submodule, name)
@@ -175,7 +175,7 @@ void free_pregenerated_keys(void);
extern int CALLED(mock_fn); \
static retval NS(mock_fn) args; int CALLED(mock_fn) = 0
#define NS_MOCK(name) MOCK(name, NS(name))
-#endif
+#endif /* !defined(COCCI) */
#define NS_UNMOCK(name) UNMOCK(name)
extern const struct testcase_setup_t passthrough_setup;
diff --git a/src/test/test_addr.c b/src/test/test_addr.c
index 44f3cd9ec1..04380d1ccf 100644
--- a/src/test/test_addr.c
+++ b/src/test/test_addr.c
@@ -70,7 +70,7 @@ test_addr_basic(void *arg)
TT_EXIT_TEST_FUNCTION \
); \
STMT_END
-#endif
+#endif /* !defined(COCCI) */
/** Helper: Assert that two strings both decode as IPv6 addresses with
* tor_inet_pton(), and both decode to the same address. */
@@ -138,7 +138,7 @@ test_addr_basic(void *arg)
TT_DIE(("Failed: tor_addr_compare_masked(%s,%s,%d) %s 0", \
a, b, m, #op)); \
STMT_END
-#endif
+#endif /* !defined(COCCI) */
/** Helper: assert that <b>xx</b> is parseable as a masked IPv6 address with
* ports by tor_parse_mask_addr_ports(), with family <b>f</b>, IP address
diff --git a/src/test/test_confparse.c b/src/test/test_confparse.c
index 33979813fd..206fe0fc0a 100644
--- a/src/test/test_confparse.c
+++ b/src/test/test_confparse.c
@@ -1044,7 +1044,7 @@ test_confparse_find_option_name(void *arg)
#define BADVAL_TEST(name) \
{ "badval_" #name, test_confparse_assign_badval, 0, \
&passthrough_setup, (void*)&bv_ ## name }
-#endif
+#endif /* !defined(COCCI) */
struct testcase_t confparse_tests[] = {
CONFPARSE_TEST(init, 0),
diff --git a/src/test/test_connection.c b/src/test/test_connection.c
index 3148c1af66..0d28276702 100644
--- a/src/test/test_connection.c
+++ b/src/test/test_connection.c
@@ -888,7 +888,7 @@ test_failed_orconn_tracker(void *arg)
/* where arg is an expression (constant, variable, compound expression) */
#define CONNECTION_TESTCASE_ARG(name, fork, setup, arg) \
{ #name "_" #arg, test_conn_##name, fork, &setup, (void *)arg }
-#endif
+#endif /* !defined(COCCI) */
struct testcase_t connection_tests[] = {
CONNECTION_TESTCASE(get_basic, TT_FORK, test_conn_get_basic_st),
diff --git a/src/test/test_controller.c b/src/test/test_controller.c
index c68e35ab3b..793696c474 100644
--- a/src/test/test_controller.c
+++ b/src/test/test_controller.c
@@ -130,7 +130,7 @@ test_controller_parse_cmd(void *arg)
{ &syntax, \
ARRAY_LENGTH(array), \
array }
-#endif
+#endif /* !defined(COCCI) */
static const parser_testcase_t one_to_three_tests[] = {
ERR("", "Need at least 1 argument(s)"),
diff --git a/src/test/test_crypto.c b/src/test/test_crypto.c
index ae9c51e00a..2c57e50a4c 100644
--- a/src/test/test_crypto.c
+++ b/src/test/test_crypto.c
@@ -2982,7 +2982,7 @@ test_crypto_failure_modes(void *arg)
#define ED25519_TEST(name, fl) \
ED25519_TEST_ONE(name, (fl), "donna"), \
ED25519_TEST_ONE(name, (fl), "ref10")
-#endif
+#endif /* !defined(COCCI) */
struct testcase_t crypto_tests[] = {
CRYPTO_LEGACY(formats),
diff --git a/src/test/test_crypto_slow.c b/src/test/test_crypto_slow.c
index 4f0cfbc641..9ee7b100b3 100644
--- a/src/test/test_crypto_slow.c
+++ b/src/test/test_crypto_slow.c
@@ -595,7 +595,7 @@ test_crypto_ed25519_fuzz_donna(void *arg)
#define ED25519_TEST(name, fl) \
ED25519_TEST_ONE(name, (fl), "donna"), \
ED25519_TEST_ONE(name, (fl), "ref10")
-#endif
+#endif /* !defined(COCCI) */
struct testcase_t slow_crypto_tests[] = {
CRYPTO_LEGACY(s2k_rfc2440),
diff --git a/src/test/test_dir.c b/src/test/test_dir.c
index 77cc1f97b4..02988d2471 100644
--- a/src/test/test_dir.c
+++ b/src/test/test_dir.c
@@ -7220,7 +7220,7 @@ test_dir_format_versions_list(void *arg)
/* where arg is a string constant */
#define DIR_ARG(name,flags,arg) \
{ #name "_" arg, test_dir_##name, (flags), &passthrough_setup, (void*) arg }
-#endif
+#endif /* !defined(COCCI) */
struct testcase_t dir_tests[] = {
DIR_LEGACY(nicknames),
diff --git a/src/test/test_entrynodes.c b/src/test/test_entrynodes.c
index 598babe22b..dec7874c7d 100644
--- a/src/test/test_entrynodes.c
+++ b/src/test/test_entrynodes.c
@@ -3062,7 +3062,7 @@ static const struct testcase_setup_t upgrade_circuits = {
&upgrade_circuits, (void*)(arg REASONABLY_FUTURE) }, \
{ #name "_reasonably_past", test_entry_guard_ ## name, TT_FORK, \
&upgrade_circuits, (void*)(arg REASONABLY_PAST) }
-#endif
+#endif /* !defined(COCCI) */
struct testcase_t entrynodes_tests[] = {
NO_PREFIX_TEST(node_preferred_orport),
diff --git a/src/test/test_link_handshake.c b/src/test/test_link_handshake.c
index 1371e1ef74..18c1700031 100644
--- a/src/test/test_link_handshake.c
+++ b/src/test/test_link_handshake.c
@@ -1528,7 +1528,7 @@ AUTHENTICATE_FAIL(missing_ed_auth,
#define TEST_AUTHENTICATE_ED(name) \
{ "authenticate/" #name "_ed25519" , test_link_handshake_auth_ ## name, \
TT_FORK, &setup_authenticate, (void*)3 }
-#endif
+#endif /* !defined(COCCI) */
struct testcase_t link_handshake_tests[] = {
TEST_RSA(certs_ok, TT_FORK),
diff --git a/src/test/test_options.c b/src/test/test_options.c
index aeb5de210a..6b6a52ae83 100644
--- a/src/test/test_options.c
+++ b/src/test/test_options.c
@@ -696,7 +696,7 @@ test_options_validate__logs(void *ignored)
tt_int_op(ret, OP_EQ, -1);
#else
tt_int_op(ret, OP_EQ, 0);
-#endif
+#endif /* defined(_WIN32) */
free_options_test_data(tdata);
tdata = get_options_test_data("");
diff --git a/src/test/test_util.c b/src/test/test_util.c
index 09978a13c9..3e4975fcd8 100644
--- a/src/test/test_util.c
+++ b/src/test/test_util.c
@@ -317,7 +317,7 @@ test_util_write_chunks_to_file(void *arg)
_TFE(a, b, tm_min ); \
_TFE(a, b, tm_sec ); \
TT_STMT_END
-#endif
+#endif /* !defined(COCCI) */
static void
test_util_time(void *arg)
@@ -6284,7 +6284,7 @@ test_util_map_anon_nofork(void *arg)
{ "compress_dos/" #name, test_util_decompress_dos, 0, \
&compress_setup, \
(char*)(identifier) }
-#endif
+#endif /* !defined(COCCI) */
#ifdef _WIN32
#define UTIL_TEST_NO_WIN(n, f) { #n, NULL, TT_SKIP, NULL, NULL }