Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding wchar_t* file name support for file_mapping constructor #28

Closed
wants to merge 1 commit into from

Conversation

wingfiring
Copy link

@wingfiring wingfiring commented Aug 24, 2016

Hi,

I'm a developer from Autodesk.

More and more peoples in our company want to use file mapping cross platforms, but file_mapping of boost.interprocess lacks of wide char file name supporting on Windows. I think it's valuable for us to support wide char file name.

-Jackson

@wingfiring wingfiring changed the title Add wchar_t* file name support for file_mapping constructor Adding wchar_t* file name support for file_mapping constructor Aug 24, 2016
@egorpugin
Copy link

+1 for unicode windows file names.

@zachChilders
Copy link

Is there any specific reason this PR has not been merged in? We're having some trouble with file paths that contain unicode as well and its becoming a pain to maintain multiple codepaths to handle native support on every build configuration we target. Its one of the few areas where we cannot be truly xplat right now.

@mdoppler
Copy link

mdoppler commented Jul 2, 2019

The same limitation applies to boost::interprocess::sync::file_lock.

This patch adds support for wide character file paths in this class:

boost_1_70_0-windows-unicode-filelock.zip

@sepavel
Copy link

sepavel commented Jul 3, 2019

How can I vote for this patch?

@seanlis
Copy link

seanlis commented Oct 26, 2020

+1 for unicode windows file names.

@igaztanaga
Copy link
Member

Finally wide filename support for Windows was added in commit:
140b50e

@igaztanaga igaztanaga closed this Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants