| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
| |
The take/drop methods in IndexedSeqOptimized were overridden to call
slice without checking bounds, causing it to fail when the argument is
too large. Restored expected behavior. No review.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
When compilation fails because of an unimplemented abstract var, give
a more precise error message about what happened. Also avoid issuing
the same error twice because neither getter nor setter is implemented.
Closes #36, review by rytz.
|
| |
|
|
|
|
|
|
|
| |
Abandoned any pretense of selectivity and put forkjoin.jar on every
compilation path from locker to strap. Review postponed until the
afterlife.
|
|
|
|
|
|
|
|
|
|
|
| |
Took a cue from mharrah that we don't need to build global static data
to keep track of something when we know where it's kept. Altered the
Enumeration deserialization scheme to use reflection, preserving the
singleton property by delivering the MODULE$ singleton. This solves
the GC issue and lets us drop synchronization to boot. Also added some
graceful failure for malformed Enumerations. All tests look good but a
second opinion is in order: closes #2214, review by phaller.
|
|
|
|
|
|
|
|
|
|
| |
Renaming files and inserting dummies to please ant. Note to committers:
whenever there is a source file which does not generate a classfile
whose name and path exactly match the name and path of the source file,
then ant will recompile the file every time it compiles anything. In
this batch there were 7 such files, so any 1-char change meant an 8-file
recompile. Someday we'll be rid of ant, but until then... 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.
|
|
|
|
|
|
|
| |
Created mutable.SeqLike so as to mix in Cloneable like
mutable.{Set, Map} do. Closes #3590, review by odersky.
|
|
|
|
|
|
| |
Removing some dead code from SyncVar and cleaning up a little. Closes
#3490, no review.
|
|
|
|
|
|
| |
Separate option Ordering into a trait so it can be reused. Closes #3539,
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.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
Disposed of a gordian knot by transforming stringbuilder into a straight
wrapper of java's. No review.
|
|
|
|
|
|
| |
Fixed problem with `remove` in JavaConversions which affected Java set
and map wrappers and which was noted on the scala list.
|
|
|
|
|
|
|
| |
sense that there are a few code paths where null may be checked
redundantly, but I deemed further optimization too risky. Review by
odersky.
|
|
|
|
|
|
| |
Moved parallel collections to library dir, changed sabbus script. Added
`par` to some of the classes. No review.
|
| |
|
| |
|
|
|
|
|
|
| |
temporarily reversing r22260; will be shortly re-committed in two
separate portions.
|
| |
|
| |
|
|
|
|
|
| |
No review.
|
| |
|
| |
|
|
|
|
|
|
| |
Further improved combine for hash tries, cutting of another 30ms (160
downto 130). Review by rompf.
|
|
|
|
|
| |
Continued working on hash trie map combine - work in progress. No review yet.
|
| |
|
|
|
|
|
|
| |
Under test/benchmarks there is a `bench` script to run benchmarks - it can be invoked after running building the library.
Review by rompf.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
| |
Fixed problem discovered by Paul that views do not support a filter in
for expressions. review by extempore.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
| |
Sorted out some buck passing among TraversableOnce and its subclasses
about how exactly one creates a Stream. This is a continuation of r22115
for #3516. Review by prokopec.
|
|
|
|
|
|
| |
Codec changes in scala.tools.nsc.io corresponding to those made in
r22154. No review.
|
|
|
|
|
|
|
|
|
| |
Restored Source factories to a form source compatible with 2.7.7. No
default implicit arguments, now low priority saves the day with a low
priority codec which io.Codec offers as last resort. Dropped the line
separator argument to getLines and made it act in a separator agnostic
way (any of \r\n, \r, or \n is a separator.) Review by community.
|
| |
|
| |
|
| |
|