| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
These were already working, due to a serendipitous
ordering of transformations. The tests will keep
it this way.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bridge building operates on unusual method signatures:
after uncurry, so parameter lists are collapsed; but before
erasure, so dependently typed parameters are still around.
Original:
def foo(a: T)(b: a.type, c: a.U): Unit
During computeBridges:
(a: T, b: a.type, c: a.U)Unit
This signature no longer appears to override the corresponding
one in a superclass, because the types of `b` and `c` are dependent
on method parameters.
The root of the problem is uncurry, which leaves the trees in
a poor state. This commit changes uncurry to remedy this.
An example illustrates it best:
// source
def foo(a: A)(b: a.type): b.type = b
// post uncurry before this patch.
// not well typed code!
def foo(a: A, b: a.type): a.type = {
// post uncurry after this patch
def foo(a: A, b: A): A = {
val b$1 = b.asInstanceOf[a.type]
b$1
}
|
|\
| |
| | |
Backport 1821
|
| |
| |
| |
| |
| |
| |
| |
| | |
the manifests for Any, Object/AnyRef, AnyVal, Null and Nothing
now have their `newArray` methods overridden to avoid reflective overhead
of array instantiation.
(backport of 45ef0514e, part 2)
|
| |
| |
| |
| | |
(backport of 45ef0514e, part 1)
|
|\ \
| | |
| | | |
SI-6897, lubs and varargs star.
|
| | |
| | |
| | |
| | |
| | | |
Don't allow lubs to calculate refinement types which contain
a varargs star outside of legal varargs star position.
|
|\ \ \
| | | |
| | | | |
SI-6896, spurious warning with overloaded main.
|
| |/ /
| | |
| | |
| | |
| | | |
Make sure there's no legit main signature before issuing
any warnings about missing main methods.
|
|\ \ \
| | | |
| | | | |
SI-6911, regression in generated case class equality.
|
| | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Caught out by the different semantics of isInstanceOf and
pattern matching.
trait K { case class CC(name: String) }
object Foo extends K
object Bar extends K
Foo.CC("a") == Bar.CC("a")
That expression is supposed to be false, and with this
commit it is once again.
|
|\ \ \
| | | |
| | | | |
Fix Iterator#copyToArray (fixes SI-6827).
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
As pointed out in #scala, when using a non-zero start it's possible
to get an ArrayIndexOutOfBoundsException due to an incorrect bounds
check. This patch fixes this, as well as another potential bounds
error, and adds test cases.
Incorporates some other suggestions by Som-Snytt to ensure that
callers will get useful error messages in cases where the start
parameter is wrong (negative or out-of-array-bounds).
Review by @som-snytt.
|
|\ \ \
| |/ /
|/| | |
SI-5017 Poor performance of :+ operator on Arrays
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Control performance of :+ and +: operator on my machine were 700-800 ms
After adding size hint on the implementation in SeqLike, it went down to 500-600 ms
But with specialixed implementation in ArrayOps, brings it down to 300-400 ms
Unfortunatly, this method will only be called when the Array object is being referenced directly as it's type, but that should be the case enough times to justify the extra method.
I ended up removing the sizeHint in SeqLike because it made the execution of the "benchmark" slower when the Array was being manipulated as a Seq.
Side note: Interestingly enough, the benchmark performed better on my virtualized Fedora 17 with JDK 7 than natively on Mac OS X with JDK 6
|
|\ \ \
| |_|/
|/| | |
SI-6194, repl crash.
|
| | |
| | |
| | |
| | |
| | |
| | | |
Always a bad idea to use replaceAll on unknown strings,
as we saw here when windows classpaths arrived containing
escape-requiring backslashes.
|
|\ \ \
| | | |
| | | | |
Remove -deprecation from partest default options.
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
Who knows why it was ever like this; it's not like anyone
sees the deprecation warnings. In PR #1807 there is now a
test which depends on partest not making this move, so it's
a good time to finally expunge it.
|
|\ \ \
| | | |
| | | | |
SI-6746 Fixes MANIFEST.MF package entry (s.r.makro -> s.r.macros)
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Stream.zip naturalsEx example does not compile => remove extra zip call
|
| | | | | |
|
|\ \ \ \ \
| |_|_|/ /
|/| | | | |
SI-6415, Stream#lengthCompare
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Had to fix up an iffy test: not only was it testing undefined
behavior, it demanded just the right numbers be printed in a
context where all negative or positive numbers are equivalent.
It's the ol' "get them coming and going" trick.
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The lengthCompare method in LinearSeqOptimized was looking one
step further than it needed to in order to give the correct
result, which was creating some unwanted side effects related to
Streams.
|
|\ \ \ \
| | | | |
| | | | | |
SI-6829, NPE during erroneous compilation.
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
Have to intercept trees which have a null type due to errors
before they leave the warm confines of 'def typed' because from
that point everything assumes tree.tpe != null.
|
|\ \ \ \
| | | | |
| | | | | |
Remove stray debugging output line.
|
|/ / / /
| | | |
| | | |
| | | | |
I finally reached my "CHECK THREAD ACCESS" limit.
|
|\ \ \ \
| | | | |
| | | | | |
fixes the unchecked warning in quick.comp
|
| | |/ /
| |/| |
| | | |
| | | |
| | | | |
All those months when I thought it was yet another spurious error
in the new pattern matcher...
|
|\ \ \ \
| | | | |
| | | | | |
scaladoc Template: remove duplicate code and several usages of Option.get.
|
| | | | | |
|
|\ \ \ \ \
| |_|_|/ /
|/| | | | |
Merge 2.10.0-wip into 2.10.x
|
| |\ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
# By Eugene Burmako (1) and others
# Via Adriaan Moors (2) and others
* origin/2.10.0-wip:
Fixing OSGi distribution.
Fix for rangepos crasher.
SI-6685 fixes error handling in typedApply
|
| | |\ \ \ \
| | | | | | |
| | | | | | | |
Fix for rangepos crasher.
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
wrapClassTagUnapply was generating an unpositioned tree
which would crash under -Yrangepos. See SI-6338.
|
| | |\ \ \ \ \
| | | |/ / / /
| | |/| | | | |
Fixing OSGi distribution.
|
| | |/ / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
There was some kind of wierd filesystem issue where ANT would overwrite
jars or not, depending on timestamps. It was a non-repeatable failure.
Rather than overwrite JARs and rely on ANT, let's just spell out the
non-OSGI bundles. While I had hoped to avoid hard-coding these,
it's probably best we've done so.
|
| | |\ \ \ \
| | | | | | |
| | | | | | | |
Ticket/6685
|
| | |/ / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
When MissingClassTagError doesn't lead to an exception, but rather
silently sets an error, we need to bubble the resulting erroneous tree
up the responsibility chain instead of mindlessly typechecking this again.
This wasn't an issue before, because as far as I can guess the
aforementioned error setter was always throwing exceptions in the most
common usage scenarios (therefore the typecheck-again-fail-again
vicious loop wasn't triggered).
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
Implicit vars should have non-implicit setters.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Otherwise they trigger spurious feature warnings.
scala> trait T { implicit var a: Any }
<console>:7: warning: implicit conversion method a_= should
be enabled by making the implicit value
language.implicitConversions visible.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
SI-6795 simplify "abstract override" errors on type members
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Instead of saying "only allowed on non-type members of traits" use
separate errors for "not allowed on types" and "only allowed on members
of traits"
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
"abstract override" shouldn't was being allowed on types in traits but the result
made no sense and the spec says that shouldn't be allowed.
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
Cleanup MemberLookup. Better explain ambiguous link targets.
|
| | |_|_|/ / /
| |/| | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
Remove dead code from `Global`.
|