GrumpyOldTroll / draft-jholland-quic-multicast

Work in progress to propose a multicast extension to quic.
Other
7 stars 6 forks source link

MC_STATE: consider using MACRO_CASE #83

Closed LPardue closed 2 years ago

LPardue commented 2 years ago

Style comment. The state and reason fields are variable-length integers, which makes them quite like RFC 9000 Error Codes. For consistency, my suggestion is to use MACRO_CASE for the values defined in this document. E.g instead of Property Violation use PROPERTY_VIOLATION. This can make it easier to slot into existing enum-based handling or logging systems.