commit c7614e5d1e8f9139cccf6ab9b78dda16b98f7656
author Adam Lesinski <adamlesinski@google.com> Thu Mar 16 16:54:23 2017 -0700
committer Adam Lesinski <adamlesinski@google.com> Thu Mar 16 16:54:23 2017 -0700
tree 747a3c57f85d7e7d6fcacaa37463fd4132458457
parent 2bd530e1562bcbf99b6d1b2f447b4b835d3bc43e

AAPT: Version <adaptive-icon> to v26

Bug: 35908647
Change-Id: I307fb2f666cb8519adb9a7f676b87706dc6c683f
Test: manual

tools/aapt/Command.cpp

diff --git a/tools/aapt/Command.cpp b/tools/aapt/Command.cpp
index 15648bd..fdcc7aa 100644
--- a/tools/aapt/Command.cpp
+++ b/tools/aapt/Command.cpp
@@ -2501,14 +2501,21 @@
         const size_t numConfigs = gp->getFiles().size();
         for (size_t j = 0; j < numConfigs; j++) {
             status_t err = NO_ERROR;
-            if (ignoreConfig) {
-                err = builder->getBaseSplit()->addEntry(gp->getPath(), gp->getFiles().valueAt(j));
-            } else {
-                err = builder->addEntry(gp->getPath(), gp->getFiles().valueAt(j));
+            const sp<AaptFile>& file = gp->getFiles().valueAt(j);
+            if (!file->hasData()) {
+              // Empty files do not get written.
+              continue;
             }
+
+            if (ignoreConfig) {
+                err = builder->getBaseSplit()->addEntry(gp->getPath(), file);
+            } else {
+                err = builder->addEntry(gp->getPath(), file);
+            }
+
             if (err != NO_ERROR) {
                 fprintf(stderr, "Failed to add %s (%s) to builder.\n",
-                        gp->getPath().string(), gp->getFiles()[j]->getPrintableSource().string());
+                        gp->getPath().string(), file->getPrintableSource().string());
                 return err;
             }
         }