The only downside is that it will prevent the use of Poppler on some Unix/Linux distros due to older GCCs being present…
Leonard From: poppler <[email protected]> on behalf of Albert Astals Cid <[email protected]> Date: Sunday, September 26, 2021 at 1:39 PM To: [email protected] <[email protected]> Subject: [poppler] Increasing the C++ standard to C++17 https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.freedesktop.org%2Fpoppler%2Fpoppler%2F-%2Fmerge_requests%2F941&data=04%7C01%7Clrosenth%40adobe.com%7Cc33a75763a164819582908d9811496d9%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637682747700153065%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=6nEff8jwrD2KwOnWhi6xsbI1tSb9YXsl%2F%2Fcu4YQciUE%3D&reserved=0 Anyone against it? It would allow us to use std::optional which would allow us to change things like std::unique_ptr<GooString> readMetadata(); to std::optional<std::string> readMetadata(); where you can differentiate between "no metadata" and "empty metadata" without having to use a pointer. Seems better API to me. All the things in the gitlab CI built fine and I don't think the mac/windows builders would be a problem to get the increased C++ standard. Cheers, Albert
