aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix #1755: Make sure references in outer args are accessibleMartin Odersky2016-12-022-2/+42
| | | | | | | Needed a fixup action in ExplicitOuter to avoid references to module's This from outside their scope. The problem is fixed, but I wish I understood better the root cause.
* Merge pull request #1693 from dotty-staging/add-annotations-phaseodersky2016-12-0117-85/+227
|\ | | | | Add annotations phase
| * Address reviewers commentsMartin Odersky2016-12-012-3/+7
| |
| * Additional test caseMartin Odersky2016-12-011-0/+2
| | | | | | | | This PR also fixes #1649
| * Fix #1647 Evaluate annotation arguments in proper contextMartin Odersky2016-12-013-3/+29
| | | | | | | | | | | | | | | | Need to evaluate annotation arguments in an expression context, since classes defined in asuch arguments should not be entered into enclosing class. Fixes #1647
| * New phase for entering annotationsMartin Odersky2016-12-016-27/+116
| | | | | | | | | | | | If we want to do annotation macros right, we need to add annotations before completing definitions. This commit achieves that by adding a new "phase" between index and typecheck.
| * Make phasesStack more robustMartin Odersky2016-12-011-1/+4
| | | | | | | | Got a "next on empty iterator" exception before.
| * Fix import disablingMartin Odersky2016-12-013-23/+35
| | | | | | | | It was broken before, since it worked only on wildcard imports.
| * Clean up importsMartin Odersky2016-12-015-9/+7
| | | | | | | | | | Honor the new scheme where any explicit import of a root import will disable the root import.
| * Be more lazy in namedImportRefMartin Odersky2016-12-011-21/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We got a hard to track down error when changing to the new annotations elaboration scheme (should be in the next commit): When running `testNonCyclic`, `DotClass` was not found in object Trees even though it was imported from `util`. It turned out that the import was ignored because the `util` symbol was completing. This commit adds a warning when this happens. The warning currently applies only to named imports because several false negatives were encountered if we do this also on wildcard imports. I.e. we get a warning, but the searched after symbol is not a member of the wildcard qualifier. This commit also refactors namedImportRef, so that `site` is only computed when the name to reseolve appears in the selector list. That change made the previously observed error go away because less is now forced.
| * Better printing of import symbolsMartin Odersky2016-12-012-7/+7
|/ | | | | Had duplications of `import` before. Also: add import info when printing contexts.
* Merge pull request #1760 from dotty-staging/fix-#1753odersky2016-11-302-2/+26
|\ | | | | Fix #1753: Better comparison of path types
| * Fix $1753 Better comparison of path typesMartin Odersky2016-11-302-2/+26
|/ | | | | | In this case, a path went through a type parameter which was aliased to a singleton type. Need to dealias to get to the special case handling two paths.
* Merge pull request #1752 from dotty-staging/publishFelix Mulder2016-11-291-1/+2
|\ | | | | fix publishing of dotty-library
| * fix publishing of dotty-libraryliu fengyun2016-11-281-1/+2
|/
* Merge pull request #1749 from dotty-staging/fix-i1748Felix Mulder2016-11-283-0/+18
|\ | | | | fix #1748: desugaring with StringContext in PatDef
| * make test runnableliu fengyun2016-11-282-3/+5
| |
| * fix #1748: desugaring with StringContext in PatDefliu fengyun2016-11-272-0/+16
|/
* Merge pull request #1719 from dotty-staging/fix-#1705odersky2016-11-242-33/+67
|\ | | | | Fix #1707: Survive non-existing positions in parser
| * Renamed test file to give correct issue numberMartin Odersky2016-11-241-0/+0
| |
| * Rename ...Pos to ...Offset and add pointOffsetMartin Odersky2016-11-241-36/+37
| |
| * More robust scheme for taking start/end of positions when parsingMartin Odersky2016-11-242-31/+64
| | | | | | | | | | | | | | Some trees, which do not consume input have unassigned positions (so that they can fit in whatever range they are integrated). It's therefore risky to take the start or end of a parsed tree's position. This commit guards against the case where the position of the tree does not exist.
* | Merge pull request #1742 from dotty-staging/topic/colon-in-printerFelix Mulder2016-11-248-22/+60
|\ \ | |/ |/| Fix colons in printer
| * Make `findTypes` take the `resultType` of `MethodType`Felix Mulder2016-11-242-12/+13
| | | | | | | | | | This is done so that we can use show for the entire method except for the ascribed type added by the compiler on success.
| * Fix defs not being printed correctlyFelix Mulder2016-11-242-3/+31
| |
| * Update tests fixing contributor PRFelix Mulder2016-11-246-17/+17
| |
| * Add colon after method type, prefix def/val/var in REPLReto Hablützel2016-11-242-3/+12
| |
* | Merge pull request #1740 from dotty-staging/fix-#1737Guillaume Martres2016-11-242-1/+14
|\ \ | |/ |/| Fix #1737: Enable GADT checking for objects
| * Enable GADT matching for pattern valuesMartin Odersky2016-11-242-1/+14
|/ | | | | So far, only typed patterns an dunapply had GADT matching. i1737.scala shows that we need to do the same thing for objects.
* Merge pull request #1739 from dotty-staging/topic/environmentally-friendly-testsFelix Mulder2016-11-2412-64/+101
|\ | | | | Environmentally friendly tests
| * More info when unpickling failsMartin Odersky2016-11-242-8/+4
| |
| * Revert fix to #1701.Martin Odersky2016-11-242-6/+11
| | | | | | | | | | | | | | | | | | | | | | Fengyun's original solution was the right one. We cannot NOT enter a package class into its parent scope, because reloading the denotation with .member will fail. So we need to enter it and compensate by adding a clause to `qualifies` in `typedIdent`. Weirdly, this was noted only when running tasty_bootstrap from a custom classpath in the new build setup. So it was pretty tricky to diagnose.
| * Compile dotty library when bootstrappingMartin Odersky2016-11-241-2/+2
| |
| * make REPL tests look at DOTTY_EXTRA classpath entriesMartin Odersky2016-11-242-2/+4
| |
| * Honor -verbose in testsMartin Odersky2016-11-231-1/+2
| | | | | | | | | | -verbose output was suppressed like all other non-error output in the tests. Now we don't filter output if -verbose is set.
| * Get rid off custom classpath in entry point testsFelix Mulder2016-11-232-17/+2
| |
| * Make sure limitedThreads <= partest.threadsFelix Mulder2016-11-231-7/+10
| |
| * Update sequential tasks portionFelix Mulder2016-11-231-12/+21
| |
| * Make tests depend on environment variables for classpath jarsFelix Mulder2016-11-237-25/+61
|/ | | | | | | | | | | | sbt adds the correct jars to classpath and the tests depend on `packageAll` which creates these. When using something else however, these together with `sbt-interfaces` do not get propagated from the build. To remedy this and make the testing a bit more flexible, we now take these from `sys.props` instead, see `tests/dotty/Jars.scala`. If the props aren't defined we fall back to the ones default to sbt.
* Merge pull request #1736 from felixmulder/topic/docker-to-lampepflFelix Mulder2016-11-231-1/+1
|\ | | | | Change docker image to use lampepfl one
| * Change docker image to use lampepfl oneFelix Mulder2016-11-231-1/+1
| |
* | Add build status badgeFelix Mulder2016-11-231-0/+1
|/
* Merge pull request #1734 from felixmulder/topic/add-droneFelix Mulder2016-11-2386-96/+126
|\ | | | | [WIP] add drone
| * make drone use img with openjdk8 and javadocFelix Mulder2016-11-221-2/+2
| |
| * Fix scripted tests when the dotty artifact is not in cacheGuillaume Martres2016-11-221-4/+11
| |
| * Propagate memory setting to testsFelix Mulder2016-11-222-5/+10
| |
| * dotty-bridge is now called dotty-sbt-bridgeGuillaume Martres2016-11-2283-86/+86
| |
| * Redefine scripted testFelix Mulder2016-11-221-2/+8
| |
| * increase threads to 2 in reduced threads casesFelix Mulder2016-11-222-3/+4
| |
| * Add drone integrationFelix Mulder2016-11-223-2/+13
|/