Mercurial > minori
comparison dep/animia/src/strategist.cc @ 150:ffa535b6d630
*: avoid usage of std::[pair,tuple]
https://arne-mertz.de/2017/03/smelly-pair-tuple/
it's better to use real structures and such where variables are easily known...
also apparently using [] on structs is actually valid? I had no idea.
| author | Paper <mrpapersonic@gmail.com> |
|---|---|
| date | Tue, 14 Nov 2023 16:27:33 -0500 |
| parents | aa4df5a84338 |
| children | 54744a48a7d7 |
comparison
equal
deleted
inserted
replaced
| 149:e41505d24733 | 150:ffa535b6d630 |
|---|---|
| 48 | 48 |
| 49 bool Strategist::ApplyOpenFilesStrategy() { | 49 bool Strategist::ApplyOpenFilesStrategy() { |
| 50 bool success = false; | 50 bool success = false; |
| 51 | 51 |
| 52 const std::set<pid_t> pids{result_.process.pid}; | 52 const std::set<pid_t> pids{result_.process.pid}; |
| 53 std::vector<std::pair<pid_t, std::string>> files; | 53 std::vector<OpenFile> files; |
| 54 | 54 |
| 55 fd.EnumerateOpenFiles(pids, files); | 55 fd.EnumerateOpenFiles(pids, files); |
| 56 | 56 |
| 57 for (const auto& [pid, file] : files) { | 57 for (const auto& file : files) { |
| 58 success |= AddMedia({MediaInfoType::File, file}); | 58 success |= AddMedia({MediaInfoType::File, file.path}); |
| 59 } | 59 } |
| 60 | 60 |
| 61 return success; | 61 return success; |
| 62 } | 62 } |
| 63 | 63 |
