aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Recreate FreshNameCreator for each run.Martin Odersky2016-11-192-3/+4
| | | | | | | Previously only the FrontEnd got a fresh FreshNameCreator for each run, the other phases used a global one instead. This means that compiling the same file several times would create different synthetic names and classes on each run.
* Plug another space leak.Martin Odersky2016-11-192-6/+4
| | | | | | | | | | | `initInfo` was retained in Symbols. When called from `Namer`, `initInfo` referred to a completer, which referred to a context. With this space leak plugged, we can now compile 1000 times core/Comments.scala (460lines) with -Xmx400M. There still seems to be a space leak on the order of 200KB per run, though. But that seems to have to do with symbols, not contexts.
* Fix memory leakMartin Odersky2016-11-191-1/+1
| | | | | | The lazy val `dummyTree` acquires a type because of copy-on-write and that type can refer via lastDenotation to a context base.
* Revert 53cd512Martin Odersky2016-11-191-4/+3
| | | | | | Re-enable context-leak detection by reverting 53cd512. But leak detection seems to be leaky itself :-)
* More extensive stats about generated treesMartin Odersky2016-11-182-3/+6
|
* Say what is compiled under -verboseMartin Odersky2016-11-181-1/+3
|
* Don't keep full picklers around until backend.Martin Odersky2016-11-183-14/+17
| | | | | | The memory footprint captured by pickler seems to be about 1/3rd of total footprint. So we gain a lot by not making this die sooner rather than later.
* Merge pull request #1724 from dotty-staging/fix-i1648odersky2016-11-182-0/+8
|\ | | | | Fix #1648: don't define companion object for java.lang.Object
| * fix #1648: mark companion object for java.lang.Object as non-existentliu fengyun2016-11-182-0/+8
|/
* Merge pull request #1722 from dotty-staging/topic/fix#1708odersky2016-11-176-15/+51
|\ | | | | Fix #1708: duplicate symbols in package
| * Annotate error in test not revealed before prev commitFelix Mulder2016-11-172-5/+5
| |
| * 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 #1720 from dotty-staging/fix-i1701odersky2016-11-174-4/+9
|\ \ | | | | | | fix #1701: disqualify package name in type name resolution
| * | don't enter package to owner's scopeliu fengyun2016-11-172-4/+3
| | |
| * | fix neg test i1701.scalaliu fengyun2016-11-171-2/+4
| | |
| * | fix failing testliu fengyun2016-11-161-1/+1
| | |
| * | add tests for #1701liu fengyun2016-11-161-0/+3
| | |
| * | fix #1701: disqualify package name in type name resolutionliu fengyun2016-11-161-3/+4
| |/
* | Merge pull request #1711 from dotty-staging/fix-isNullableodersky2016-11-171-1/+1
|\ \ | | | | | | Fix isNullable when `<:<` term refs.
| * | Add warning on != with same type fix isNullable.Nicolas Stucki2016-11-171-1/+1
| | | | | | | | | | | | | | | Note that without the fix console tests will fail and without the warning there is no way to test the fix.
* | | Merge pull request #1696 from felixmulder/topic/assert-message-lazinessFelix Mulder2016-11-176-48/+122
|\ \ \ | | | | | | | | Make sure messages are lazily evaluated until `report` in `Reporter`
| * | | Add note detailing possible memory leak in `StoreReporter`Felix Mulder2016-11-141-3/+10
| | | |
| * | | Make sure all `Message` creation is by nameFelix Mulder2016-11-143-22/+22
| | | |
| * | | Make sure messages are lazily evaluated until `report` in `Reporter`Felix Mulder2016-11-105-26/+93
| | | |
* | | | Merge pull request #1718 from dotty-staging/fix-#1703odersky2016-11-172-1/+7
|\ \ \ \ | | | | | | | | | | Fix #1703
| * | | | Move test to neg positionMartin Odersky2016-11-162-5/+5
| | | | |
| * | | | Fix #1703 - survive illegal self type clauseMartin Odersky2016-11-162-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since self types are critical we should only install one if it is syntactcally correct. Fixes #1703.
* | | | | Merge pull request #1717 from dotty-staging/fix-#1688odersky2016-11-164-16/+44
|\| | | | | |_|_|/ |/| | | Don't allow redefinition of core classes
| * | | Don't allow redefinition of core classesMartin Odersky2016-11-104-16/+44
| |/ / | | | | | | | | | Fixes #1688.
* | | 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
| | | |
generated by cgit v1.2.3 (git 2.39.1) at 2024-08-22 01:27:26 +0000