summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Disambiguated all the implicits in JavaConversi...Paul Phillips2010-10-162-263/+287
| | | | | | | | | | Disambiguated all the implicits in JavaConversions. All the documentation had fallen out of date so I fixed that up too, in the process converting from HTML to wikiformatting. (Wouldn't it be nice to mass convert trunk out of the HTML tags?) Closes #3688, no review.
* new tests. no review.Martin Odersky2010-10-166-0/+58
|
* Small changes in layout and comments. no review.Martin Odersky2010-10-162-3/+3
|
* Closes #3837. Review by extempore.Martin Odersky2010-10-161-3/+7
|
* Closes #3614.Martin Odersky2010-10-162-16/+20
|
* Closes #3567. Review by milessabin.Martin Odersky2010-10-161-2/+11
|
* Closes #3833. Review by extempore.Martin Odersky2010-10-161-24/+39
|
* Closes #3672. Review by extempore.Martin Odersky2010-10-161-3/+14
|
* TreePrinters now go inside orginal's of TypeTre...Martin Odersky2010-10-161-1/+1
| | | | | | TreePrinters now go inside orginal's of TypeTrees to print positions. Useful for debugging IDEs
* Reverting r23232. No review.Hubert Plociniczak2010-10-155-122/+42
|
* removed docs/licences (doc/licenses already exi...Antonio Cunei2010-10-151-16/+0
| | | | | removed docs/licences (doc/licenses already exists)
* Working out some more of the flags situation.Paul Phillips2010-10-155-37/+61
|
* There was a fascinating tangle of name creation...Paul Phillips2010-10-1511-66/+26
| | | | | | | | | | | | | | | There was a fascinating tangle of name creation functions passing around positions only to reach the end of the line and discard the position since names don't have positions. I deleted all of it. The winner for most creative use of parameters was the freshName function in etaExpand, with this signature: def freshName(pos: util.Position, n: Int) And an implementation referencing neither pos nor n. "In a world beset by attrition on all sides... a people defeated by entropy... one man will show them the power of deletion." No review.
* Modification to the widening logic to treat loc...Paul Phillips2010-10-144-9/+44
| | | | | | | | | | | | | | | | | | Modification to the widening logic to treat locally defined symbols like final members thus allowing more constants to be inlined. Concretely, that means that in code like this: def f: Unit = { val b = false ; if (b) println("ok") } The call to println is no longer generated at all, and in this code: def f(x: Int) = { val X = 1 ; val Y = 2; x match { case X => 1 ; case Y => 2 } } A tableswitch is generated instead of the present if/then/else. I also added a big comment to the former widenIfNotFinal (now widenIfNecessary for obvious reasons.) Review by rytz.
* All this time with Nil only claiming to be equa...Paul Phillips2010-10-142-1/+9
| | | | | | All this time with Nil only claiming to be equal to immutable.Seqs, how is it possible? Closes #3923, no review.
* An error previously unnoticed.Aleksandar Pokopec2010-10-141-3/+2
| | | | | Review by extempore.
* Improving scalacheck update script.Aleksandar Pokopec2010-10-141-3/+122
| | | | | These improvements are based upon suggestions given by extempore, so - review by extempore.
* re-added isAbstract and isTrait to HasFlag beca...Lukas Rytz2010-10-142-3/+3
| | | | | | | | re-added isAbstract and isTrait to HasFlag because sbt build breaks otherwise (compiler-interface accesses Modifiers.isAbstract). note that the sbt build still doesn't work right now because of a bug in r23232. hubert is working on that. review by extempore.
* Fixed the broken build by my previous change.Iulian Dragos2010-10-141-1/+1
|
* Fixed wrong tag for Longs, which caused bad jav...Iulian Dragos2010-10-141-8/+10
| | | | | | Fixed wrong tag for Longs, which caused bad java signatures (and a crash building the Eclipse plugin). No review.
* Fixing issue with XMLEventReader.stop failing t...Paul Phillips2010-10-143-1/+43
| | | | | | Fixing issue with XMLEventReader.stop failing to stop the parser thread. Contributed by Jean-Laurent Huynh, reviewed by extempore. Closes #3881.
* An object with a main method can now extend its...Paul Phillips2010-10-142-153/+146
| | | | | | | | | | | | An object with a main method can now extend its companion class without losing the ability to run. Also gave genjvm a wax and polish while in the neigborhood. Closes #3487. I had to get a little tricky with this one. No review per se, but if anyone is frightened by the prospect of the forwarders not being marked final in this case (because as of now, they aren't) then speak now or ACC_FINAL hold your peace.
* Closes #3894. Review by dubochet.Martin Odersky2010-10-131-4/+4
|
* closes #3862.Adriaan Moors2010-10-133-4/+17
|
* closes #3281, #3866.Adriaan Moors2010-10-133-12/+36
| | | | | | | | | | | I don't understand why we had to clear() the undoLog in the first place, since the undoXXX methods increase and decrease its size symmetrically, so the log should always be empty once they have all unwound. Was it a (premature) optimisation or was there some kind of semantic meaning to it that I didn't see? review by odersky
* Misc. cleanups in specialization. No review.Paul Phillips2010-10-132-62/+52
|
* Added filterNot to Option so I can stop being d...Paul Phillips2010-10-131-0/+13
| | | | | | | | | | | | | | Added filterNot to Option so I can stop being driven mad by this difference. scala> Some(5) filter (_ > 2) res0: Option[Int] = Some(5) scala> Some(5) filterNot (_ < 2) res1: Iterable[Int] = List(5) No review.
* The second piece of the flags patch.Paul Phillips2010-10-1311-105/+168
| | | | | | | | | | | into Modifiers and Symbol, but touches as little as possible beyond that. It also includes some lengthy commentary (see HasFlags.scala) on the state of the flags and some of the remaining issues. One more patch which unfortunately but unavoidably touches almost every file in the compiler lies ahead. The floor is still open! But no review.
* This maps @native, @volatile, and @transient in...Paul Phillips2010-10-123-39/+36
| | | | | | | | | This maps @native, @volatile, and @transient in java source to the relevant annotations in scala, rather than discarding the modifiers completely. I have no specific motivation for this other than it seems better to do it than not. Also cleans up some old flailing of mine in the dept. of mapping java access to scala. No review.
* Two improvements (or so I claim) to MatchErrors.Paul Phillips2010-10-121-3/+9
| | | | | | | | | | | | | | toString when constructing the exception: it need not be called until/unless it is printed, which it may never be. 2) Include the name of the class which triggered it. Don't tell me you haven't wanted that as many times as I have. (Sidebar on commit message semantics: I assume nobody interprets "no review" to mean anything like "unreviewable decision!" It only means I'm pretty sure the code does what I intended. The floor is always open.) That said, no review.
* One more partest fix where output was redirecte...Aleksandar Pokopec2010-10-122-23/+62
| | | | | | One more partest fix where output was redirected using a global variable, causing errors with multiple actors. Review by plocinic
* closes #3915.Adriaan Moors2010-10-121-1/+1
|
* closes #3890: propagate inferred type parameter...Adriaan Moors2010-10-123-1/+28
| | | | | | closes #3890: propagate inferred type parameters to info of undetermined type parameters. review by imaier.
* Closes #2333. Review by dragosHubert Plociniczak2010-10-122-1/+18
|
* Fixes #3895.Hubert Plociniczak2010-10-125-42/+122
|
* Fixed substring test that was occasionally fail...Hubert Plociniczak2010-10-122-2/+1
| | | | | | | Fixed substring test that was occasionally failing on my machine (and reporting the error for a different test case). Proper fix for scalacheck is still pending. No review.
* Encapsulating name strings in StdNames and elim...Paul Phillips2010-10-117-49/+34
| | | | | | Encapsulating name strings in StdNames and eliminating misc duplication. No review.
* Introduced -Ymurmur with murmur hashcodes.Paul Phillips2010-10-117-6/+136
| | | | | | | | contributed by "archontophoenix", following in the grand tradition of code by people whose actual names I don't know. References #2537, but it doesn't close until some sensible hashcode is used by default. Review by community.
* An overhaul of the collection-oriented methods ...Paul Phillips2010-10-1195-290/+390
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An overhaul of the collection-oriented methods in Tuple2/Tuple3 (which still need to be taken all the way to Tuple5.) * Zip semantics: zip and zipped now properly terminate when any collection runs out of elements, even if others are infinite. In addition, short circuiting methods (exists and forall) will terminate if the condition is met, even in the presence of infinity. Example: scala> val ys = Stream from 1 ys: scala.collection.immutable.Stream[Int] = Stream(1, ?) scala> (ys, ys).zipped forall ((x, y) => x+y < 100) res0: Boolean = false scala> (ys, ys).zipped exists ((x, y) => x+y > 100) res1: Boolean = true * There are implicits converting Zipped2/3 to Traversable to expose all the methods which aren't defined in an arity-specific way in the tuple classes. I have mixed feelings about putting these in Predef; but if there is another way to make them visible by default I wasn't able to find it. Example putting said implicit to use: scala> (ys, ys, ys).zipped find { case (x, y, z) => x+y+z > 1000 } res0: Option[(Int, Int, Int)] = Some((334,334,334)) Already reviewed by moors, so no review.
* Abstracting out the common flags code between S...Paul Phillips2010-10-117-139/+546
| | | | | | | | | | | | | | | | | | | | | Abstracting out the common flags code between Symbol and Modifiers into HasFlags. This patch includes only the non-invasive changes: the HasFlags trait is not integrated into those classes. The remainder is complete but I'm checking this part in case anyone would like to throw some feedback my way at this point: general comments, or the open issues noted in comments in HasFlags.scala. This commit also eliminates the (now unused) MONOMORPHIC flag and includes utility methods for reflective analysis of a Flags object which generate code and comments based on the accessors found. See the comment at the top of symtab.Flags and the flagToString implementation in that class for illustration. I haven't tested the very latest, but a slightly older incarnation without material differences showed no measurable performance change. No specific review, but comments welcome.
* Updated IntelliJ sample files. no review.Lukas Rytz2010-10-1110-494/+70
|
* temporary fix for compiler crash in dependend m...Lukas Rytz2010-10-113-2/+14
| | | | | | temporary fix for compiler crash in dependend method types with annotations. review by moors.
* Some cleanups in the compiler source.Paul Phillips2010-10-1129-101/+82
| | | | | | | eliminated the import of ambiguously named classes from e.g. collection.mutable, obeyed a todo in the parser regarding dropping lbracket from statement starting tokens. No review.
* Better check for objects trying to sneak themse...Paul Phillips2010-10-113-5/+17
| | | | | | Better check for objects trying to sneak themselves into the superconstructor arguments. Closes #3913, no review.
* Settings help tweak. Closes #3910, no review.Paul Phillips2010-10-091-1/+1
|
* Abstraction improvement in Flags. No review.Paul Phillips2010-10-081-94/+45
|
* Made some adjustments to toString formatting of...Derek Chen-Beker2010-10-083-40/+154
| | | | | | | | | | | Made some adjustments to toString formatting of JSON Closes #3605 Hopefully this is the last time I have to close this ticket. In addition to default behavior, the end user can specify their own JSON value formatting function if they want to customize it.
* Added more complete output if tests fail.Aleksandar Pokopec2010-10-084-4/+22
| | | | | | | | | Log file is printed now right away if: - test fails due to an exception thrown in the a test - test fails due to a ScalaCheck test fail - test fails due to compiler errors in ScalaCheck tests Review by extempore.
* Fans of scala cinema who have viewed this types...Paul Phillips2010-10-083-65/+117
| | | | | | | | | | | | | Fans of scala cinema who have viewed this types video: http://www.scala-lang.org/sites/default/files/martin_ordersky_scala_inte rnals_2009-03-04.avi may have noticed that martin entered a variety of priceless comments in the course of the lecture and never checked them in. As the self-appointed royal stenographer I have come forward to right this wrong. It being a grainy video and almost inaudible, I may well have butchered it somewhere, so review by odersky.
* Did a bunch of symbol oriented work on checkers.Paul Phillips2010-10-077-27/+166
| | | | | | | changes in Global and Typer to accomodate this, and renamed "Checkers" to "ICodeCheckers" to make some less confusing space for the future "SymbolCheckers". No review.