Commit Graph

23187 Commits

Author SHA1 Message Date
Alexander Udalov 4c7bcc8e1a Minor, specify version for maven-jar-plugin 2015-06-09 15:01:18 +03:00
Mikhail Glukhikh a792b29a78 Exception fix: more accurate type annotation parsing #EA-69124 Fixed 2015-06-09 14:36:12 +03:00
Mikhail Glukhikh 804bf3ad22 Refactoring of AnnotationResolver.resolveAnnotation(s)Arguments 2015-06-09 14:36:08 +03:00
Pavel V. Talanov a525c2f654 Fix folding after changes to parser 2015-06-08 20:51:10 +03:00
Pavel V. Talanov 07d35d305e Create import list psi element even if no directives are present
Fix a problem where deleting last import directive via "optimize imports" could lead to incorrect psi structure resulting in CCE, see EA-64291
New imports are inserted at the head (after package directive if present) of the file (before any comments) if no imports were present
Add test for inserting import to file where a first statement is a comment
Drop code dealing with non-existing import list in ImportInsertHelperImpl
AbstractQuickFixTest: check for unexpected actions before changes to the file are made as it relies on the first line having specific format
2015-06-08 20:51:04 +03:00
Pavel V. Talanov 4c4f730c53 Minor: Remove dead code in ScopeProvider 2015-06-08 20:25:55 +03:00
Pavel V. Talanov d3315f5ebb Refactor: Do not pass rootScope to QualifiedExpressionResolver#processImportReference 2015-06-08 20:25:54 +03:00
Pavel V. Talanov 39e78cad2a Always pass moduleDescriptor to QualifiedExpressionResolver#processImportReference
Passing package view descriptor has the same effect
2015-06-08 20:25:54 +03:00
Pavel V. Talanov 7585cbfe22 Remove check in visibilities that allowed access to privates of outer packages
It doesn't work for a while now in imports and is planned to be dropped with upcoming changes to private
2015-06-08 20:25:52 +03:00
Pavel V. Talanov be66ee7015 Refactor QualifiedExpressionResolver: pass descriptor to check for visibility instead of scope 2015-06-08 20:25:51 +03:00
Pavel V. Talanov 5e0cc18be2 Minor: prettify code in ImportDirectiveProcessor 2015-06-08 20:25:50 +03:00
Pavel V. Talanov 806098f8a0 Refactor: Extract ImportDirectiveProcessor from QualifiedExpressionResolver
J2K code
2015-06-08 20:25:50 +03:00
Alexey Sedunov 8b9ddea729 Introducer Parameter: Fix PsiInvalidElementAccessException 2015-06-08 20:18:16 +03:00
Alexey Sedunov 1d59c7e0e4 Change Signature: Substitute new function name into generated this-expressions
#KT-7920 Fixed
2015-06-08 20:18:15 +03:00
Dmitry Jemerov aa8adbeddc remove unnecessary call to super.get() in CliBindingTrace 2015-06-08 18:57:36 +02:00
Pavel V. Talanov 54b1859aa7 KotlinQuickDocumentationProvider: fix quick doc for elements from type hierarchy
Fix quick doc for light classes

 #KT-7964 Fixed
