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

Commit 7d70bc53 authored by Yurii Zubrytskyi's avatar Yurii Zubrytskyi
Browse files

Address Alex's comments in the previous CL 

Bug: 282215580
Test: builds
Change-Id: I69456bd7b0b947bb4f26922f31e0988a9862de10
parent 05ea79c4

libs/androidfw/AssetManager2.cpp

+3 −3
Original line number Diff line number Diff line
@@ -260,7 +260,7 @@ void AssetManager2::DumpToLog() const { 


  auto op = StartOperation();

  std::string list;

  for (size_t i = 0; i < apk_assets_.size(); ++i) {

  for (size_t i = 0, s = apk_assets_.size(); i < s; ++i) {

    const auto& assets = GetApkAssets(i);

    base::StringAppendF(&list, "%s,", assets ? assets->GetDebugName().c_str() : "nullptr");

  }
@@ -353,7 +353,7 @@ bool AssetManager2::GetOverlayablesToString(android::StringPiece package_name, 
                                            std::string* out) const {

  auto op = StartOperation();

  uint8_t package_id = 0U;

  for (size_t i = 0; i != apk_assets_.size(); ++i) {

  for (size_t i = 0, s = apk_assets_.size(); i != s; ++i) {

    const auto& assets = GetApkAssets(i);

    if (!assets) {

      continue;
@@ -412,7 +412,7 @@ bool AssetManager2::GetOverlayablesToString(android::StringPiece package_name, 


bool AssetManager2::ContainsAllocatedTable() const {

  auto op = StartOperation();

  for (size_t i = 0; i != apk_assets_.size(); ++i) {

  for (size_t i = 0, s = apk_assets_.size(); i != s; ++i) {

    const auto& assets = GetApkAssets(i);

    if (assets && assets->IsTableAllocated()) {

      return true;