Windows failed to run - Githubissues

kyr0 / libsharedmemory

Cross-platform shared memory stream/buffer, header-only library for low-level IPC in C/C++.

MIT License

30 stars 7 forks source link

Windows failed to run #1

Open leielyq opened 2 years ago

leielyq commented 2 years ago

It is easy to get started by looking at the description api, do you still maintain it?

kyr0 commented 2 years ago

Hey @leielyq :)

I'd be happy to fix the Windows build. I'm just lacking a Windows machine atm. Could you provide me with some information about the errors and your build environment please?

leielyq commented 2 years ago

@kyr0

https://github.com/kyr0/libsharedmemory/blob/master/include/libsharedmemory/libsharedmemory.hpp line:94 “size_ ” Undefined

funatsufumiya commented 1 month ago

I researched reasons.

size_ undefined
- => should be replaced to _size

unistd.h not found

=> should be replaced to io.h on WIN32

#if defined(_WIN32)
#include <io.h>
#else
#include <unistd.h>
#endif

cannot link because Memory definition is separated into header and body
- => this can be fixed separate body into .cpp, but this is not ideal as header-only library.

Mainly because of 3rd reason, I cannot prepare well defined header-only PR, but I can create PR with .cpp. (You may achieve header-only as redifining whole Memory class in each OS specific macros.)

funatsufumiya commented 1 month ago

You can refer below, as a real working PR on Win/Mac/Linux, but I totally rewritten as Godot specific code, so some fixes are needed as for the original code. https://github.com/funatsufumiya/godot_sharedmemory/tree/main/libsharedmemory

funatsufumiya commented 1 month ago

I also ported to OpenFrameworks. Tested working well on Win/Mac/Linux. https://github.com/funatsufumiya/ofxSharedMemory