Compiler warnings on `quic/streams.cc`

[targos]

[4376/4521] CXX obj/src/quic/libnode.streams.o
../../src/quic/streams.cc:842:13: warning: comparison of integers of different signs: 'stream_id' (aka 'long long') and 'const size_t' (aka 'const unsigned long') [-Wsign-compare]
  842 |   DCHECK(id < kMaxStreamId);
      |          ~~ ^ ~~~~~~~~~~~~
../../src/util.h:195:30: note: expanded from macro 'DCHECK'
  195 |   #define DCHECK(expr) CHECK(expr)
      |                              ^~~~
../../src/util.h:179:11: note: expanded from macro 'CHECK'
  179 |     if (!(expr)) [[unlikely]] {                                                \
      |           ^~~~
../../src/quic/streams.cc:899:13: warning: comparison of integers of different signs: 'stream_id' (aka 'long long') and 'const size_t' (aka 'const unsigned long') [-Wsign-compare]
  899 |   DCHECK(id < kMaxStreamId);
      |          ~~ ^ ~~~~~~~~~~~~
../../src/util.h:195:30: note: expanded from macro 'DCHECK'
  195 |   #define DCHECK(expr) CHECK(expr)
      |                              ^~~~
../../src/util.h:179:11: note: expanded from macro 'CHECK'
  179 |     if (!(expr)) [[unlikely]] {                                                \
      |           ^~~~
2 warnings generated.

@nodejs/quic

[xiaocainiao633]

I think the cause of the problem is that sizeof_t is an unsigned number, while id is of type long long. There will be problems when comparing an unsigned number with a signed number. It may be effective to eliminate the type mismatch problem through type conversion.