Skip to content

Comments

Replace std::ptr_fun for c++17#36

Open
kraj wants to merge 1 commit intoGregwar:masterfrom
kraj:c++17
Open

Replace std::ptr_fun for c++17#36
kraj wants to merge 1 commit intoGregwar:masterfrom
kraj:c++17

Conversation

@kraj
Copy link

@kraj kraj commented Jan 17, 2023

std::ptr_fun was deprecated in C++11, and removed completely in C++17. Similarly, std::not1 is deprecated since C++17.

Modern compilers like clang >= 16 have started to notice it

src/FatUtils.h:41:46: error: use of undeclared identifier 'ptr_fun'
| s.erase(find_if(s.rbegin(), s.rend(), not1(ptr_fun<int, int>(isspace))).base(), s.end());

Therefore replace ptr_fun with lambda

Also use 'unsigned char' parameter to std::isspace, for reason see [1]

[1] https://en.cppreference.com/w/cpp/string/byte/isspace#Notes

Signed-off-by: Khem Raj raj.khem@gmail.com

std::ptr_fun was deprecated in C++11, and removed completely in C++17.
Similarly, std::not1 is deprecated since C++17.

Modern compilers like clang >= 16 have started to notice it

src/FatUtils.h:41:46: error: use of undeclared identifier 'ptr_fun'
|   s.erase(find_if(s.rbegin(), s.rend(), not1(ptr_fun<int, int>(isspace))).base(), s.end());

Therefore replace ptr_fun with lambda

Also use 'unsigned char' parameter to std::isspace, for reason see [1]

[1] https://en.cppreference.com/w/cpp/string/byte/isspace#Notes

Signed-off-by: Khem Raj <raj.khem@gmail.com>
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.

1 participant