Does not build on platforms where size_t is not 64-bit

lpereira / lwan

Experimental, scalable, high performance HTTP server

https://lwan.ws

GNU General Public License v2.0

5.92k stars 549 forks source link

Does not build on platforms where size_t is not 64-bit #293

Closed angelsl closed 3 years ago

angelsl commented 3 years ago

https://github.com/lpereira/lwan/commit/62a25b122c4af32a3d60287a0a65b3c25f55a24f#diff-94ee1f5fd74ecb92be0520d80665a5a8R124

lpereira commented 3 years ago

It's been a while since I built Lwan on a 32-bit platform; it used to work fine. Which platform are you on (processor architecture, OS, C library, etc)?

angelsl commented 3 years ago

I was trying to build on Debian Stretch armel. I linked the offending commit and line, actually. Not sure why size_t needs to be used there..

lpereira commented 3 years ago

That's because a 64-bit integer is passed to a function that expects a size_t. I'll look into this over the weekend; thanks for reporting. (Will probably set up a 32-bit buildbot instance too.)

lpereira commented 3 years ago

This should be fixed. Please reopen if you still see the issue.

angelsl commented 3 years ago

Thanks!