Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | Rename unused phases. | Dmitry Petrashko | 2016-02-20 | 3 | -0/+0 | |
| | | | | ||||||
| * | | | FirstTransform: simplify needsCompanion code | Dmitry Petrashko | 2016-02-20 | 1 | -4/+6 | |
| | | | | ||||||
| * | | | Do not create companions that will be dropped later. | Dmitry Petrashko | 2016-02-20 | 5 | -6/+31 | |
|/ / / | | | | | | | | | | | | | | | | | | | | | | Fix blocker bug reported in #1114 I dislike this fix as now phase needs to know in advance if it will ever need a companion for the class. On the bright side, this change makes it clear which phases need companions | |||||
* | | | Merge pull request #1112 from dotty-staging/value-classes-private-this-param | Dmitry Petrashko | 2016-02-20 | 5 | -7/+22 | |
|\ \ \ | | | | | | | | | Value classes: add support for private[this] parameter | |||||
| * | | | fix to neg_valueClasses test | VladimirNik | 2016-02-19 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Add support for private[this] parameter in value classes | VladimirNik | 2016-02-19 | 4 | -6/+21 | |
| | | | | ||||||
* | | | | Merge pull request #1120 from dotty-staging/fix-1114-patmat | odersky | 2016-02-20 | 2 | -1/+22 | |
|\ \ \ \ | | | | | | | | | | | PatMat: remove one more trace of doing several steps at a time | |||||
| * | | | | Test that #1114 is fixed. | Dmitry Petrashko | 2016-02-20 | 1 | -0/+21 | |
| | | | | | ||||||
| * | | | | PatMat: remove one more trace of doing several steps at a time | Dmitry Petrashko | 2016-02-20 | 1 | -1/+1 | |
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | When creating subPatRefs the consider returned type of accessor, not the type of binder. Fixes #1114 | |||||
* | | | | Merge pull request #1119 from dotty-staging/fix-#1116 | Dmitry Petrashko | 2016-02-20 | 1 | -1/+1 | |
|\ \ \ \ | |/ / / |/| | | | Default getters accessibility fix | |||||
| * | | | Don't copy annotations from method to default getter. | Martin Odersky | 2016-02-20 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Default getters accessibility fix | Martin Odersky | 2016-02-20 | 1 | -1/+1 | |
|/ / / | | | | | | | | | | | | | | | | Default getters should have same accessibility as the method they belong to. Previously, it was the accessibility of the parameter, which makes no sense. Fixes #1116. | |||||
* | | | Merge pull request #1102 from dotty-staging/fix-#1096 | odersky | 2016-02-19 | 1 | -6/+4 | |
|\ \ \ | | | | | | | | | Keep named arguments during eta expansion | |||||
| * | | | Keep named arguments during eta expansion | Martin Odersky | 2016-02-18 | 1 | -6/+4 | |
| | | | | | | | | | | | | | | | | Fixes #1096. | |||||
* | | | | Merge pull request #1066 from dotty-staging/fix-#997 | odersky | 2016-02-19 | 13 | -16/+140 | |
|\ \ \ \ | | | | | | | | | | | Fix #997 | |||||
| * | | | | Add error markers | Martin Odersky | 2016-02-19 | 1 | -2/+2 | |
| | | | | | ||||||
| * | | | | Merge parentsWithArgs and instantiatedParents | Martin Odersky | 2016-02-19 | 4 | -9/+6 | |
| | | | | | ||||||
| * | | | | Fix expected error count | Martin Odersky | 2016-02-19 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Move leak detection to Checking | Martin Odersky | 2016-02-19 | 3 | -34/+44 | |
| | | | | | | | | | | | | | | | | | | | | Also: include a test that private aliases are transparent. | |||||
| * | | | | Fix two private leaks in dotty compiler itself. | Martin Odersky | 2016-02-19 | 2 | -3/+3 | |
| | | | | | ||||||
| * | | | | Add checking for leaking private definitions | Martin Odersky | 2016-02-19 | 5 | -5/+98 | |
| | | | | | | | | | | | | | | | | | | | | First version. Fixes #997. | |||||
| * | | | | New utility method: Reporter#errorOrMigrationWarning | Martin Odersky | 2016-02-19 | 1 | -0/+3 | |
| | | | | | ||||||
| * | | | | Copy access flags to derived definitions during desugaring | Martin Odersky | 2016-02-19 | 2 | -5/+26 | |
| | |/ / | |/| | | | | | | | | | | Previously, some definitions were too public, others too private. | |||||
* | | | | Merge pull request #1072 from dotty-staging/change-isVolatile-2 | odersky | 2016-02-19 | 29 | -108/+957 | |
|\ \ \ \ | |/ / / |/| | | | Change is volatile 2 | |||||
| * | | | Fix merge breakage. | Martin Odersky | 2016-02-19 | 1 | -0/+1 | |
| | | | | ||||||
| * | | | Remove unused method, fix comments. | Martin Odersky | 2016-02-19 | 2 | -19/+9 | |
| | | | | | | | | | | | | | | | | | | | | Addendum to change-isVolatile. Changes did not make it in by accident before that branch was merged. | |||||
| * | | | Simplify logic in matchNamed | Martin Odersky | 2016-02-19 | 1 | -4/+3 | |
| | | | | ||||||
| * | | | Allow Named Arguments in TypeArgs | Martin Odersky | 2016-02-19 | 10 | -20/+121 | |
| | | | | | | | | | | | | | | | | Lets one also pass named arguments to methods. | |||||
| * | | | Use hasNamedArgs instead of repeating test inline. | Martin Odersky | 2016-02-19 | 4 | -8/+9 | |
| | | | | ||||||
| * | | | Add some flexibility in comparing named and unnamed parameterized types. | Martin Odersky | 2016-02-19 | 2 | -3/+96 | |
| | | | | ||||||
| * | | | Take defult parameters into account for overloading resolution. | Martin Odersky | 2016-02-19 | 1 | -11/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current Scala spec only considers methods without default parameters for overloading resolution (unless only a single one remains anyway after filtering by shape). This is needlessly restrictive. But dropping this restriction (as dotty does) can lead to ambiguity errors, which is why run/t8197 did not compile anymore. We fix the problem by a last try rule: If after asSpecific tests there are still several alternatives, and only one of them is without default arguments, pick that one. I tried an alternative rule which would make the distinction on default params earlier but that one fails for the overloaded tree copier functions in Trees.scala (the method with default parameters is also the one which is more specific). | |||||
| * | | | Fix problem of overloading resolution when receiver is not stable. | Martin Odersky | 2016-02-19 | 4 | -7/+9 | |
| | | | | ||||||
| * | | | Fix review comment in previous PR | Martin Odersky | 2016-02-19 | 1 | -2/+1 | |
| | | | | | | | | | | | | | | | | Was not fixed by accident then, so we do it here now. | |||||
| * | | | Fix assertion error message | Martin Odersky | 2016-02-19 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Add test to illustrate overloading problem. | Martin Odersky | 2016-02-19 | 1 | -0/+15 | |
| | | | | ||||||
| * | | | Make type parameter reordering generally available. | Martin Odersky | 2016-02-19 | 2 | -11/+20 | |
| | | | | ||||||
| * | | | Add doc comment | Martin Odersky | 2016-02-19 | 1 | -0/+1 | |
| | | | | ||||||
| * | | | Address reviewer comments | Martin Odersky | 2016-02-19 | 2 | -2/+3 | |
| | | | | ||||||
| * | | | More named param tests | Martin Odersky | 2016-02-19 | 3 | -0/+437 | |
| | | | | | | | | | | | | | | | | Also, add an unrelated test file to pos. | |||||
| * | | | Annotate test with // error indications | Martin Odersky | 2016-02-19 | 1 | -6/+7 | |
| | | | | | | | | | | | | | | | | | | | | We had same fleyness in number of errors of cycle.scala which prompted this. | |||||
| * | | | Check named type params for welformedness rules. | Martin Odersky | 2016-02-19 | 3 | -4/+75 | |
| | | | | ||||||
| * | | | Fix PostTyper normalization for named args | Martin Odersky | 2016-02-19 | 1 | -6/+4 | |
| | | | | | | | | | | | | | | | | | | | | Needs to work also if named arg refers to an abstract type, not a parameter. | |||||
| * | | | Fix pending test for pickling | Martin Odersky | 2016-02-19 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | There is a before/after difference having to do with the order in which class declarations show up. | |||||
| * | | | Move failing test to pending | Martin Odersky | 2016-02-19 | 1 | -0/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Named parameters cause some elements to be unpickled in a different order as they are pickled. In particular term parameter aliases and type parameter aliases seem to be swapped. | |||||
| * | | | Avoid cyclic reference error when building dotty. | Martin Odersky | 2016-02-19 | 1 | -2/+1 | |
| | | | | ||||||
| * | | | Make named parameter alias handling more robust | Martin Odersky | 2016-02-19 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After unpickling we might see an alias X = pre.X where pre =:= the ThisType of the enclosing class. But it might not be `eq` to it. | |||||
| * | | | Augment test case | Martin Odersky | 2016-02-19 | 2 | -6/+15 | |
| | | | | ||||||
| * | | | Make some operations surivive partial named parameter lists | Martin Odersky | 2016-02-19 | 3 | -8/+19 | |
| | | | | | | | | | | | | | | | | | | | | Types#underlyingClassRef and PostTyper#normalizeTree need to be changed so they can deal with partial named parameter lists. | |||||
| * | | | First version of named type arguments | Martin Odersky | 2016-02-19 | 6 | -24/+121 | |
| | | | | ||||||
| * | | | Support named type parameters | Martin Odersky | 2016-02-19 | 4 | -3/+11 | |
| | | | | | | | | | | | | | | | | | | | | Changes needed to support simple named type parameters. Not yet implemented: named arguments. |