CLEANUP: trace: remove the QUIC-specific ifdefs
There are ifdefs at several places to only define TRC_ARGS_QCON when
QUIC is defined, but nothing prevents this code from building without.
Let's just remove those ifdefs, the single "if" they avoid is not worth
the extra maintenance burden.
diff --git a/include/haproxy/trace-t.h b/include/haproxy/trace-t.h
index dcdaec3..e08714c 100644
--- a/include/haproxy/trace-t.h
+++ b/include/haproxy/trace-t.h
@@ -43,54 +43,42 @@
#define TRC_ARG_SESS (1 << 1)
#define TRC_ARG_STRM (1 << 2)
#define TRC_ARG_CHK (1 << 3)
-#ifdef USE_QUIC
#define TRC_ARG_QCON (1 << 4)
-#endif
#define TRC_ARG1_PRIV (TRC_ARG_PRIV << 0)
#define TRC_ARG1_CONN (TRC_ARG_CONN << 0)
#define TRC_ARG1_SESS (TRC_ARG_SESS << 0)
#define TRC_ARG1_STRM (TRC_ARG_STRM << 0)
#define TRC_ARG1_CHK (TRC_ARG_CHK << 0)
-#ifdef USE_QUIC
#define TRC_ARG1_QCON (TRC_ARG_QCON << 0)
-#endif
#define TRC_ARG2_PRIV (TRC_ARG_PRIV << 8)
#define TRC_ARG2_CONN (TRC_ARG_CONN << 8)
#define TRC_ARG2_SESS (TRC_ARG_SESS << 8)
#define TRC_ARG2_STRM (TRC_ARG_STRM << 8)
#define TRC_ARG2_CHK (TRC_ARG_CHK << 8)
-#ifdef USE_QUIC
#define TRC_ARG2_QCON (TRC_ARG_QCON << 8)
-#endif
#define TRC_ARG3_PRIV (TRC_ARG_PRIV << 16)
#define TRC_ARG3_CONN (TRC_ARG_CONN << 16)
#define TRC_ARG3_SESS (TRC_ARG_SESS << 16)
#define TRC_ARG3_STRM (TRC_ARG_STRM << 16)
#define TRC_ARG3_CHK (TRC_ARG_CHK << 16)
-#ifdef USE_QUIC
#define TRC_ARG3_QCON (TRC_ARG_QCON << 16)
-#endif
#define TRC_ARG4_PRIV (TRC_ARG_PRIV << 24)
#define TRC_ARG4_CONN (TRC_ARG_CONN << 24)
#define TRC_ARG4_SESS (TRC_ARG_SESS << 24)
#define TRC_ARG4_STRM (TRC_ARG_STRM << 24)
#define TRC_ARG4_CHK (TRC_ARG_CHK << 24)
-#ifdef USE_QUIC
#define TRC_ARG4_QCON (TRC_ARG_QCON << 24)
-#endif
/* usable to detect the presence of any arg of the desired type */
#define TRC_ARGS_CONN (TRC_ARG_CONN * 0x01010101U)
#define TRC_ARGS_SESS (TRC_ARG_SESS * 0x01010101U)
#define TRC_ARGS_STRM (TRC_ARG_STRM * 0x01010101U)
#define TRC_ARGS_CHK (TRC_ARG_CHK * 0x01010101U)
-#ifdef USE_QUIC
#define TRC_ARGS_QCON (TRC_ARG_QCON * 0x01010101U)
-#endif
enum trace_state {
@@ -122,9 +110,7 @@
TRACE_LOCKON_SESSION, // lock on the session that started the trace
TRACE_LOCKON_STREAM, // lock on the stream that started the trace
TRACE_LOCKON_CHECK, // lock on the check that started the trace
-#ifdef USE_QUIC
TRACE_LOCKON_QCON, // lock on the QUIC connection that started the trace
-#endif
TRACE_LOCKON_ARG1, // lock on arg1, totally source-dependent
TRACE_LOCKON_ARG2, // lock on arg2, totally source-dependent
TRACE_LOCKON_ARG3, // lock on arg3, totally source-dependent
diff --git a/src/trace.c b/src/trace.c
index 9c0b3f2..d085685 100644
--- a/src/trace.c
+++ b/src/trace.c
@@ -87,9 +87,7 @@
const struct stream *strm = NULL;
const struct connection *conn = NULL;
const struct check *check = NULL;
-#ifdef USE_QUIC
const struct quic_conn *qc = NULL;
-#endif
const void *lockon_ptr = NULL;
struct ist ist_func = ist(func);
char tnum[4];
@@ -116,10 +114,8 @@
if (src->arg_def & TRC_ARGS_CHK)
check = trace_pick_arg(src->arg_def & TRC_ARGS_CHK, a1, a2, a3, a4);
-#ifdef USE_QUIC
if (src->arg_def & TRC_ARGS_QCON)
qc = trace_pick_arg(src->arg_def & TRC_ARGS_QCON, a1, a2, a3, a4);
-#endif
if (!sess && strm)
sess = strm->sess;
@@ -180,9 +176,7 @@
case TRACE_LOCKON_STREAM: lockon_ptr = strm; break;
case TRACE_LOCKON_CHECK: lockon_ptr = check; break;
case TRACE_LOCKON_THREAD: lockon_ptr = ti; break;
-#ifdef USE_QUIC
case TRACE_LOCKON_QCON: lockon_ptr = qc; break;
-#endif
case TRACE_LOCKON_ARG1: lockon_ptr = a1; break;
case TRACE_LOCKON_ARG2: lockon_ptr = a2; break;
case TRACE_LOCKON_ARG3: lockon_ptr = a3; break;