summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #2374 from adriaanm/ticket-6815Jason Zaugg2013-05-1719-77/+224
|\ | | | | SI-6815 untangle isStable and hasVolatileType
| * remove unused methods: checkStable, isStableExceptVolatileAdriaan Moors2013-05-161-23/+0
| |
| * SI-6815 untangle isStable and hasVolatileTypeAdriaan Moors2013-05-1617-54/+215
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `Symbol::isStable` is now independent of `Symbol::hasVolatileType`, so that we can allow stable identifiers that are volatile in ident patterns. This split is validated by SI-6815 and the old logic in RefChecks, which seems to assume this independence, and thus I don't think ever worked: ``` if (member.isStable && !otherTp.isVolatile) { if (memberTp.isVolatile) overrideError("has a volatile type; cannot override a member with non-volatile type") ``` Introduces `admitsTypeSelection` and `isStableIdentifierPattern` in treeInfo, and uses them instead of duplicating that logic all over the place. Since volatility only matters in the context of type application, `isStableIdentifierPattern` is used to check patterns (resulting in `==` checks) and imports.
| * make treeInfo more globally visibleAdriaan Moors2013-05-163-0/+9
| |
* | Merge pull request #2519 from som-snytt/issue/regex-revertAdriaan Moors2013-05-162-5/+18
|\ \ | | | | | | SI-6406 Restore deprecated API
| * | SI-6406 Restore deprecated APISom Snytt2013-05-122-5/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original patch for SI-6406 was intended for 2.10 but during those volatile weeks of early autumn, it missed the boat. A deprecated method was incorrectly tagged at 2.10 and later removed; this restores the method and its test, and resets the deprecation clock to 2.11. The deprecation tool should confirm that changes occur on the git timeline as claimed.
* | | Merge pull request #2534 from retronym/ticket/3943Adriaan Moors2013-05-162-0/+21
|\ \ \ | | | | | | | | SI-3943 Test case for already-fixed Java interop bug
| * | | SI-3943 Test case for already-fixed Java interop bugJason Zaugg2013-05-152-0/+21
| | | | | | | | | | | | | | | | Confirmed that this was fixed in edee27f59.
* | | | Merge pull request #2543 from soc/SI-7469-loggingAdriaan Moors2013-05-169-94/+19
|\ \ \ \ | |_|_|/ |/| | | SI-7469 Remove @deprecated scala.util.logging
| * | | SI-7469 Remove @deprecated scala.util.loggingSimon Ochsenreither2013-05-169-94/+19
|/ / /
* | | Merge pull request #2527 from soc/SI-7476Jason Zaugg2013-05-161-4/+50
|\ \ \ | | | | | | | | SI-7476 Add documentation to GenericTraversableTemplate
| * | | SI-7476 Add documentation to GenericTraversableTemplateSimon Ochsenreither2013-05-131-4/+50
| | |/ | |/|
* | | Merge pull request #2526 from dcsobral/doc/patMatAdriaan Moors2013-05-151-2/+6
|\ \ \ | | | | | | | | Fix formatting for couple of scaladocs in the compiler
| * | | Fix formatting for couple of docs in the compilerDaniel C. Sobral2013-05-131-2/+6
| |/ / | | | | | | | | | | | | | | | | | | Source code on scaladoc for the virtual pattern matcher was not being correctly identified as such. Also, I had to use an html code tag instead of backticks when referring to the __match identifier, otherwise the double underscore would be treated as start-of-underline wiki markup.
* | | Merge pull request #2520 from soc/SI-7469-concurrentJason Zaugg2013-05-155-6/+6
|\ \ \ | | | | | | | | SI-7469 Make @deprecated elems in scala.concurrent private[scala]
| * | | SI-7469 Make @deprecated elems in scala.concurrent private[scala]Simon Ochsenreither2013-05-125-6/+6
| |/ / | | | | | | | | | They can't be removed yet because scala.actors depends on it.
* | | Merge pull request #2521 from soc/SI-7469-combinatorJason Zaugg2013-05-153-79/+0
|\ \ \ | | | | | | | | SI-7469 Remove deprecated elements in s.u.parsing.combinator
| * | | SI-7469 Remove deprecated elements in s.u.parsing.combinatorSimon Ochsenreither2013-05-123-79/+0
| |/ /
* | | Merge pull request #2523 from soc/SI-7469-hashingJason Zaugg2013-05-152-206/+0
|\ \ \ | | | | | | | | SI-7469 Remove @deprecated MurmurHash elements
| * | | SI-7469 Remove @deprecated MurmurHash elementsSimon Ochsenreither2013-05-122-206/+0
| |/ /
* | | Merge pull request #2514 from scalamacros/topic/5923-masterJason Zaugg2013-05-1528-64/+228
|\ \ \ | | | | | | | | makes sense of implicit macros! now in master
| * | | removes duplication in inferImplicitValueEugene Burmako2013-05-123-30/+18
| | | | | | | | | | | | | | | | This is a port of https://github.com/scala/scala/commit/4e64a2731d from 2.10.x.
| * | | SI-7047 fixes silent for c.inferImplicitXXXEugene Burmako2013-05-125-9/+32
| | | | | | | | | | | | | | | | This is a port of https://github.com/scala/scala/commit/b4da864247 from 2.10.x.
| * | | SI-7167 implicit macros decide what is divergenceEugene Burmako2013-05-1215-25/+117
| | | | | | | | | | | | | | | | | | | | | | | | This is a port of https://github.com/scala/scala/commit/8168f118c9 from 2.10.x, with an additional change to the `enclosingImplicits` and `openImplicits` APIs, which encapsulates tuples of `pt` and `tree` into `ImplicitCandidate`.
| * | | macroExpandAll is now triggered in all invocations of typedEugene Burmako2013-05-122-0/+4
| | | | | | | | | | | | | | | | This is a port of https://github.com/scala/scala/commit/bb73b9669a from 2.10.x.
| * | | SI-5923 instantiates targs in deferred macro applicationsEugene Burmako2013-05-127-10/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In January I submitted a pull request that, as I thought back then, fixes SI-5923: https://github.com/scala/scala/commit/fe60284769. The pull request was merged, and everyone was happy that the bug got fixed. Unfortunately, the fix was: a) incomplete, b) broke something else, as noticed by Miles in https://groups.google.com/d/topic/scala-internals/7pA9CiiD3u8/discussion. Now we got a real fix in 2.10.x (https://github.com/scala/scala/commit/90ac5c4e13), and it's my pleasure to port it to master.
* | | | Merge pull request #2503 from paulp/issue/6039Jason Zaugg2013-05-153-2/+26
|\ \ \ \ | | | | | | | | | | SI-6039 Harden against irrelevant filesystem details
| * | | | SI-6039 Harden against irrelevant filesystem detailsPaul Phillips2013-05-083-2/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The symbol loader need not create and populate package symbols merely because there is a directory somewhere. Every package created based on the existence of a directory should contain a classfile, either directly or indirectly.
* | | | | Merge pull request #2498 from rladstaetter/patch-1Jason Zaugg2013-05-151-1/+1
|\ \ \ \ \ | | | | | | | | | | | | fix typo in comment
| * | | | | fix typo in commentRobert Ladstätter2013-05-061-1/+1
| | | | | |
* | | | | | Merge pull request #2399 from rjolly/scripting8Jason Zaugg2013-05-151-1/+1
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | AbstractFile.getDirectory does not return null when outDir is "."
| * | | | | AbstractFile.getDirectory does not return null when outDir is "."Raphael Jolly2013-04-221-1/+1
| | | | | |
* | | | | | Merge pull request #2525 from soc/SI-7469-astPaul Phillips2013-05-132-381/+0
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | SI-7469 Remove @deprecated scala.util.parsing.ast
| * | | | | SI-7469 Remove @deprecated scala.util.parsing.astSimon Ochsenreither2013-05-132-381/+0
|/ / / / /
* | | | | Merge pull request #2501 from paulp/pr/tailcalls-tostringPaul Phillips2013-05-122-7/+7
|\ \ \ \ \ | |_|_|_|/ |/| | | | Limit unnecessary calls to Type#toString.
| * | | | Limit unnecessary calls to Type#toString.Paul Phillips2013-05-082-7/+7
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Logging revealed a few thousand calls to the often expensive Type#toString emerging from tailcalls. The error message was being generated for all methods even though it was only issued in rare cases (and for the particular tailrec failure which made the call, extremely rare.) The remaining boatload of unnecessary Type#toString calls are much harder to fix due to the design of "AbsTypeError" and the fact that the compiler approaches mutability like a cat approaches a loaded gun. See SI-6149.
* | | | Merge pull request #2470 from ihji/masterPaul Phillips2013-05-112-2/+13
|\ \ \ \ | | | | | | | | | | SI-7432 Range.min should throw NoSuchElementException on empty range
| * | | | SI-7432 add testcasesHeejong Lee2013-04-301-0/+11
| | | | |
| * | | | SI-7432 Range.min should throw NoSuchElementException on empty rangeHeejong Lee2013-04-301-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | For consistency, range.max and range.min should throw NoSuchElementException on an empty range.
* | | | | Merge pull request #2512 from som-snytt/issue/partest-exec-logPaul Phillips2013-05-111-4/+5
|\ \ \ \ \ | | | | | | | | | | | | Par-Test includes log in transcript of failed exec
| * | | | | Par-Test includes log in transcript of failed execSom Snytt2013-05-101-4/+5
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous behavior was that the Failure is generated before the log is appended to the transcript. That meant that the summary transcripts wouldn't include the log file. Luckily, the transcript would say something like "jvm > showFail-run.log".
* | | | | Merge pull request #2425 from paulp/pr/no-package-objectsPaul Phillips2013-05-102-116/+123
|\ \ \ \ \ | | | | | | | | | | | | Boil out some duplicated parser logic.
| * | | | | Boil out some duplicated parser logic.Paul Phillips2013-05-101-116/+108
| | | | | | | | | | | | | | | | | | | | | | | | Our focus today is on packages and package objects.
| * | | | | Flesh out copyMemberDef methods with copyModuleDef.Paul Phillips2013-05-101-0/+15
|/ / / / /
* | | | | Merge pull request #2462 from magarciaEPFL/backendish9Paul Phillips2013-05-1021-50/+86
|\ \ \ \ \ | | | | | | | | | | | | SI-6863 root cause fixed using factory of scala.runtime.*Ref
| * | | | | indentation typoMiguel Garcia2013-04-301-6/+6
| | | | | |
| * | | | | SI-6863 root cause fixed using factory of scala.runtime.*RefMiguel Garcia2013-04-2721-49/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit does away with an error-prone division of labor between UnCurry and LambdaLift, a division of labor by which UnCurry had to anticipate under which circumstances LambdaLift creates a scala.runtime.*Ref whose initial value is given by a an expression including a Try in non-statement position. That sounds complicated, and it is. The solution so far (fixing SI-6863) is replaced by a simpler approach, at the cost of forward binary comptability with pre-2.11 releases, this time fixing the root cause of SI-6863. From now on, a s.r.*Ref is instantiated via invocation of a static factory method in the s.r.*Ref class in question. Unlike the code that was emitted so far (which involved NEW refclass, DUP, expr, INVOKESPECIAL refclass.<init>) the "expr" doesn't appear on the operand stack on top of the *Ref value being initialized. In other words, the *Ref initialization is in statement position provided "expr" is.
* | | | | | Merge pull request #2489 from som-snytt/issue/partest-show-diffPaul Phillips2013-05-102-1/+13
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Partest can --show-diff again after incremental report.
| * | | | | Partest can --show-diff again after incremental report.Som Snytt2013-05-032-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A flag is set in NestUI to look for "diff" in the transcript of failing tests.
* | | | | | Merge pull request #2493 from paulp/pr/null-hardeningPaul Phillips2013-05-077-8/+16
|\ \ \ \ \ \ | | | | | | | | | | | | | | Hardening against nulls for deserialization.
generated by cgit v1.2.3 (git 2.39.1) at 2024-08-25 08:36:43 +0000