JS backend: JsDescriptorUtils.isAsPrivate remake to isSimpleProperty

This commit is contained in:
Erokhin Stanislav
2013-09-05 20:06:05 +04:00
parent 638beff551
commit 550cf581c4
3 changed files with 7 additions and 9 deletions
@@ -328,7 +328,7 @@ public final class StaticContext {
} }
String name = descriptor.getName().asString(); String name = descriptor.getName().asString();
if (!isEcma5() || JsDescriptorUtils.isAsPrivate((PropertyDescriptor) descriptor)) { if (!isEcma5() || !JsDescriptorUtils.isSimpleProperty((PropertyDescriptor) descriptor)) {
name = Namer.getKotlinBackingFieldName(name); name = Namer.getKotlinBackingFieldName(name);
} }
@@ -58,11 +58,9 @@ public final class PropertyTranslator extends AbstractTranslator {
@Nullable JetProperty declaration, @Nullable JetProperty declaration,
@NotNull List<JsPropertyInitializer> result, @NotNull List<JsPropertyInitializer> result,
@NotNull TranslationContext context) { @NotNull TranslationContext context) {
if (context.isEcma5() && !JsDescriptorUtils.isAsPrivate(descriptor)) { if (!JsDescriptorUtils.isSimpleProperty(descriptor)) {
return; new PropertyTranslator(descriptor, declaration, context).translate(result);
} }
new PropertyTranslator(descriptor, declaration, context).translate(result);
} }
private PropertyTranslator(@NotNull PropertyDescriptor descriptor, @Nullable JetProperty declaration, @NotNull TranslationContext context) { private PropertyTranslator(@NotNull PropertyDescriptor descriptor, @Nullable JetProperty declaration, @NotNull TranslationContext context) {
@@ -140,10 +140,10 @@ public final class JsDescriptorUtils {
return accessorDescriptor == null || accessorDescriptor.isDefault(); return accessorDescriptor == null || accessorDescriptor.isDefault();
} }
public static boolean isAsPrivate(@NotNull PropertyDescriptor propertyDescriptor) { public static boolean isSimpleProperty(@NotNull PropertyDescriptor propertyDescriptor) {
return isExtension(propertyDescriptor) || return !isExtension(propertyDescriptor) &&
!isDefaultAccessor(propertyDescriptor.getGetter()) || isDefaultAccessor(propertyDescriptor.getGetter()) &&
!isDefaultAccessor(propertyDescriptor.getSetter()); isDefaultAccessor(propertyDescriptor.getSetter());
} }
public static boolean isStandardDeclaration(@NotNull DeclarationDescriptor descriptor) { public static boolean isStandardDeclaration(@NotNull DeclarationDescriptor descriptor) {