Commit Graph

31106 Commits

Author SHA1 Message Date
Valentin Kipyatkov 3dc32fb5db Minor change on code review 2016-04-19 15:22:06 +03:00
Valentin Kipyatkov bade304bf3 Better presentation 2016-04-19 15:22:06 +03:00
Valentin Kipyatkov 022075995b KT-11957 No "catch" and "finally" keywords in completion
#KT-11957 Fixed
2016-04-19 15:22:06 +03:00
Valentin Kipyatkov c7a96401da Refactoring 2016-04-19 15:22:06 +03:00
Valentin Kipyatkov 2a3b87f783 KT-11784 Completion for 'if' statement should add parentheses automatically
#KT-11784 Fixed
2016-04-19 15:22:06 +03:00
Valentin Kipyatkov bf4a376831 Use CommentSaver instead of manual comment handling 2016-04-19 15:22:05 +03:00
Valentin Kipyatkov e72e6bf7db KT-11870 "Replace with Elvis" refactoring doesn't change the variable type from T? to T
#KT-11870 Fixed
2016-04-19 15:22:05 +03:00
Valentin Kipyatkov 6bdf3622ed KT-11890 Completion for callable references does not propose static Java members
#KT-11890 Fixed
2016-04-19 15:22:05 +03:00
Valentin Kipyatkov f3bb3e9c6e KT-11912 String interpolation is not converted to ${} form when accessing this.property
#KT-11912 Fixed
2016-04-19 15:22:05 +03:00
Dmitry Petrov e441f0509f KT-11943 Do not use intrinsic 'java' property if receiver type is not acceptable 2016-04-19 10:31:30 +03:00
Valentin Kipyatkov 120bced842 Fixing build 2016-04-18 23:05:59 +03:00
Alexey Tsvetkov 23c21396f7 Make Kotlin2JsGradlePluginIT test compatible with gradle 2.x 2016-04-18 22:12:08 +03:00
Alexey Tsvetkov 51104a5c2b Make gradle daemon thread leak test more thorough
Former test made the following assertion:
for any of 3 sequent builds on the same gradle daemon
the difference in used memory between build start and finish
should not exceed 2500 kb.

Current test makes the following assertion:
the difference of used memory after first and last
of 10 sequent builds on the same daemon should not exceed 200 kb.
2016-04-18 21:31:41 +03:00
Alexey Tsvetkov d446e60e70 Remove ThreadTracker from gradle plugin
ThreadTracker was introduced to detect memory leaks
when using gradle daemon. However, it makes
quite strong assumption about build lifecycle:
there should not be any builds started after
kotlin plugin is configured that are alive when
buildFinished callback is called.

This assumption can be broken by some gradle configurations
namely our android tests and gradle 2.13-rc.

