aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Another test fixedMartin Odersky2017-04-051-1/+1
|
* Update test and add check fileMartin Odersky2017-04-052-1/+9
|
* Implementation of proposal changesMartin Odersky2017-04-055-50/+107
| | | | | - rename utility methods - generate utility methods also for object cases
* Add check fileMartin Odersky2017-04-051-0/+1
|
* New and updated testsMartin Odersky2017-04-044-3/+37
|
* Infer enum type args from type parameter boundsMartin Odersky2017-04-042-12/+57
| | | | | | Infer type arguments for enum paraments from corresponding type parameter bounds. This only works if the type parameter in question is variant and its bound is ground.
* Fix rebase breakageMartin Odersky2017-04-041-1/+1
|
* Avoid assertion failure on neg testMartin Odersky2017-04-041-1/+8
| | | | This commit can hopefully be reverted once #2121 is in.
* Fix neg test error countMartin Odersky2017-04-041-1/+1
| | | | | Previous expansion caused 3 spurious follow-on errors which are now avoided.
* Support cases with type parameters that extend a non-parameterized baseMartin Odersky2017-04-043-16/+46
| | | | | | | | Support cases with type parameters that implicitly extend a non-parameterized base without needing their own extends clause. The proposal has been updated to make clear that this is supported. Also address other reviewers comments.
* Support comma-separated enum constantsMartin Odersky2017-04-045-30/+48
|
* Change enumeration members.Martin Odersky2017-04-045-26/+32
| | | | | | | | | | | | | Based on the discussion in #1970, enumeration objects now have three public members: - valueOf: Map[Int, E] - withName: Map[String, E] - values: Iterable[E] Also, the variance of case type parameters is now the same as in the corresponding type parameter of the enum class.
* Change return type of `apply`.Martin Odersky2017-04-042-8/+11
| | | | | | | | | | | In an enum case like case C() extends P1 with ... with Pn ... apply now returns `P1 & ... & Pn`, where before it was just P1. Also, add to Option test.
* Don't change the return type of the `copy` methodMartin Odersky2017-04-041-1/+1
| | | | | `copy` should always return the type of it's rhs. The discussion of #1970 concluded that no special treatment for enums is needed.
* 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
| | | | | |