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

wording #2

Open
matu3ba opened this issue Jul 4, 2023 · 1 comment
Open

wording #2

matu3ba opened this issue Jul 4, 2023 · 1 comment

Comments

@matu3ba
Copy link

matu3ba commented Jul 4, 2023

The C++ STL is one of the most complete and reusable algorithm libraries available.
This single header file brings 80% of that functionality to C99 in a non-intrusive way.
There are no new data structures. Just include the library and call functions on C arrays.

"algorithm libraries" is unspecified and https://en.cppreference.com/w/cpp/algorithm says "The algorithms library defines functions for a variety of purposes (e.g. searching, sorting, counting, manipulating) that operate on ranges of elements. Note that a range is defined as [first, last) where last refers to the element past the last element to inspect or modify."
You might want to clarify this.

"A range of elements" here does not specify if or if not the underlying bits are continuous memory, so you might want to clarify that as well.

If you really want to stick to C++ algorithms, you could mention https://en.cppreference.com/w/c/algorithm and/or constrained algorithms.

I'd rather explain what you code does.

@clibraries
Copy link
Owner

clibraries commented Jul 4, 2023

"algorithm libraries" in that paragraph refers to a library of useful computer science algorithms, not any narrow C++ definition.

Ranges are a tool to implement algorithm, not the definition of what the library does. For example, swap does not operate on ranges.

you could mention https://en.cppreference.com/w/c/algorithm

I'll consider adding a link. It would be helpful as documentation gets filled in.

Thanks for feedback.

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

No branches or pull requests

2 participants