From d4be23180ae1c045638e011db2e95e329c0e4313 Mon Sep 17 00:00:00 2001 From: Alexey Kudravtsev Date: Thu, 23 Jun 2011 13:10:54 +0400 Subject: [PATCH] cleanup --- .../psi/impl/source/parsing/GTTokens.java | 70 +++++++++---------- .../intellij/psi/IdentitySmartPointer.java | 2 +- .../com/intellij/psi/impl/ElementBase.java | 2 +- 3 files changed, 36 insertions(+), 38 deletions(-) diff --git a/java/java-impl/src/com/intellij/psi/impl/source/parsing/GTTokens.java b/java/java-impl/src/com/intellij/psi/impl/source/parsing/GTTokens.java index 7793e0f020b1..7b35bb508fd8 100644 --- a/java/java-impl/src/com/intellij/psi/impl/source/parsing/GTTokens.java +++ b/java/java-impl/src/com/intellij/psi/impl/source/parsing/GTTokens.java @@ -27,52 +27,50 @@ import com.intellij.util.CharTable; /** * @author dsl */ -class GTTokens implements JavaTokenType { +class GTTokens { private static final Logger LOG = Logger.getInstance("#com.intellij.psi.impl.source.parsing.GTTokens"); static IElementType getTokenType(Lexer lexer) { - if (lexer.getTokenType() != GT) return lexer.getTokenType(); + if (lexer.getTokenType() != JavaTokenType.GT) return lexer.getTokenType(); final LexerPosition originalPosition = lexer.getCurrentPosition(); final int prevTokenEnd = lexer.getTokenEnd(); lexer.advance(); if (lexer.getTokenStart() != prevTokenEnd) { lexer.restore(originalPosition); - return GT; + return JavaTokenType.GT; } final IElementType resultType; IElementType i1 = lexer.getTokenType(); - if (i1 == EQ) { - resultType = GE; + if (i1 == JavaTokenType.EQ) { + resultType = JavaTokenType.GE; } - else if (i1 == GT) { - { - final int prevTokenEnd2 = lexer.getTokenEnd(); - lexer.advance(); - if (lexer.getTokenStart() != prevTokenEnd2) { - resultType = GTGT; - } - else { - IElementType i = lexer.getTokenType(); - if (i == GT) { - final int prevTokenEnd3 = lexer.getTokenEnd(); - lexer.advance(); - if (lexer.getTokenStart() != prevTokenEnd3 || lexer.getTokenType() != EQ) { - resultType = GTGTGT; - } - else { - resultType = GTGTGTEQ; - } - } - else if (i == EQ) { - resultType = GTGTEQ; + else if (i1 == JavaTokenType.GT) { + final int prevTokenEnd2 = lexer.getTokenEnd(); + lexer.advance(); + if (lexer.getTokenStart() != prevTokenEnd2) { + resultType = JavaTokenType.GTGT; + } + else { + IElementType i = lexer.getTokenType(); + if (i == JavaTokenType.GT) { + final int prevTokenEnd3 = lexer.getTokenEnd(); + lexer.advance(); + if (lexer.getTokenStart() != prevTokenEnd3 || lexer.getTokenType() != JavaTokenType.EQ) { + resultType = JavaTokenType.GTGTGT; } else { - resultType = GTGT; + resultType = JavaTokenType.GTGTGTEQ; } } + else if (i == JavaTokenType.EQ) { + resultType = JavaTokenType.GTGTEQ; + } + else { + resultType = JavaTokenType.GTGT; + } } } else { - resultType = GT; + resultType = JavaTokenType.GT; } lexer.restore(originalPosition); return resultType; @@ -80,38 +78,38 @@ class GTTokens implements JavaTokenType { static TreeElement createTokenElementAndAdvance(IElementType tokenType, Lexer lexer, CharTable table) { final TreeElement result; - if (tokenType == GTGT || tokenType == GE) { + if (tokenType == JavaTokenType.GTGT || tokenType == JavaTokenType.GE) { result = mergeTokens(1, lexer, tokenType, table); } - else if (tokenType == GTGTEQ || tokenType == GTGTGT) { + else if (tokenType == JavaTokenType.GTGTEQ || tokenType == JavaTokenType.GTGTGT) { result = mergeTokens(2, lexer, tokenType, table); } - else if (tokenType == GTGTGTEQ) { + else if (tokenType == JavaTokenType.GTGTGTEQ) { result = mergeTokens(3, lexer, tokenType, table); } else { LOG.assertTrue(tokenType == lexer.getTokenType()); - result = ParseUtil.createTokenElement(lexer, table); + result = ParseUtilBase.createTokenElement(lexer, table); } lexer.advance(); return result; } static void advance(IElementType tokenType, Lexer lexer) { - if(lexer.getTokenType() != GT){ + if(lexer.getTokenType() != JavaTokenType.GT){ lexer.advance(); } else { - if (tokenType == GTGTGTEQ) { + if (tokenType == JavaTokenType.GTGTGTEQ) { lexer.advance(); lexer.advance(); lexer.advance(); } - else if (tokenType == GTGTEQ || tokenType == GTGTGT) { + else if (tokenType == JavaTokenType.GTGTEQ || tokenType == JavaTokenType.GTGTGT) { lexer.advance(); lexer.advance(); } - else if (tokenType == GTGT || tokenType == GE) { + else if (tokenType == JavaTokenType.GTGT || tokenType == JavaTokenType.GE) { lexer.advance(); } lexer.advance(); diff --git a/platform/lang-api/src/com/intellij/psi/IdentitySmartPointer.java b/platform/lang-api/src/com/intellij/psi/IdentitySmartPointer.java index bed08c02efde..2cbbcb1d94dd 100644 --- a/platform/lang-api/src/com/intellij/psi/IdentitySmartPointer.java +++ b/platform/lang-api/src/com/intellij/psi/IdentitySmartPointer.java @@ -33,7 +33,7 @@ public class IdentitySmartPointer implements SmartPsiEleme myFile = file; } - public IdentitySmartPointer(final T element) { + public IdentitySmartPointer(@NotNull T element) { this(element, element.getContainingFile()); } diff --git a/platform/lang-api/src/com/intellij/psi/impl/ElementBase.java b/platform/lang-api/src/com/intellij/psi/impl/ElementBase.java index ba2f991512bf..5ea69fde9f2c 100644 --- a/platform/lang-api/src/com/intellij/psi/impl/ElementBase.java +++ b/platform/lang-api/src/com/intellij/psi/impl/ElementBase.java @@ -175,7 +175,7 @@ public abstract class ElementBase extends UserDataHolderBase implements Iconable ):baseIcon; } - public static class ElementIconRequest extends ComparableObject.Impl { + private static class ElementIconRequest extends ComparableObject.Impl { public ElementIconRequest(PsiElement element, int flags) { super(new Object[] {createPointer(element), flags}); }