Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ffe39ea5 authored by Tom Cherry's avatar Tom Cherry Committed by Gerrit Code Review
Browse files

Merge "Cleanup some trivial TODOs."

parents ac0cee53 85f2bc93

base/properties.cpp

+0 −1
Original line number Diff line number Diff line
@@ -133,7 +133,6 @@ static void DurationToTimeSpec(timespec& ts, const std::chrono::milliseconds d) 
  ts.tv_nsec = ns.count();

}



// TODO: boot_clock?

using AbsTime = std::chrono::time_point<std::chrono::steady_clock>;



static void UpdateTimeSpec(timespec& ts, std::chrono::milliseconds relative_timeout,

init/parser.cpp

+1 −1
Original line number Diff line number Diff line
@@ -41,7 +41,7 @@ void Parser::AddSingleLineParser(const std::string& prefix, LineCallback callbac 
}



void Parser::ParseData(const std::string& filename, std::string* data) {

    data->push_back('\n');  // TODO: fix tokenizer

    data->push_back('\n');

    data->push_back('\0');



    parse_state state;

init/tokenizer_test.cpp

+1 −1
Original line number Diff line number Diff line
@@ -28,7 +28,7 @@ namespace { 


void RunTest(const std::string& data, const std::vector<std::vector<std::string>>& expected_tokens) {

    auto data_copy = std::string{data};

    data_copy.push_back('\n');  // TODO: fix tokenizer

    data_copy.push_back('\n');

    data_copy.push_back('\0');



    parse_state state;

init/ueventd.cpp

+0 −1
Original line number Diff line number Diff line
@@ -285,7 +285,6 @@ int ueventd_main(int argc, char** argv) { 


    // Keep the current product name base configuration so we remain backwards compatible and

    // allow it to override everything.

    // TODO: cleanup platform ueventd.rc to remove vendor specific device node entries (b/34968103)

    auto hardware = android::base::GetProperty("ro.hardware", "");



    auto ueventd_configuration = ParseConfig({"/system/etc/ueventd.rc", "/vendor/ueventd.rc",