aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * Run some tests sequentially to avoid exhausting available memoryGuillaume Martres2016-11-221-0/+39
| | | | | | | | | | | | Some tests are run with "-Ytest-pickler" which uses a huge amount of memory. By running these tests one by one when no other test is running, we avoid running out of memory.
| * Fix remaining tests with incorrect paths following moveGuillaume Martres2016-11-224-254/+253
| |
| * Move PatmatExhaustivityTest.scala in its proper placeGuillaume Martres2016-11-221-0/+0
| |
| * ExtractDependencies: correctly record inheritance on type aliasGuillaume Martres2016-11-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This fix the test "Extracted source dependencies from public members" which previously failed with: Set('G, 'E) is not equal to Set('B, 'E) (DependencySpecification.scala:34) `H` extends `G.T[Int]` which is an alias of `B`, so the `topLevelInheritanceDepndencies` of `H` should contain `B`, this was not the case before because we didn't dealias before looking for the top-level class of the dependency, so we ended up with `G`, the top-level class in which the alias `T` is contained.
| * Do not force symbols of annotations when unpicklingGuillaume Martres2016-11-222-12/+18
| | | | | | | | | | | | | | | | | | | | | | | | This lead to stale symbol errors in `tasty_tools` because the symbol forcing was bringing forward symbols from the previous run before the corresponding symbols for the current run were created. We fix this by adding Annotations#deferredSymAndTree which behaves similarly to Annotations#deferred but take a by-name symbol. We also remove TreeUnpickler#LazyAnnotationReader which was apparently never used.
| * Workaround last issue with -Ytest-picklerGuillaume Martres2016-11-221-1/+1
| | | | | | | | | | | | | | We run checkNoPrivateLeaks in the unpickler since last commit and this is causing an issue related to privacy leaks. Give up and workaround it since we're going to redesign how we handle privacy leaks because of https://github.com/lampepfl/dotty/issues/1723 anyway.
| * checkNoPrivateLeaks is now also done in TreeUnpicklerGuillaume Martres2016-11-223-0/+17
| | | | | | | | | | This could be abstracted better but we'll end up replacing checkNoPrivateLeaks soon anyway due to https://github.com/lampepfl/dotty/issues/1723
| * checkNoPrivateLeaks: handle defs in local classesGuillaume Martres2016-11-223-1/+20
| | | | | | | | | | | | The access boundary of a def in a local class is the owner of that class, previously it was set to the access boundary of the owner of the class instead.
| * checkNoPrivateLeaks: handle references to companion membersGuillaume Martres2016-11-223-2/+40
| | | | | | | | | | | | | | | | | | | | | | Previously Outer2#Inner#foo failed to compile with: ``` non-private method foo refers to private value x in its type signature ``` This should compile because the boundary of `foo` is `class Outer2` and the boundary of `x` is `object Outer2`. This commit fixes this by also considering the linked boundary in `checkNoPrivateLeaks`.
| * bin/common: jar is dotty-compiler, not dottyGuillaume Martres2016-11-221-1/+1
| |
| * TreeUnpickler: Add missing case for ENUMconstGuillaume Martres2016-11-221-0/+2
| |
| * -Ytest-pickler: avoid forcing anything when there's a differenceGuillaume Martres2016-11-221-1/+1
| | | | | | | | | | This might cause a stale symbol exception and make it harder to find the source of the problem.
| * -Ytest-pickler: homogenize template body output orderGuillaume Martres2016-11-221-1/+14
| |
| * TypedTreeCopier#Select: don't use unstable prefixesGuillaume Martres2016-11-221-1/+1
| | | | | | | | | | | | | | | | | | This manifested itself as a pickling difference in tasty_tools Note that there are probably more issues in this method, in particular the old type is reused when `qualifier.tpe eq tree.qualifier.tpe` even if the `name` is different. But I'm only trying to get the tests to pass for now.
| * Fix flags for default getters of constructorsGuillaume Martres2016-11-222-1/+9
| | | | | | | | This manifested itself as a pickling difference in tasty_tools
| * Move bridge to correct location, fix unmanaged sources in buildFelix Mulder2016-11-22508-2/+2
| |
| * The root project now aggregates the other projectsGuillaume Martres2016-11-221-1/+7
| | | | | | | | | | Before this commit, commands such as "test" would not run anything, which means that the JUnit tests were never run.
| * Run dotty tests with the correct baseDirectoryGuillaume Martres2016-11-221-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this commit, Build.scala contained: baseDirectory in run := baseDirectory.value / "..", The goal of this line is that when invoking the compiler using `run` or `dotc`, the working directory is the root of the dotty project, which is convenient for testing. However, this line also affects the working directory of `dotty-compiler/runPartestRunner`, but not of `dotty-compiler/test`. The result was that `dotty-compiler/test` was broken, and fixing the paths in the code would break `partest`. We solve this by using a different baseDirectory when running partest.
| * Fix partest commandsGuillaume Martres2016-11-221-6/+6
| |
| * Add packageAll alias in root projectGuillaume Martres2016-11-221-0/+1
| | | | | | | | | | Otherwise `packageAll` by itself as used in aliases like `partest` won't work.
| * Move `dottydoc` -> `doc-tool`Felix Mulder2016-11-2245-2/+2
| |
| * Remove unused examples dir, duplicate contained in tests/posFelix Mulder2016-11-224-10/+3
| |
| * Update documentation for new sbt commandsFelix Mulder2016-11-224-21/+63
| |
| * Make `dotr` run binaries similar to script in binFelix Mulder2016-11-221-2/+29
| |
| * Add `dotc` command with sane classpathFelix Mulder2016-11-221-5/+22
| |
| * Add `repl` commandFelix Mulder2016-11-221-6/+23
| |
| * Fix rebuild on change in scriptsFelix Mulder2016-11-221-0/+1
| |
| * Move compiler and compiler tests to compiler dirFelix Mulder2016-11-22329-74/+84
| |
| * Move (most) unit tests to correct locationsFelix Mulder2016-11-2249-473/+139
| | | | | | | | | | | | Should still perhaps move `test/dotc/tests.scala` and the others in the same directory to a better more cohesive location. Would like to delete the worksheets as well - but maybe they hold sentimental value...
| * Make homogenize apply HKApplysFelix Mulder2016-11-221-0/+2
| | | | | | | | | | The pickler doesn't unpickle to HKApply, as such - we need to print them in their applied form.
| * TastyPrinter: fix RefinedType printingGuillaume Martres2016-11-221-1/+1
| |
| * dottyDependsOnBootedLib should not be a JUnit testGuillaume Martres2016-11-221-1/+1
| | | | | | | | As indicated by the comment right above it.
| * Fix pickle_pickleOK testGuillaume Martres2016-11-222-0/+4
| | | | | | | | | | | | | | It failed because two files in the same package compiled at the same time defined respectively a class Test and an object Test. I'm guessing this did not fail before because failure depends on the order of the files on the command line.
| * Fix "sbt run"Guillaume Martres2016-11-221-0/+3
| | | | | | | | | | | | Broken since the dotty jars were removed from the JVM bootclasspath. I think the proper solution is to use the Scala bootclasspath to pass all the necessary jars but this will do for now.
| * PathResolver: fix priorityClassPathGuillaume Martres2016-11-221-1/+1
| | | | | | | | | | It was wrongly defined before with respect to `commandLineFor` (this is what happens when you rely on strings instead of types!)
| * Fix partest compilation of java-interop tests by moving themGuillaume Martres2016-11-2276-1/+1
| | | | | | | | | | | | | | | | | | | | Previously, the tests where all in tests/pos/java-interop which means that partest would try to run them as one single test, this failed because some of these tests define classes with the same name. We could fix this by putting them all in separate packages but for now it's simple to move them to tests/pos-java-interop (they will still be run individually as they should thanks to the `java_all` test defined in tests.scala)
| * Increanse Jenkins max heap for 1.3G to 1.5GGuillaume Martres2016-11-221-1/+1
| | | | | | | | | | | | The fix to partest to make it compile files in subdirectories means that we're compiling more things and using more heap and has resulted in tests failing with OutOfMemoryError. Hopefully this fixes that.
| * Fix JUnit tests by always packaging everythingGuillaume Martres2016-11-221-8/+12
| | | | | | | | | | | | Running `packageAll` is now required before running the tests, this commit enforces this, by making `javaOptions` depend on `packageAll`, this requires rewriting `packageAll` to be a task instead of a command alias.
| * Move sjs, make sure that partest compiles everything in dirsFelix Mulder2016-11-2218-187/+149
| |
| * Add bin project to separate scripted tests from compiler testsFelix Mulder2016-11-222-18/+31
| |
| * Add tags file to gitignoreFelix Mulder2016-11-221-0/+1
| |
| * Add dotty-library to classpath in dottydoc testsFelix Mulder2016-11-2212-80/+111
| |
| * Create dotty-lib.jar for run testsFelix Mulder2016-11-2217-135/+268
| |
| * Remove compiler when running applications via dotrFelix Mulder2016-11-222-3/+6
| |
| * separate lib from compilerFelix Mulder2016-11-22166-31/+43
| |
| * Move sbt-bridgeFelix Mulder2016-11-22508-1/+1
|/
* Merge pull request #1725 from dotty-staging/change-pickle-earlyGuillaume Martres2016-11-229-31/+37
|\ | | | | Don't retain picklers until backend.
| * 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.