From 98299da0fda612b42ab933c47f18163cfef5fa71 Mon Sep 17 00:00:00 2001 From: Vincent Ambo Date: Mon, 25 May 2020 01:19:02 +0100 Subject: refactor(3p/nix/libutil): Replace string2Int & trim functions Replaces these functions with corresponding functions from Abseil, namely absl::StripAsciiWhitespace and absl::SimpleAtoi. In the course of doing this some minor things I encountered along the way were also refactored. This also changes the signatures of the various custom readFile functions to use absl::string_view types. --- third_party/nix/src/nix-env/nix-env.cc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'third_party/nix/src/nix-env/nix-env.cc') diff --git a/third_party/nix/src/nix-env/nix-env.cc b/third_party/nix/src/nix-env/nix-env.cc index e827f31e5d..d17bde34f3 100644 --- a/third_party/nix/src/nix-env/nix-env.cc +++ b/third_party/nix/src/nix-env/nix-env.cc @@ -4,6 +4,7 @@ #include #include +#include #include #include #include @@ -1303,7 +1304,7 @@ static void opSwitchGeneration(Globals& globals, Strings opFlags, } int dstGen; - if (!string2Int(opArgs.front(), dstGen)) { + if (!absl::SimpleAtoi(opArgs.front(), &dstGen)) { throw UsageError(format("expected a generation number")); } @@ -1369,7 +1370,7 @@ static void opDeleteGenerations(Globals& globals, Strings opFlags, } std::string str_max = std::string(opArgs.front(), 1, opArgs.front().size()); int max; - if (!string2Int(str_max, max) || max == 0) { + if (!absl::SimpleAtoi(str_max, &max) || max == 0) { throw Error(format("invalid number of generations to keep ‘%1%’") % opArgs.front()); } @@ -1378,7 +1379,7 @@ static void opDeleteGenerations(Globals& globals, Strings opFlags, std::set gens; for (auto& i : opArgs) { unsigned int n; - if (!string2Int(i, n)) { + if (!absl::SimpleAtoi(i, &n)) { throw UsageError(format("invalid generation number '%1%'") % i); } gens.insert(n); -- cgit 1.4.1