| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
Fix for SI-6340, error message regression.
|
| | |
|
|\ \
| | |
| | | |
SI-6359 Deep prohibition of templates in value class
|
| |/
| |
| |
| | |
This seems to have been the intent of 95d532 / SI-5882.
|
|\ \
| | |
| | | |
Fixes SI-6259. Unable to use typeOf in super call of top-level object.
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| | |
This works around the issue of the inability to use classOf for top-level object classes by inventing a new
anonymous class and instantiating it just to grab its class. Since the class is a nested type of the
top-level object it'll be in the same classloader unless some kind of evil behavior is afoot.
This patch should be undone if ever SI-2453 ever gets fixed, or we wind up with a direct way to grab
the class of an object.
|
|\ \
| | |
| | | |
Fixes binary repo push for new typesafe repo layouts.
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
This adds a new/fixed push.jar and adapts the binary-repo-lib script to use different URLs
for pulling than pushing.
This also adjustst the script to attempt the backup repo in case new artifacts haven't
synched to the fast/stable repository.
|
|\ \
| | |
| | | |
Fix for SI-6367, exponential time in inference.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This pathology is not new - it can be witnessed in 2.9,
where compiling the test case enclosed with this ticket with
-Yinfer-debug will print a line with (pinky to lips) one million
type parameters. 1048576 actually, aka 2^20. But in 2.9 we were
somehow getting away with creating the list, presumably by not
spending much time looking at it. Somewhere between there and M1,
that changed.
I cut it off at the knees - don't create a list of one million
upper bound constraints when 1 will suffice. It would not be too
surprising if this proves to be a boon for performance.
|
|\ \ \
| | | |
| | | | |
210 errors and trees
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Because friends don't tell friends:
"wrong number of arguments for <none>"
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
For too long, Tree Builder has ruled over those who would
like to build their own trees. Today marks the start of a
new era. Trees are for building, not for parsers to hoard.
It's in Global now.
The particular motivation is exposing makeNew, and I also
added makeAnonymousNew, so rather than this...
Apply(Select(
Block(List(ClassDef(
Modifiers(scala.tools.nsc.symtab.Flags.FINAL), tpnme.ANON_CLASS_NAME, Nil,
Template(List(Ident(definitions.AnyRefClass)), emptyValDef, List(
DefDef(NoMods, nme.CONSTRUCTOR, Nil, List(Nil), TypeTree(), Block(
List(Apply(Select(Super(This(tpnme.EMPTY), tpnme.EMPTY), nme.CONSTRUCTOR), Nil)),
Literal(Constant(()))
)))))),
Apply(Select(New(Ident(tpnme.ANON_CLASS_NAME)), nme.CONSTRUCTOR), Nil))
, sn.GetClass), Nil)
We can write this.
Apply(Select(makeAnonymousNew(Nil), nme.getClass_), Nil)
|
| | | | |
|
| | | | |
|
|\ \ \ \
| |/ / /
|/| | | |
Fix SI-6306 on testcase
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix SI-6306 by removing the offending code. Moreover, clarify comment
and add testcase.
This pattern match only matches on testcases triggering SI-6306; the
transformation it performs in that case is unsafe.
The intended optimization is to undo eta-expansion of nullary functions,
that is, transform `() => foo()` to `foo`. But that's only valid when
`foo` is an instance of `Function0`, so the optimization is unsafe.
Moreover, the pattern match will fail because at the end of typer that
code has become `() => foo.apply()`, and `isExprSafeToInline(foo.apply)`
always (correctly) fails the isExprSafeToInline test. The pattern match
should thus be different - this code was dead even when it was
introduced (45bcd02f6ba099277bedbf83ec2bda07435c7797), since it was not
invoked either when building the compiler or when compiling function
attempt2() in the included testcase.
Thanks to all commenters on SI-6306 and
https://github.com/scala/scala/pull/1255, in particular to Jason Zaugg
for diagnosing the underlying fault and Lukas Rytz for understanding the
goal of the code.
|
|\ \ \ \
| | | | |
| | | | | |
Fix t6114 - ++ on JList wrapper modifies underlying collection.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We solve this by overriding clone for JListWrapper to actually do a full clone.
Note: This fix may need to be included other places, *but* we're not sure we've cloned the collection sensibly. I.e. is ArrayList a good default?
|
|\ \ \ \ \
| |_|_|/ /
|/| | | | |
several fixes to scala.concurrent.util.Duration
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
- also use type FiniteDuration due to a previous change to
Deadline’s type signature
|
| |\ \ \ \ |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- copy partest TestUtil.intercept change from PR 1279 branch
- add comment on non-obvious match cases
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- make constants for fromNanos(Long) actually inlined constants
- clarify origin of require() check constants in FiniteDuration
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- Inf / Zero == Inf
- add some more missing test cases
- clarify magic constant
- move exception descriptions into proper @throws docs
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- make Duration behave consistent with Double's non-finite semantics
- add ScalaDoc
- add complete test suite
- change overflow protection impl after review comments
- clean up code
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- add test cases (migrated from Akka sources)
- add overflow checking (will throw IllegalArgumentException instead of
giving wrong results)
- make string parsing more precise when giving >100days in nanoseconds
- make method signatures more precise in retaining FiniteDuration
throughout calculations
- fix mul/div of infinities by negative number
- add Ordering for Deadline (was accidentally left out earlier)
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | | |
Fixes SI-6268. Improper multistring setting unparse.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This reverts a refactoring from https://github.com/scala/scala/commit/963aabbeb4
MultiString settings would not properly write an unparse string that could be
reparsed, leading to failures when forking scalac in ant. Specifically, if a
setting was empty, it was getting added to the unparse string and causing
scalac to fail. This at least reverts to previous behavior (also more correct
for multiple values). Whatever we do here has to work with the @file style
argument reading and can't place empty options on the command line.
Also, This assumes there are tests around the behvior the REPL needs.
Note: The test is not automated yet, so unfortunately, we need to improve ANT testing capabilities to prevent regressons
|
|\ \ \ \ \ \
| |_|_|_|_|/
|/| | | | | |
Avoid spurious warning for `def foo = x.foo`.
|
|/ / / / /
| | | | |
| | | | |
| | | | | |
Followup to SI-6276.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
fix usage of Duration in Promise impl
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- scaladoc the exceptions thrown by Await.* and Awaitable.*
- move intercept[Exception] into partest’s TestUtil object
- improve Promise.tryAwait implementation following Viktor’s comments
and make use of Deadline to avoid calling System.nanoTime too often
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- correctly treat MinusInf and Undefined
- don't toMillis in the timeout message (could be MinusInf)
- also notice that Inf did not actually wait unbounded
- and further notice that tryAwait swallows InterruptedException instead
of bailing out early => changed to do so and added throws annotation
- also removed some unused imports of Duration
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Topic/inliner logging
|
| |\ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Conflicts:
src/compiler/scala/tools/nsc/backend/icode/GenICode.scala
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Quieted down many logging statements which contribute
disproportionate noise. Made others emit something more sensible.
Spent lots of time on the inliner trying to find a regular
format to make the logs more readable. Long way to go here but
it'd be so worth it to have readable logs instead of mind-numbing
indiscriminate text dumps.
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
While hopefully still hearing about certain things (like symbol renames)
when it's not a specialized symbol.
|
| | | | | | | | |
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
But it's commented out because I want to get this logging
code into 2.10 without including anything "interesting".
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | | |
Disable test t2868
|
| | |/ / / / / /
| |/| | | | | |
| | | | | | | |
| | | | | | | | |
it keeps failing randomly, e.g. http://scala-webapps.epfl.ch/artifacts/e3b0c7abbf637dacce7bcd7b69d5655820e8e714/buildLog.txt
|
|\ \ \ \ \ \ \ \
| |/ / / / / / /
|/| | | | | | | |
Fixes SI-6271 - Missing isEmpty override for views.
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
GenIterableView didn't override isEmpty for views to look at *filtered* iterator, but was instead pulling
unfiltered iterator and causing issues. Chalk up another bizzare bug to lack of insight/visibility into
linearization and what havoc overriding new methods can spew on our library.
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | | |
SI-6335 More precise location of the implicit class synthetic method.
|
| | | | | | | | | |
|