summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* A few yards short of the goal posts attempt at ...Paul Phillips2010-03-0467-259/+252
| | | | | | | | | | | A few yards short of the goal posts attempt at making our usage of Throwable subclasses more consistent. This patch eliminates a lot of ad hoc Exception/Error/etc. creation and various arbitrary choices are rendered slightly less arbitrary. From now on let's try not to use the word "Exception" or "Error" in the names of Throwable subclasses unless they actually derive (and make sense to derive) from Exception or Error. Review by community.
* Attempt to fix the typing-a-whileloop problem.Martin Odersky2010-03-031-20/+28
|
* Closes #3130. No review necessary.Martin Odersky2010-03-031-0/+3
|
* Added --grep command line option to partest.Paul Phillips2010-03-025-34/+66
| | | | | | | | | test with the string "Manifest" in the source file, you may now do: ./partest --grep Manifest No review.
* Removed the symlinks between scalacheck jars to...Paul Phillips2010-03-026-27/+36
| | | | | | Removed the symlinks between scalacheck jars to satisfy windows. Tweaked partest to accomodate. No review.
* Improved equality for Manifests.Paul Phillips2010-03-022-5/+23
| | | | | | | | | | | | | | | | | | canEquals, and has ClassManifests compare according to erasure but full manifests also compare type arguments. Preserving symmetry means that some things you might expect to be equal are not: val m1 = scala.reflect.ClassManifest.fromClass(classOf[List[String]]) val m2 = manifest[List[String]] (m1 == m2) // false However you can always compare the erasures. (m1.erasure == m2.erasure) // true Review by dpp.
* #3128 fixedilyas2010-03-023-4/+38
|
* Whipped ShowPickled until it would print out pr...Paul Phillips2010-03-012-15/+108
| | | | | | | | | | Whipped ShowPickled until it would print out private[scope] from the signature, and infrastructure created along the way. Only now at this late hour do I realize that this work would be a lot better aimed at creating a fake Universe and then adapting UnPickler.Scan so you can reuse the real logic. My advice to the next guy: do that instead. No review.
* trailing spaces in decompiled annotations are t...ilyas2010-03-012-4/+6
| | | | | trailing spaces in decompiled annotations are trimmed
* scalap tests fixedilyas2010-03-017-12/+11
|
* Undeprecated Function.tupled based on the type ...Paul Phillips2010-03-011-4/+7
| | | | | | | | | Undeprecated Function.tupled based on the type inference issues documented at: http://stackoverflow.com/questions/2354277/function-tupled-and-placehold er-syntax We should revisit if anon function inference improves. Review by community.
* typo in test fixedilyas2010-03-011-1/+1
|
* #2885 fixedilyas2010-03-013-0/+7
|
* Added missing file that broke the build.Martin Odersky2010-03-011-0/+53
|
* #3060 fixedilyas2010-03-014-80/+123
|
* Closes #3076. Review by community.Martin Odersky2010-03-013-2/+8
|
* Following a suggestion of jrudolph, made filter...Martin Odersky2010-03-014-26/+57
| | | | | | | | Following a suggestion of jrudolph, made filterKeys and mapValues transform abstract maps, and duplicated functionality for immutable maps. Moved transform and filterNot from immutable to general maps. Review by phaller.
* closed #3101. Review by community.Martin Odersky2010-03-012-8/+24
|
* Added one previously overlooked case for comput...Martin Odersky2010-03-011-4/+3
| | | | | | Added one previously overlooked case for computing the right tparams of glbs of polytypes. This is a postscript to the fix of #3082.
* Minor printer fix for singleton typesilyas2010-03-011-4/+5
|
* Enabled scalacheck tests.Paul Phillips2010-03-0114-63/+82
| | | | | | | | what must be legacy scalatest.* properties to partest.*, boldly assuming that the fact that partest is pretty much unusable outside of scalac means there are no users outside of scalac who might be disrupted by eliminating old property names. Review by community.
* Fixed specialized pattern matches.Iulian Dragos2010-02-285-2/+93
| | | | | specialized implementations.
* Modification to r21009 to preserve that classic...Paul Phillips2010-02-282-12/+7
| | | | | | Modification to r21009 to preserve that classic invariant, (x || !x) && !(x && !x). No review.
* Added ## method to Any as our scala hashCode me...Paul Phillips2010-02-2811-9/+64
| | | | | | | | | | | | | | Added ## method to Any as our scala hashCode method which provides consistent answers for primitive types. And I'm sure we're all tired of new starrs, but it's hard to add a method to Any without one. This patch only brings ## into existence, but nothing calls it yet. // some true assertions scala> assert(5.5f.## == 5.5f.hashCode) scala> assert(5.0f.## != 5.0f.hashCode && 5.0f.## == 5L.##) No review. (Already reviewed by odersky.)
* While working on Any.## I ran across some inter...Paul Phillips2010-02-282-19/+28
| | | | | | | | | | | | | | | | | While working on Any.## I ran across some interesting tests being made in TreeBuilder: val buf = new ListBuffer[(Name, Tree, Position)] [...] if (buf.iterator forall (name !=)) ... This is always true because a Name will never equal a Tuple3. Oh universal equality, will you never tire of toying with us? Given that this bug has existed since r12886 one might reasonably question the necessity of the conditionals falling prey to this, but leaving that for another day, it should at least check what it's trying to check. No review.
* Expanded the check from #1392 to enclose #3123 ...Paul Phillips2010-02-274-5/+20
| | | | | | Expanded the check from #1392 to enclose #3123 as well so that "case Int => " doesn't crash. Closes #3123. Review by odersky.
* Special cased an error message for the common s...Paul Phillips2010-02-272-8/+21
| | | | | | Special cased an error message for the common situation of calling AnyRef methods on Any or AnyVal. Review by odersky.
* Some library reorganization I discussed with ma...Paul Phillips2010-02-2710-72/+54
| | | | | Some library reorganization I discussed with martin. No review.
* Removed the partest restriction that individual...Paul Phillips2010-02-273-110/+69
| | | | | | | | | | | | | | | | | | | | | Removed the partest restriction that individual files must be in the same set. Haven't you always wanted to do this? Now you can. Review by phaller. % ./partest `ack --type=scala -l HashSet | head -6` Testing individual files testing: [...]/files/jvm/serialization.scala [ OK ] testing: [...]/files/jvm/t1600.scala [ OK ] Testing individual files testing: [...]/files/pos/collections.scala [ OK ] testing: [...]/files/pos/t2183.scala [ OK ] Testing individual files testing: [...]/files/run/bug1074.scala [ OK ] testing: [...]/files/run/bug2512.scala [ OK ]
* Quite a lot more work on XML equality than I ca...Paul Phillips2010-02-2651-497/+434
| | | | | | | | | | | | | | | | | | Quite a lot more work on XML equality than I can properly justify spending time on, but you know how it is once you get started on something. This commit will likely break some code out there in the world but this is impossible to avoid if we are to achieve sane equality in trunk. For anyone who was relying upon the 2.7 equality behavior for scala.xml.* classes, using "xml_==" instead of "==" for comparisons will restore the old behavior. The standard == on xml elements now attempts to behave in such a way that symmetry and hashCode contracts will be preserved. It's probably not 100% there yet, but I can tell you this: it is closer today than it was yesterday. Review by community.
* closes #3082, review by rytzMartin Odersky2010-02-267-17/+38
|
* Undid my awful code which had broken the thread...Paul Phillips2010-02-262-4/+13
| | | | | | | Undid my awful code which had broken the thread scheduler selection. Further unbroke it beyond that unbreaking hopefully to the point where java 1.6 on OSX is recognized as such. Review by haller.
* close #3071.Lukas Rytz2010-02-262-11/+21
|
* closes #2741 closes #3079 no reviewAdriaan Moors2010-02-263-0/+35
| | | | | worksforme
* closes #2421 -- now also deals with chained imp...Adriaan Moors2010-02-262-5/+20
| | | | | | closes #2421 -- now also deals with chained implicits no review
* see #2634: updated docs to indicate zipped is s...Adriaan Moors2010-02-262-0/+10
| | | | | see #2634: updated docs to indicate zipped is strict
* closes #2956Adriaan Moors2010-02-263-4/+19
| | | | | | | the problem was that corresponds on Seq's does not check length of sequences before testing the predicate, whereas in some cases that predicate relied on this invariant (when it was doing substitution)
* closes #2797 -- no review (already done in tick...Adriaan Moors2010-02-262-9/+15
| | | | | | | | | | closes #2797 -- no review (already done in ticket by Martin) 1) isHigherKindedType is now false for singletontype 2) toInstance recurses when pre is a typevar: the problem is that pre.widen.typeSymbol isNonBottomSubClass symclazz is true while pre.baseType(symclazz) is NoType
* Added a --bare option to ShowPickled so I can d...Paul Phillips2010-02-253-23/+37
| | | | | | Added a --bare option to ShowPickled so I can diff signatures without all the explicit indices blowing any points of similarity. No review.
* More return type annotation to work around my o...Paul Phillips2010-02-252-6/+6
| | | | | | More return type annotation to work around my other frequent guest in the world of #3082-connected pickler bugs. No review.
* What appears to be a workaround for #3082, whic...Paul Phillips2010-02-251-1/+1
| | | | | | What appears to be a workaround for #3082, which I am hitting literally 20 times a day. Will detail in ticket. Review by odersky.
* More partest cleanups, and putting back a coupl...Paul Phillips2010-02-258-31/+28
| | | | | | More partest cleanups, and putting back a couple lines in build.xml which were left a little too commented out. No review.
* Looking at iulian's patch to the squeezer sent ...Paul Phillips2010-02-252-39/+25
| | | | | | | Looking at iulian's patch to the squeezer sent me off looking at equalsStructure, which clearly was written in a bygone era. Rewritten with modern tools. No review.
* Working on making the bootstrap process transpa...Paul Phillips2010-02-255-82/+67
| | | | | | Working on making the bootstrap process transparent and consistent. And removed a bunch of what is now cruft in partest. No review.
* Made doc comment consistent. No review.Philipp Haller2010-02-251-2/+2
|
* Physically moved ActorGC to scheduler directory.Philipp Haller2010-02-251-0/+0
|
* Made the squeezer worthy of its name: a block w...Iulian Dragos2010-02-251-2/+9
| | | | | | | | | Made the squeezer worthy of its name: a block with an empty list of stats and another block as the result expression is rewritten to the inner block (recursively). This makes the output from the pattern matcher nicer, eliminating loads of empty nested blocks. Review by extempore.
* Fixes #3088. No review.Aleksandar Pokopec2010-02-251-0/+2
|
* It turns out some of the weirdness lately is be...Paul Phillips2010-02-253-3/+3
| | | | | | | | | It turns out some of the weirdness lately is because changes to classpath handling have a way of not taking effect until they're installed via starr, and presently we have a starr with different logic than trunk. No choice but to roll up one more starr based on r20984. No review.
* Tweaking the sealed logic in light of #3097.Paul Phillips2010-02-256-14/+58
| | | | | | | Reorganizes children a little so they always come back sorted the same way the pickler does. Taking advantage of -Yfatal-warnings in the test case. Review by community.