aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Check that cases with type parameters also have an extends clauseMartin Odersky2017-04-042-1/+6
|
* Fix typo in syntaxMartin Odersky2017-04-041-1/+1
|
* Fix cheeky comment in nested scopeFelix Mulder2017-04-042-6/+47
|
* More fine-grained distinctions when flags are defined.Martin Odersky2017-04-044-9/+39
| | | | | | | | | | | | | | Flags like Trait are in fact not always defined when a symbol is created. For symbols loaded from class files, this flag, and some other is defined only once the classfile has been loaded. But this happens in general before the symbol is completed. We model this distinction by separating from the `FromStartFlags` set a new set `AfterLoadFlags` and distinguishing between the two sets in `SymDenotations#is`. Test case is enum-Option.scala. This erroneously complained before that `Enum` was not a trait.
* Fix typoMartin Odersky2017-04-041-1/+1
|
* New test caseMartin Odersky2017-04-043-0/+44
|
* Be more systematic about result of apply methodMartin Odersky2017-04-041-8/+18
| | | | | | The same type needs to be used as a result type for the copy method, and the function type implemented by the companion module.
* Link generic test to actual implementations of Enum and EnumValuesMartin Odersky2017-04-043-26/+2
| | | | These are now implemented in scala.runtime.
* Another testMartin Odersky2017-04-042-0/+11
|
* Make `getDocComment` referentially transparentFelix Mulder2017-04-041-36/+12
|
* Fix "closest" computation for docstringsMartin Odersky2017-04-041-1/+3
|
* Improvement to REPL testMartin Odersky2017-04-041-1/+4
| | | | In case of difference, dump transcript to file for easier comparisons.
* Comment out unused method in ContextMartin Odersky2017-04-041-2/+3
|
* Add testsMartin Odersky2017-04-047-0/+47
|
* Implement enum desugaringMartin Odersky2017-04-046-36/+215
|
* Change handling of enum defs.Martin Odersky2017-04-042-15/+34
| | | | | | | The previous scheme did not work because desugaring cannot deal with repeated expansions. We now sidestep the issue by doing the expansion in the parser. Luckily, positions work out perfectly, so that one can reconstruct the source precisely from the parsed untyped trees.
* Adapt generic tests to model modified enum values schemeMartin Odersky2017-04-045-21/+35
|
* Allow value expansion of modules in mergeCompanionDefsMartin Odersky2017-04-041-1/+0
|
* Simplify syntaxMartin Odersky2017-04-046-17/+17
| | | | `enum' only allowed as a prefix of classes, dropped from traits and objects.
* Don't pass docstring as a parameter.Martin Odersky2017-04-041-39/+33
| | | | | It's completely redundant, docstring is just the comment found at the `start` offset, which is passed anyway.
* Add enum syntaxMartin Odersky2017-04-044-29/+115
| | | | Modify syntax.md and Tokens/Parser/untpd to support enums.
* Fix mal-formatting.Martin Odersky2017-04-042-3/+5
| | | | Insert an empty line before "where" in an explanation.
* Merge pull request #2162 from abeln/infix-opDmitry Petrashko2017-04-042-3/+8
|\ | | | | Fix #1959: infix type operators in the REPL
| * Fix #1959: infix type operators in the REPLAbel Nieto2017-04-022-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Infix type operators were broken in the REPL. The REPL uses fold methods from the untpd package, but those were skipping the operator subtree when folding over an InfixOp. Fix the issue by taking the operator into account. Tested: 1) Verified that only the REPL code uses the modified fold method. 2) Added repl test.
* | Merge pull request #2149 from dotty-staging/fix/false-companion-2Dmitry Petrashko2017-04-041-5/+16
|\ \ | | | | | | Create dummy companions for classes without a real one
| * | Create dummy companions for classes without a real oneGuillaume Martres2017-03-291-5/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In #2139 I added code to create dummy companions for companion-less objects, but not for companion-less classes because I thought it wasn't needed. But it turns out that even if the classpath only has `Foo.class` and not `Foo$.class`, a module for `Foo` is entered by `initializeFromClassPath` when it calls `enterClassAndModule`, so we have to add dummy companions to classes. I don't have a test to illustrate this issue, but note that it fixes the incremental compilation bug demonstrated by https://github.com/dotty-staging/dotty/commits/incremental-compilation-bug. Note: I verified that adding a dummy companion to "class Foo" did not cause the backend to emit a Foo$.class, so this should have no visible impact.
* | | Merge pull request #2167 from dotty-staging/fix-copyDmitry Petrashko2017-04-041-19/+19
|\ \ \ | | | | | | | | fix copy of flags for ModuleDef and refactor code
| * | | fix copy of flags for ModuleDef and refactor codeliu fengyun2017-04-031-19/+19
|/ / /
* | | Merge pull request #2159 from dotty-staging/fix-hashcodeDmitry Petrashko2017-04-032-6/+7
|\ \ \ | | | | | | | | Make case class hashCode take class into account
| * \ \ Merge branch 'master' into fix-hashcodeFelix Mulder2017-04-0218-93/+298
| |\ \ \
| * | | | Use full name of case classes for hashingMartin Odersky2017-03-311-2/+3
| | | | |
| * | | | Optimize hashCode computationMartin Odersky2017-03-312-6/+6
| | | | | | | | | | | | | | | | | | | | Also, update check file.
| * | | | Make case class hashCode take class into accountMartin Odersky2017-03-301-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, elements of the following classes had the same hash code: case class A() case class B() Now they are distinguished.
* | | | | Merge pull request #2161 from dotty-staging/si4400Dmitry Petrashko2017-04-031-0/+35
|\ \ \ \ \ | | | | | | | | | | | | Test that SI-4400 stays fixed.
| * \ \ \ \ Merge branch 'master' into si4400Felix Mulder2017-04-0216-82/+254
| |\ \ \ \ \ | | | |/ / / | | |/| | |
| * | | | | Test that SI-4400 stays fixed.Dmitry Petrashko2017-03-311-0/+35
| | | | | |
* | | | | | Merge pull request #2157 from dotty-staging/patmat-dead-codeFelix Mulder2017-04-021-109/+3
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Remove dead code in pattern matching
| * | | | | Remove unreachable `tupleExtractor` methodOlivier Blanvillain2017-03-301-22/+1
| | | | | |
| * | | | | Remove dead code from pattern matcherOlivier Blanvillain2017-03-301-87/+2
| | |/ / / | |/| | | | | | | | | | | | | Starting from unreachable case https://github.com/lampepfl/dotty/blob/f75caad00256db52bbd3310f245e51d23c2a76cc/compiler/src/dotty/tools/dotc/transform/PatternMatcher.scala#L1408
* | | | | Merge pull request #2154 from dotty-staging/topic/fix-#2151Felix Mulder2017-03-319-61/+119
|\ \ \ \ \ | | | | | | | | | | | | Fix #2151: don't die for wrong number of typeargs applied
| * | | | | Fix deadlock in `runAll`Felix Mulder2017-03-313-49/+51
| | | | | |
| * | | | | Improve summary report by dumping all to stdout on CIFelix Mulder2017-03-315-8/+41
| | | | | |
| * | | | | Make DiffUtil's rendering readable in logsFelix Mulder2017-03-313-3/+20
| | | | | |
| * | | | | Fix #2151: don't die for wrong number of typeargs appliedFelix Mulder2017-03-312-1/+7
| |/ / / /
* | | | | Merge pull request #2153 from dotty-staging/topic/dottest-usermanFelix Mulder2017-03-315-7/+129
|\ \ \ \ \ | |_|_|_|/ |/| | | | Add documentation for new parallel testing suite
| * | | | Add documentation for new parallel testing suiteFelix Mulder2017-03-305-7/+129
| | |_|/ | |/| |
* | | | Merge pull request #2160 from dotty-staging/make-testing-more-robustFelix Mulder2017-03-311-3/+2
|\ \ \ \ | | | | | | | | | | Make testing more robust
| * | | | Avoid NPE in ParallelTesting on junk filesMartin Odersky2017-03-311-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Emacs often produces temporary files in directories. These used to cause NPEs in the new testing framework. We now fix this by only compiling file names that designate source files.
* | | | | Merge pull request #2158 from dotty-staging/fix-i2165odersky2017-03-314-22/+48
|\| | | | | |_|/ / |/| | | Fix #2156
| * | | Fix ClassfileParserMartin Odersky2017-03-312-11/+4
| | |/ | |/| | | | | | | | | | | | | | | | #2158 has uncovered flaws in the classfile parser. Matches that used to always miss led to code that made no sense. The function naming was terrible too, that's why nobody understood what was going on. `findSourceFile` to find the class file, seriously?