Since we already have daemon memory leak test (KotlinGradlePluginIT#testKotlinOnlyDaemonMemory),
after some discussion it was decided to remove ThreadTracker
2016-04-18 21:31:41 +03:00
Alexander Udalov 2df8e56b94 Do not store project in JPS tests after they're executed
This saves a couple dozen megabytes currently
2016-04-18 17:26:59 +03:00
Alexander Udalov e941ba761a Minor, fix case in test data file name 2016-04-18 17:26:58 +03:00
Valentin Kipyatkov bd5af61879 Moved ShortenReferences from idea-analysis to idea-core, added dependency from idea-analysis to idea-core 2016-04-18 17:15:52 +03:00
Valentin Kipyatkov da4dee3716 Moved generateUtil.kt into idea-core 2016-04-18 17:15:52 +03:00
Valentin Kipyatkov b32859cf68 KT-11633 Kotlin code editor applies wrong indentation after completing a statement in data class
#KT-11633 Fixed
2016-04-18 17:15:52 +03:00
Ilya Chernikov 6b3a691367 Fix KT-10507 - "missing extension point" with parallel compilation 2016-04-18 15:16:48 +02:00
Mikhail Glukhikh 3f266647f0 Does not replace explicit lambda parameter with 'it' if conflicting nested literal available #KT-11849 Fixed 2016-04-18 15:26:42 +03:00
Nikolay Krasko 2e1a2fa8d1 Fallback to default source provider for non-gradle projects 2016-04-18 14:06:45 +03:00
Mikhail Glukhikh c7b2042b77 Do not suggest to add variance modifier if both in and out are possible 2016-04-18 13:40:55 +03:00
Mikhail Glukhikh e60930d5ce Add variance modifier fix corrected: primary constructor parameters are taken into account 2016-04-18 13:40:50 +03:00
Alexander Udalov 027cc898e7 Minor, fix warnings in core/ modules 2016-04-17 13:35:17 +03:00
Mikhail Zarechenskiy f773966458 Create artifact with 'formatter' module output to share it with Eclipse plugin 2016-04-15 18:15:13 +03:00
Mikhail Zarechenskiy 37e0b71584 Split KotlinBlock on two clases to share independent logic about
formatting  between Idea and Eclipse

 Also make 'formatter' module depend on fewer dependencies
2016-04-15 18:15:13 +03:00
Mikhail Zarechenskiy 4c006300ef Move files related to formatting to module 'formatter' as is 2016-04-15 18:15:13 +03:00
Mikhail Zarechenskiy a7c397fe84 Create 'formatter' module
Purpose of this module is to extract common code  about formatting and reuse it in Eclipse
2016-04-15 18:15:13 +03:00
Mikhail Zarechenskiy ac1ca890fd J2K KotlinBlock: convert and prettify
Also get rid of ContainerUtil
2016-04-15 18:15:13 +03:00
Mikhail Zarechenskiy 22289cf337 J2K KotlinBlock: rename 2016-04-15 18:15:13 +03:00
Natalia Ukhorskaya e8f46f5315 Debugger: 'Step Over' unexpectedly resume if condition of 'while' loop contains inline function call
#KT-11932 Fixed
2016-04-15 18:11:36 +03:00
Natalia Ukhorskaya f0badb05ea Debugger: fix step over for inline argument with non-local return 2016-04-15 18:11:36 +03:00
Natalia Ukhorskaya f0cfd450e5 Debugger: fix step over inside inline function (do not step into function literal argument) 2016-04-15 18:11:36 +03:00
Alexey Andreev 8ad339836d KT-11875 Moves code that uncovers the following JS function:
```
function foo (closureParam, closureParam2, ...) {
    return function() {
        // function body
    }
}
```

from FunctionContext to FunctionInlineMutator. The old code causes bug when calling this function like this: `foo(this, ...)`, since first `closureParam` is replaced with `this`. Later, FunctionInlineMutator can't make difference between original `this` inside *function body* and former `closureParam` reference that became `this` after replacement. So, let FunctionInlineMutator replace `this` in *function body* and then substitute closure parameters.

KT-11875 Fixed
2016-04-15 17:35:30 +03:00
Natalia Ukhorskaya 8129901d82 Minor: clarify exception messages during Evaluate Expression 2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya 316057873b Fix EA-72969 - assert: ExtractFunctionForDebuggerUtilKt$getFunctionForExtractedFragment$.invoke : ERROR_TYPES
Context element for codeFragment can be a KtFile (not  only KtExpression) ex. in case when file was changed after debugger starts and there is no expression at line with breakpoint
2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya 7fb3099206 Fix EA-80854 - IOOBE: KotlinLineBreakpointType.getLambdaByOrdinal.
This could happen when breakpoint in lambda was set but than the code changed and there is no lambdas at this line anymore
2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya a929df6fa1 Debugger, minor: do not throw NPE when receiverTypes are null in completion 2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya d42823d4ee Debugger: cannot evaluate expression with raw types
#KT-11831 Fixed
2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya 070ee10872 Debugger: do not insert ambiguous imports inside codeFragments
#KT-11927 Fixed
2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya d7a87b2ff0 Debugger: fix breakpoints inside generic crossinline argument body
#KT-11848 Fixed
2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya 7a5ab93f9e Debugger: write line number for catch clause declaration 2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya 8e3ec5b7f0 Debugger: fix breakpoints inside inline functions in libraries sources 2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya 3ef51159b8 Debugger: remove unnecessary whitespaces in presentation of line breakpoint variants 2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya e55a1ad49c Get correct target platform for codeFragments 2016-04-15 15:58:07 +03:00
Natalia Ukhorskaya 0d4fa46cfa Debugger: correct scope for runtime type calculation 2016-04-15 15:58:07 +03:00
Mikhail Glukhikh e806341cdc Increase visibility: make not available in case of potential exposed visibility problems (see KT-11920) 2016-04-15 15:20:35 +03:00
Mikhail Glukhikh 6acd4bc21d Refactoring: ExposedVisibilityChecker extracted from DeclarationsChecker 2016-04-15 15:20:30 +03:00
Mikhail Glukhikh ed4ccc6a19 Minor refactoring: EffectiveVisibility 2016-04-15 15:20:26 +03:00