ludocode / mpack

MPack - A C encoder/decoder for the MessagePack serialization format / msgpack.org[C]
MIT License
533 stars 82 forks source link

C89/99 #67

Closed dirkson closed 5 years ago

dirkson commented 5 years ago

Hey there!

Currently your code fails to compile when c_std=gnu89. However, the only C99 feature that you appear to use is the ability to declare variables inside for loops. Would you be interested in a PR that moves the variable declarations out of the for() loops, and thus makes your code compaticle with gnu89?

Cheers!

Edit: It was only a little work, so I just went ahead and submitted the PR: https://github.com/ludocode/mpack/pull/68