| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
| |
Disallowed super.XX calls to Any methods which are presently either
crashing the compiler (isInstanceOf) or leading to runtime failure (==
and !=) in addition to one which was being rewritten to a this call but
makes more sense disallowed like the others (##).
Closes #3736, review by odersky.
|
|
|
|
|
|
| |
fixes names/defaults when using :_* for specifying repeated parameters.
close #3697, no review.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
scala> def ser(o: AnyRef) = new java.io.ObjectOutputStream(new java.io.ByteArrayOutputStream()).writeObject(o)
ser: (o: AnyRef)Unit
scala> @serializable class Outer {
| case class Inner(x: Int)
| }
defined class Outer
scala> val o = new Outer
o: Outer = Outer@34469729
scala> ser(new o.Inner(1))
scala> o.Inner // initialize the Inner$module field of o
res1: o.Inner.type = Inner
scala> ser(new o.Inner(1))
java.io.NotSerializableException: Outer$Inner$
review by extempore.
|
|
|
|
|
|
| |
added @deprecatedName annotation, allowing to deprecate parameter names.
review by prokopec.
|
|
|
|
|
| |
Fixes priority queues and makes them iterables now.
|
| |
|
| |
|
|
|
|
|
|
| |
Moved the burden of forgivingness for string slices into StringOps where
it belongs. Review by odersky.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
Closes #1766 (structural method dispatch broken in class with multiple
constructors). Review by prokopec.
|
|
|
|
|
|
|
| |
Made a null output sink and applied it to the recently failing test,
which is displaying non-deterministic output by way of the underlying
parser. No review.
|
| |
|
|
|
|
|
|
| |
Fixed an infinite loop in the xml parser on invalid input. Also found an
off by one bug in Source while fixing it. No review.
|
|
|
|
|
|
|
|
|
|
|
|
| |
correctly simply by inspecting method signatures (because a Value from
a different Enumeration stored in a val looks identical to one from
this Enumeration) so I have Value store the outer Enum for comparison
purposes. This won't make anything new uncollectable because they
already have an $outer pointing there. This also simplified the
reflection logic: it's an eq test rather than a series of heuristics.
Closes #3616, #3615. Review by phaller.
|
| |
|
|
|
|
|
|
|
|
| |
A wholesale reversion of the pattern matcher to r21939, motivated by
the appearance of #3578. Closes #3578, reopens #2800, #3050. Review by
moors. (I can't keep saying "no review" when every move I make in here
seems to break something.)
|
| |
|
|
|
|
|
|
| |
removed integration of placeholder syntax and named arguments. review by
odersky
|
|
|
|
|
|
| |
temporarily reversing r22260; will be shortly re-committed in two
separate portions.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
is now guaranteed to be initialized at most once, even when accessed
from different threads. Closes #3007, review by odersky.
|
|
|
|
|
|
|
|
| |
Most of the iterate implementations were calling the given function one
too many times, leading to tragic failure if the function could not
handle this (such as repeatedly applying tail.) Closes #3540, review by
prokopec.
|
|
|
|
|
|
|
|
| |
but in the interest of not breaking backwards compatibility, the
JSON.parse method has been marked deprecated for now.
Unit tests have been fixed so that this won't break the build this time.
|
|
|
|
|
|
|
| |
ran across some tortured logic trying to accomodate the long abandoned
idea of having 5 != 5L, so simplified the contains method. Closes #3518,
no review.
|
|
|
|
|
|
|
| |
Burned by a last minute adjustment, I lost the downward counting
direction. It is a seriously fiddly process to adjust Range and I don't
recommend it to anyone. Closes #3529 over again. Review by prokopec.
|
|
|
|
|
|
|
|
|
|
|
|
| |
scala> (1 to 1 drop 1) == (1 to 1)
res0: Boolean = true
It was introduced in r21349 which was to fix #2535, but led to #3529.
I humbly suggest we can't afford to introduce bugs of this severity in
the pursuit of corner cases such as Ranges which use Int.MaxValue as a
boundary. And looking at the patch I find the "after" code a lot harder
to follow. Closes #3529. Review by prokopec.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
This commit reimplements filter for Streams, but does not reimplement
map in StreamWithFilter.
The problem is that GC can't collect instances of Streams residing on the stack if there are multiple references to the Stream (more than a single one on the stack on which a Stream method is invoked). In the case of a StreamWithFilter, being an inner class, there is always an `$outer` reference to the outer object, so there is little GC can do. Possible solution - change the return type of WithFilter to something else (in TraversableLike) to allow it to return objects that don't have to subclass TraversableLike.WithFilter, and reimplement the withFilter method in Stream to simply call `filter` method - in the case of Streams, `withFilter` has little sense in either case...
|
| |
|
| |
|
|
|
|
|
| |
according to my tests, they all currently work.
|
|
|
|
|
|
| |
Adds comments for #3471, and a test case which should have been added
earlier. Review by extempore.
|
| |
|
|
|
|
|
|
| |
Fixed an xml issue arising from arrays no longer being recognized as
sequences. Review by dpp.
|
|
|
|
|
|
| |
I figured it'd be a miracle if I got the script test past our fickle
robot overseers. Disabled, no review.
|
|
|
|
|
|
|
|
| |
Changed the script runner mechanism to alchemize from AST atoms rather
than generating wrapper source, and fixed script position reporting.
This patch does not include a discussed change to mark some positions as
synthetic. Closes #3119, #3121. Review by milessabin.
|
|
|
|
|
|
| |
Misoptimization for list extractors caused non-sequences to incorrectly
match. Closes #3050, #2800. No review.
|
| |
|
|
|
|
|
| |
Fixed a BigDecimal/Long comparison bug reported on the list. No review.
|
| |
|