aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix #1708: return ErrorTree in `typedPackageDef` if pkg has no symbolFelix Mulder2016-11-172-9/+14
|
* Only issue error, no unlinkingFelix Mulder2016-11-172-8/+8
| | | | | | Unlinking proved to cause other problems in the typer, specifically if typechecking members of non-existing package or things referring to the unlinked package
* Unlink type when entering clashing packageFelix Mulder2016-11-175-4/+35
| | | | | | | | | Here we unlink the existing type that clashes with the package to be entered into the symbol table, issue an error and the proceed to enter the rest of the symbols. My concern with this approach is what happens during typechecking if other things reference the unlinked type.
* Merge pull request #1697 from dotty-staging/fix-#1664Dmitry Petrashko2016-11-162-8/+24
|\ | | | | Fix #1664: Refine isOuterRef condition
| * Fix #1664: Refine isOuterRef conditionMartin Odersky2016-11-102-8/+24
| | | | | | | | | | | | | | | | We forgot the case where a hoistable method can still refer to free variables that have to be passed using outer pointers. Fixes #1664.
* | Merge pull request #1666 from dotty-staging/fix-#1653odersky2016-11-165-12/+24
|\ \ | | | | | | Fix #1653: Check "no inherit from final" earlier.
| * | Refactor testMartin Odersky2016-11-072-2/+9
| | | | | | | | | | | | | | | | | | | | | After refactoring, one test (cannot extend final class) shadowed the others, which wer done in a later phase. Splitting in two to get all errors again.
| * | Fix #1653: Check "no inherit from final" earlier.Martin Odersky2016-11-063-10/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test case is an illegal inheritance from a primitive value class, which makes this an illegal value class. Previously, the error was detected by refchecks, but crashes occured earlier (when generating synthesized methods) or at the same phase block (in extension methods). The problem is avoided by moving the test to Namer. Review by @smarter.
* | | Merge pull request #1695 from dotty-staging/fix-#1662odersky2016-11-162-2/+9
|\ \ \ | | | | | | | | Fix #1662: Add missing case for singleton
| * | | Fix #1662: Add missing case for singletonMartin Odersky2016-11-102-2/+9
| | |/ | |/| | | | | | | | | | | | | Have to handle also SkolemTypes there. Fixes #1662.
* | | Merge pull request #1700 from maseev/iss1589-illegal-start-simple-patternFelix Mulder2016-11-162-1/+80
|\ \ \ | | | | | | | | Add an error message for illegal start of simple pattern - Parsers.scala:1437
| * | | Add an error message for illegal start of simple patternMiron Aseev2016-11-162-1/+80
|/ / /
* | | Merge pull request #1715 from lampepfl/DarkDimius-patch-1Felix Mulder2016-11-161-1/+1
|\ \ \ | | | | | | | | Try to fix the CI
| * | | Try to fix the CIDmitry Petrashko2016-11-151-1/+1
|/ / /
* | | Merge pull request #1698 from maseev/iss1589-sequence-wildcard-pattern-messageFelix Mulder2016-11-142-2/+25
|\ \ \ | | | | | | | | Add an error message for incorrect sequence wildcard position Parsers.scala:1425 and Parsers.scala:1079
| * | | Add an error message for incorrect sequence wildcard pattern positionMiron Aseev2016-11-142-2/+25
|/ / /
* | | Merge pull request #1571 from krasinski/report_githash_in_replFelix Mulder2016-11-143-1/+26
|\ \ \ | | | | | | | | Closes #1319 Report git-hash used to package the distribution in REPL
| * | | Report git-hash used to package the distribution - Closes #1319Bartosz KrasiƄski2016-11-123-1/+26
|/ / /
* | | Merge pull request #1678 from Blaisorblade/sync-diagramsGuillaume Martres2016-11-122-3/+10
|\ \ \ | | | | | | | | Resync copy of the type diagram in docs
| * | | Resync copy of the type diagram in docsPaolo G. Giarrusso2016-11-072-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Also add a note that the two diagrams should be kept in sync, both ways, so there's a chance that the copies are kept in sync. Alternatively one could drop a copy.
* | | | Merge pull request #1699 from Blaisorblade/clarify-errorodersky2016-11-113-9/+12
|\ \ \ \ | |_|/ / |/| | | Clarify error for unbound identifiers and add/fix a few comments
| * | | Clarify error for unbound identifierPaolo G. Giarrusso2016-11-111-3/+5
| | | |
| * | | Clarify ScalaDocs for MessagePaolo G. Giarrusso2016-11-111-1/+2
| | | |
| * | | Fix typosPaolo G. Giarrusso2016-11-112-4/+4
| | | |
| * | | Fix extra spacePaolo G. Giarrusso2016-11-111-1/+1
|/ / /
* | | Merge pull request #1677 from dotty-staging/fix-#1647odersky2016-11-105-25/+10
|\ \ \ | | | | | | | | Use inline flag instead of @inline annotation
| * | | Separate addInlineInfo and addAnnotationsMartin Odersky2016-11-071-3/+3
| | | |
| * | | Address reviewer commentsMartin Odersky2016-11-071-1/+0
| | | |
| * | | Use inline flag instead of @inline annotationMartin Odersky2016-11-075-23/+9
| | | | | | | | | | | | | | | | | | | | Convert `@inline` annotations to `inline` flags, not the other way round as was done before.
* | | | Merge pull request #1609 from dotty-staging/patmat-testFelix Mulder2016-11-0950-417/+257
|\ \ \ \ | | | | | | | | | | add back the forgotten test file for patmat exhuastivity check
| * | | | Change `ClassicReporter` to `TestReporter` in test sourcesFelix Mulder2016-11-0951-506/+171
| | | | |
| * | | | add the forgotten patmat testliu fengyun2016-11-092-1/+91
| | | | |
| * | | | add back classic reporterliu fengyun2016-11-091-0/+85
|/ / / /
* | | | Update feature list on webpageFelix Mulder2016-11-091-7/+2
| | | |
* | | | Merge pull request #1684 from dotty-staging/fix-1672liu fengyun2016-11-081-0/+7
|\ \ \ \ | |_|/ / |/| | | Fixed #1672: Add regression test.
| * | | Fixed #1672: Add regression test.Nicolas Stucki2016-11-081-0/+7
|/ / / | | | | | | | | | This issue was solved by some other fix.
* | | Merge pull request #1607 from felixmulder/topic/fix-inline-untypedodersky2016-11-073-3/+23
|\ \ \ | | | | | | | | Fix #1605: don't inline methods that have errors
| * | | Inline iff reporter has no new errors post typing `rhs`Felix Mulder2016-11-072-4/+17
| | | |
| * | | Fix #1605: don't inline methods that have errorsFelix Mulder2016-11-072-2/+9
| | | |
* | | | Merge pull request #1676 from felixmulder/topic/docs-add-searchFelix Mulder2016-11-0725-130/+263
|\ \ \ \ | |/ / / |/| | | Fix a number of doc issues
| * | | Fix highlighting in search, add highlights in forwarding linksFelix Mulder2016-11-077-21/+78
| | | |
| * | | Fix #1675: add `edit on github` buttonFelix Mulder2016-11-073-0/+11
| | | |
| * | | Update READMEFelix Mulder2016-11-071-81/+13
| | | |
| * | | Fix #1674: add search to doc siteFelix Mulder2016-11-0722-46/+179
|/ / /
* | | Merge pull request #1632 from dotty-staging/fix/hash-assertFelix Mulder2016-11-072-6/+9
|\ \ \ | | | | | | | | Avoid calling unsupported computeHash in some situations
| * | | Avoid calling unsupported computeHash in some situationsGuillaume Martres2016-10-262-6/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some types do not implement `computeHash`, instead they override `myHash` directly. This works fine as long as `myHash` is not equal to `HashUnknown` but this was not guaranteed before this commit, if `myHash` is equal to `HashUnknown` then `computeHash` is called by `CachedGroundType#hash` or `CachedProxyType#hash` causing an exception: https://gist.github.com/smarter/6b642db0495e995d8f3c26d614cf54d6 This commit fixes this by making sure we never compute a hash equal to `HashUnknown`, instead `HashUnknownAlt` should be used.
* | | | Fix typo in `IllegalLiteral` messageFelix Mulder2016-11-061-1/+1
| | | |
* | | | Merge pull request #1669 from dos65/docs_encodingFelix Mulder2016-11-061-0/+2
|\ \ \ \ | | | | | | | | | | Fix #1668 - add encoding for default docs layout
| * | | | Fix #1668 - add encoding for default docs layoutdos652016-11-061-0/+2
| | | | |
* | | | | Merge pull request #1667 from maseev/iss1589-literal-error-messageFelix Mulder2016-11-062-1/+15
|\ \ \ \ \ | | | | | | | | | | | | Add error message for illegal literals - Parser.scala:602