2015-06-08 18:46:44 +03:00
Dmitry Jemerov 220403b6f6 avoid unnecessary creation of empty containers 2015-06-08 15:10:40 +02:00
Dmitry Jemerov 109c09cf7c get rid of WritableScopeWithImports 2015-06-08 15:10:39 +02:00
Michael Bogdanov baa44e3d01 Fix default parameter remapping on inlining: support store instruction
#KT-7963 Fixed
2015-06-08 16:07:39 +03:00
Michael Bogdanov 7a1625fc9e Support extension lambda inlining
#KT-7962 Fixed
2015-06-08 16:07:38 +03:00
Dmitry Jemerov 18753a16d7 a bunch of fixes to the grammar 2015-06-08 14:46:28 +02:00
Valentin Kipyatkov 5f0171aa04 Added two more inspections based on existing intentions 2015-06-08 15:37:59 +03:00
Valentin Kipyatkov 5b9680fe11 Made all inspections making code more verbose low priority 2015-06-08 15:37:59 +03:00
Valentin Kipyatkov 63b30cf7f6 KT-7955 "Add replaceWith to specify replacement pattern" intention/inpesction has too large a range
#KT-7955 Fixed
2015-06-08 15:37:58 +03:00
Valentin Kipyatkov e1efb95259 Added test 2015-06-08 15:37:40 +03:00
Valentin Kipyatkov 7dddd6c718 Code simplification 2015-06-08 15:33:58 +03:00
Valentin Kipyatkov 8243d27fb2 Smart completion: argument with "*" is not necessary the only one for vararg parameter 2015-06-08 15:33:58 +03:00
Valentin Kipyatkov 0d992ee38f Moved tests 2015-06-08 15:33:58 +03:00
Valentin Kipyatkov 54e9a46f12 Smart completion: fixed in argument with "*" 2015-06-08 15:33:58 +03:00
Valentin Kipyatkov 2197d554ca Smart completion: a bit more correct logic for last vararg parameter 2015-06-08 15:33:58 +03:00
Valentin Kipyatkov ee15eae6b3 Smart completion: function literal argument can correspond only to the last parameter! 2015-06-08 15:33:57 +03:00
Valentin Kipyatkov 6284f557e6 Smart completion: sometimes we can insert comma after vararg argument too 2015-06-08 15:33:57 +03:00
Valentin Kipyatkov 02e876679c Completion: more correct code for lambda template insertion 2015-06-08 15:33:57 +03:00
Valentin Kipyatkov 64e9ecc7b4 Moved test data 2015-06-08 15:33:57 +03:00
Valentin Kipyatkov 7eec200887 Moved tests from deprecated to generated test fixture 2015-06-08 15:33:57 +03:00
Valentin Kipyatkov 560b7dae75 Minor correction 2015-06-08 15:33:56 +03:00
Nikolay Krasko e4c7f9bc22 Keep NullableNotNullManager methods for J2K converter 2015-06-08 15:21:03 +03:00
Valentin Kipyatkov 0cb2348b2f KT-6807 J2K: Convert "==" to "===" if class has equals() method
#KT-6807 Fixed
2015-06-06 11:00:55 +03:00
Alexey Sedunov dae7717a69 Introduce Variable: Reimplement in-place refactoring using AbstractInplaceIntroducer. Extract common superclass AbstractKotlinInplaceIntroducer 2015-06-05 21:10:36 +03:00
Alexey Sedunov 4a56b99cff Introduce Parameter: Reimplement in-place refactoring using AbstractInplaceIntroducer
This simplifies implementation and fixes bug with unnecessarily detailed command history available through undo/redo
2015-06-05 21:10:35 +03:00
Alexander Udalov d475401108 Ant task: don't pass kotlin-runtime.jar to javac on "-no-stdlib" 2015-06-05 16:19:55 +03:00
Alexander Udalov f5ec8d6850 Minor, delete useless test
'withKotlinNoJavaSources' tests exactly the same
2015-06-05 16:19:42 +03:00
Alexander Udalov e224337ff3 Ant task: support "compilerarg" element in withKotlin as in kotlinc 2015-06-05 16:19:19 +03:00
Alexander Udalov dda44a61a1 Rewrite integration tests to JUnit3, make Ant JVM task tests generated 2015-06-05 16:19:17 +03:00
Alexander Udalov 0f6c23e1c4 Don't always run code in Ant task tests
But when running code, do it via <java> task, not via custom testing code. Most
of the time the compiled code need not be runned, because it's irrelevant to
the Ant task itself
2015-06-05 15:55:59 +03:00
Alexander Udalov 968d362b77 Don't check exit code in Ant tests
It's already checked in the build log in each test
2015-06-05 15:55:58 +03:00
Alexander Udalov 89408f45bb Ant task: pass kotlin-runtime to javac's classpath; simplify tests
'kotlinCompiler' test is deleted because it's a very roundabout way to compile
Kotlin and Java sources, and there's no point in testing it.

'javacCompiler' test is deleted for similar reasons: using javac's "compiler"
option is not an advertised way to use Kotlin Ant task, and anyway it will work
with any CompilerAdapter instance, so it's hard to break it (apart from
deleting "withKotlin" adapter).

There are tests which check compilation/running of the same code but with the
recommended method. The pretext for deletion of these tests is to make Ant
tests generated, and that would be easier if they wouldn't depend on any outer
variables
2015-06-05 15:55:58 +03:00
Alexander Udalov 49c78602af Minor, move and rename CompilerClassLoaderHolder -> KotlinAntTaskUtil 2015-06-05 15:55:56 +03:00
Alexander Udalov 8f5826793d Ant task: warn when Ant version < 1.8.2 2015-06-05 15:55:56 +03:00
Alexander Udalov 0ad6dce7ce Ant task: support 'failOnError' attribute in kotlinc and withKotlin 2015-06-05 15:55:54 +03:00