Comment on Add heartbeat extension bounds checkparentComments−kzrdude12yyou'd have a define with an understandable name instead of just the magic constant 20.−mikeash12yYou could even make a constant for 1, 3, and 16 separately, then still add them together in the code. It may seem excessive for such a small case, but in my experience it really helps readability to have kWhateverFieldSize instead of 1.
Comments
you'd have a define with an understandable name instead of just the magic constant 20.
You could even make a constant for 1, 3, and 16 separately, then still add them together in the code. It may seem excessive for such a small case, but in my experience it really helps readability to have kWhateverFieldSize instead of 1.