Minor, simplify serialization of enum entries

Enum entries are never reordered during sort (see MemberComparator)
This commit is contained in:
Alexander Udalov
2014-11-26 15:36:07 +03:00
parent 122b14dd7d
commit fd2ddff750
@@ -120,10 +120,13 @@ public class DescriptorSerializer {
}
}
Collection<DeclarationDescriptor> nestedClasses = classDescriptor.getUnsubstitutedInnerClassesScope().getAllDescriptors();
for (DeclarationDescriptor descriptor : sort(nestedClasses)) {
if (!isEnumEntry(descriptor)) {
builder.addNestedClassName(nameTable.getSimpleNameIndex(descriptor.getName()));
for (DeclarationDescriptor descriptor : sort(classDescriptor.getUnsubstitutedInnerClassesScope().getAllDescriptors())) {
int name = nameTable.getSimpleNameIndex(descriptor.getName());
if (isEnumEntry(descriptor)) {
builder.addEnumEntry(name);
}
else {
builder.addNestedClassName(name);
}
}
@@ -132,15 +135,6 @@ public class DescriptorSerializer {
builder.setClassObject(classObjectProto(classObject));
}
if (classDescriptor.getKind() == ClassKind.ENUM_CLASS) {
// Not calling sort() here, because the order of enum entries matters
for (DeclarationDescriptor descriptor : nestedClasses) {
if (isEnumEntry(descriptor)) {
builder.addEnumEntry(nameTable.getSimpleNameIndex(descriptor.getName()));
}
}
}
return builder;
}