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

Add STL compatibility checks #104

Open
adamfowleruk opened this issue Sep 3, 2021 · 0 comments
Open

Add STL compatibility checks #104

adamfowleruk opened this issue Sep 3, 2021 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@adamfowleruk
Copy link
Contributor

Please start with a use case description for a USER of this enhancement

[Who] As an application developer

[What] I need to use Herald classes with or instead of C++ STL classes in a compatible way, interchangeably

[Value] In order to achieve the principle of least astonishment to the use of the Herald API (thus saving me time during development)

Describe the potential solution you'd like

A clear and concise description of what you want to happen within Herald.

  • Ensure all datatypes are Move+CopyConstructable and Move+CopyAssignable
  • Ensure containers are interchangeable with their STL equivalents they are replacing

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

Considered leaving the API's compatibility as best efforts or as needed, but this is likely to introduce inconsistency to the API, to cause bugs, and to make the Herald C++ API hard to consume.

Additional context

Add any other context or screenshots about the feature request here.

Relative priority

Please give an indication of a relative priority for this enhancement.

Ensures correct function of internal core classes, and so tests for this are considered high priority.

@adamfowleruk adamfowleruk added enhancement New feature or request good first issue Good for newcomers labels Sep 3, 2021
@adamfowleruk adamfowleruk added this to the v2.1 milestone Sep 3, 2021
@adamfowleruk adamfowleruk modified the milestones: v2.1, v2.2 Jan 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
Development

No branches or pull requests

1 participant