-
Notifications
You must be signed in to change notification settings - Fork 85
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
Add pilfer to Core #117
Comments
I don't understand what's this about. Closing as spam. @vinniefalco Please be clear about the problem description in future issues and avoid jargonisms. |
It's a request to add That'd actually be a legitimate addition to Core, for a change - the component is already present in another library and is needed as common infrastructure in others. |
You may want to reconsider the name. This facility is a optimization on move semantics, and is thus special-purpose and less widely used. In such cases we should consider a longer, more expressive name, such as |
I like |
Add pilfer to Core so that it can be used by more than just Boost.JSON
The text was updated successfully, but these errors were encountered: