erlio / vmq_commons

[OBSOLETE] Common Functionality for VerneMQ
https://vernemq.com
Apache License 2.0
8 stars 10 forks source link

Add variable max packet size parser option #17

Closed dergraf closed 8 years ago

dergraf commented 8 years ago

@larshesel please review

larshesel commented 8 years ago

LGTM - did you micro-benchmark it? I wonder if it's even faster as all the cases have been turned into function (and guard) tests - my gut feeling would be that that is actually faster (if measurable)

dergraf commented 8 years ago

Haven't benched it, but looks cleaner to me.