Add some comments about StringPool sort order.

Change-Id: I57f24d46328a6bfef883819eaf95c03114d573bb

diff --git a/StringPool.cpp b/StringPool.cpp
index 46a4c3262b66471d359560a22223a4bf871e2b93..963ae59cdb5c6c1b08eb04fa1c6f8cd310fe6f7d 100644 (file)
--- a/StringPool.cpp
+++ b/StringPool.cpp
@@ -71,14 +71,16 @@ String8 StringPool::entry::makeConfigsString() const {
 }
 
 int StringPool::entry::compare(const entry& o) const {
-    // Strings with styles go first, to reduce the size of the
-    // styles array.
+    // Strings with styles go first, to reduce the size of the styles array.
+    // We don't care about the relative order of these strings.
     if (hasStyles) {
         return o.hasStyles ? 0 : -1;
     }
     if (o.hasStyles) {
         return 1;
     }
+
+    // Sort unstyled strings by type, then by logical configuration.
     int comp = configTypeName.compare(o.configTypeName);
     if (comp != 0) {
         return comp;