summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Attempt to fix NoSymbol does not have owner pro...Martin Odersky2010-12-174-8/+13
| | | | | Attempt to fix NoSymbol does not have owner problem in Eclipse.
* Discovered bug in mapConserve introduced in r23...Paul Phillips2010-12-171-3/+1
| | | | | | | | | | Discovered bug in mapConserve introduced in r23038 when a tail recursive version was submitted. I am optimistic this is a player in recent performance degradation based on the timing of the commit and the fact that I was examining mapConserve because of info from my homemade profiler. (But it's late and I only just found it, so I'll be the last to know.) Review by odersky.
* do not add entry in InnerClass attribute if out...michelou2010-12-162-112/+116
| | | | | | | | | | | | | | | | do not add entry in InnerClass attribute if outer name is null (according to comment for outerName method) in order to remove the following warning message reported by the dx tool (Android): warning: Ignoring InnerClasses attribute for an anonymous inner class (test$$anonfun$1) that doesn't come with an associated EnclosingMethod attribute. This class was probably produced by a compiler that did not target the modern .class file format. The recommended solution is to recompile the class from source, using an up-to-date compiler and without specifying any "-target" type options. The consequence of ignoring this warning is that reflective operations on this class will incorrectly indicate that it is *not* an inner class.
* Added more logging to the presentation compiler.Iulian Dragos2010-12-162-1/+23
|
* Removed the 'codeGenerator' field, which was a ...Iulian Dragos2010-12-162-50/+63
| | | | | | | | | | Removed the 'codeGenerator' field, which was a source of leaks. Refactored javaName/javaType and the like. Now javaName does only name mangling, and it is overriden by the BytecodeGenerator to keep track of inner classes. JVMUtils now can be instantited by third party tools like Eclipse and use javaName without polluting the code generator's state. review by extempore.
* Make the ownPhaseCache field a weak reference.Iulian Dragos2010-12-161-5/+11
|
* Added advanceGeneration for memory profiling, a...Iulian Dragos2010-12-164-0/+18
| | | | | | Added advanceGeneration for memory profiling, and resident-mode profile option. review by extempore.
* Added secret option to sbt build which makes it...Paul Phillips2010-12-161-1/+9
| | | | | | Added secret option to sbt build which makes it only recompile changed files. No review.
* Stops barking up the wrong tree with -Ywarn-dea...Paul Phillips2010-12-1512-50/+90
| | | | | | | | | | | | | Stops barking up the wrong tree with -Ywarn-dead-code. The origin of its issues was twofold: 1) synchronized acts by-name without being by-name (ticket #4086) 2) warnings are swallowed if context.reportGeneralErrors is false Those two plus a dash of bitrot. In any case it's at its all time happiest now. It found all the dead code related fixes in this commit. Way to go, -Ywarn-dead-code! Review by odersky.
* Fixed various issues with -Ywarn-dead-code.Paul Phillips2010-12-144-3/+48
| | | | | enjoy fewer spurious warnings. Closes #1681, no review.
* Modified TreeMap to return a fresh empty node u...Paul Phillips2010-12-141-0/+1
| | | | | | | Modified TreeMap to return a fresh empty node upon removing the last element rather than the existing one, so old objects don't remain uncollectable forever. No review.
* More repl hardening against its fickle master, ...Paul Phillips2010-12-141-1/+7
| | | | | More repl hardening against its fickle master, Global. No review.
* Enabled delayedInit. Review by extempore.Martin Odersky2010-12-141-10/+13
|
* Preparing to enable delayedInit with new starr.Martin Odersky2010-12-143-20/+102
|
* Closes #4024. No review.Martin Odersky2010-12-142-1/+11
|
* An alteration in jline had hosed the ability to...Paul Phillips2010-12-142-1/+3
| | | | | | An alteration in jline had hosed the ability to resume from ctrl-Z. Fixed it. No review.
* Make the java iterator wrapper linear time.Paul Phillips2010-12-131-11/+13
| | | | | Closes #4077, review by dragos.
* More repl hardening, and a new jline jar which ...Paul Phillips2010-12-133-4/+16
| | | | | | More repl hardening, and a new jline jar which fixes a paste issue on OSX. No review.
* Hardening the repl against my own lack of ept.Paul Phillips2010-12-133-17/+15
|
* Told the boy trying to plug the dike holes that...Paul Phillips2010-12-126-19/+15
| | | | | | | | | | Told the boy trying to plug the dike holes that he can go home. Instead of trying to catch unpositioned EmptyTrees from the XML parser, don't generate them. It's the kind of commit you would tell your grandkids about if your grandkids could sit still for five seconds: fixes bugs by deleting code and eliminates extraneous error messages to boot (see .check diffs.) No review.
* New jline: BSD licensed, based on [ https://git...Paul Phillips2010-12-12105-7217/+7287
| | | | | | | | | | | | | | | | | | | | | | New jline: BSD licensed, based on [ https://github.com/jdillon/jline2 ] with additional code thanks to [ https://github.com/huynhjl/jline2 ]. Replaces lib/jline.jar with build of these sources, and modifies trunk code to work with the new jar. Hopeful improvements including baseline functionality on cygwin and 64bit windows, as well as more accurate line wrapping / cursor positioning on all platforms and ctrl-R history search. For the time being the canonical source repository is this: https://github.com/paulp/jline2 The enclosed sources are a mirror of that repository, and should be treated as read-only in the scala svn repository. No review, codewise, but people are very strongly encouraged to try it out and report any regressions.
* That cunning underscore tricked me into quoting...Paul Phillips2010-12-121-1/+2
| | | | | | That cunning underscore tricked me into quoting it and changing some test output. No review.
* A little more finnesse needed with backquoting.Paul Phillips2010-12-123-11/+35
|
* Finally figured out what needed to happen forPaul Phillips2010-12-124-25/+53
| | | | | | | import scala.`package`.Throwable not to break the repl. Closes #3673, no review.
* Eliminating all possible warnings from trunk.Paul Phillips2010-12-1113-30/+34
| | | | | | | one deprecation, one unchecked, and one "other", each of which volunteers no mechanism for suppression. (It would be nice to change this.) No review.
* Turned an assert/compiler crash into an error m...Paul Phillips2010-12-116-18/+35
| | | | | | | | Turned an assert/compiler crash into an error message. Admittedly the error output is still terrible (the XML positioning looks like it came from an RNG) but more helpful than the crash. Also misc advantage of a handy new method. Closes #4069, no review.
* When was the last time -Xcheckinit was run? It ...Paul Phillips2010-12-1116-73/+55
| | | | | | | | | | When was the last time -Xcheckinit was run? It must have been a long time. All these changes are to address bugs revealed by -Xcheckinit, mostly in test cases, some in the compiler. I'm guessing the partest -Xcheckinit runs are hanging the first time they run into a failure, so if it starts "working" again after this commit don't get too confident. No review.
* Eliminated the 10 unchecked warnings emanating ...Paul Phillips2010-12-111-9/+9
| | | | | | Eliminated the 10 unchecked warnings emanating from RedBlack.scala. No review.
* I found myself unable to make the necessary par...Paul Phillips2010-12-094-834/+828
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I found myself unable to make the necessary parser improvements for the pattern matcher without breaking it down better. This patch represents a lot of work. Too many booleans were being threaded in an arbitrary way through parse productions, resulting in an abundance of expressions like annotations(true, false) path(false, true) Also there was too much duplication of code, and unnecessary use of mutable state. So here are the main points: * much less code duplication * many listbuffers as arguments eliminated in favor of good old fashioned call a method and use the result * several booleans eliminated in favor of grouping methods by calling context, so the booleans emerge automatically * misc bug fixes, such as the fact that most operators which start with a letter have been getting the highest precedence instead of the lowest. (I can say most because it's definitely true by the numbers: nobody has reported it because a-zA-Z do the right thing, it's _, $, and the entirety of the Unicode letters which were wrong) * sporadically introduced/improved documentation * misc better error messages, like: scala> import scala.{ _, String => Bob } // before <console>:1: error: '}' expected but ',' found. import scala.{ _, String => Bob } // after <console>:1: error: Wildcard import must be in last position import scala.{ _, String => Bob } ^ Anticipating the performance objection, I measured it and the difference is immeasurable. And the usual finish: I can't imagine anyone being interested in or willing to review this, but am all ears if there are counterexamples. Meanwhile, no review.
* closes #4064.Adriaan Moors2010-12-093-2/+11
|
* Minor changes.Aleksandar Pokopec2010-12-093-4/+8
| | | | | No review.
* Made parallel collections serializable.Aleksandar Pokopec2010-12-0925-34/+189
| | | | | No review.
* Array combiners implementation changed from arr...Aleksandar Pokopec2010-12-0926-244/+567
| | | | | | | | | Array combiners implementation changed from array buffers to doubling unrolled buffers to avoid excessive copying. Still evaluating the benefits of this. No review.
* Fixed parallel ranges to use the same range log...Aleksandar Pokopec2010-12-0910-51/+97
| | | | | | | | | | Fixed parallel ranges to use the same range logic under the hood, and not introduce code duplication. Slight fix in Tasks. No review.
* Fixing jvm 1.5 support for parallel collections.Aleksandar Pokopec2010-12-0927-128/+371
| | | | | | | | | | | | | | | | | | Special cased with thread pool executor scheduling. Fixed an ugly concurrency bug where futures returned by a thread pool executor didn't remove the task from the queue when cancel was called. Note to self and others: don't cancel futures returned by thread pool executors, it might lead to unexpected behaviour. Modified the executor to add new threads if all the active threads are syncing, in order to avoid deadlocks. Fixed a hidden bug in AdaptiveWorkStealingTasks, where correct behaviour depended on the execution order of the tasks. This didn't fail before with ForkJoinTasks, since there the execution order is well-defined. Scalachecked 1.5 & 1.6 support. No review.
* minor code updates (detach plugin)michelou2010-12-0814-174/+50
|
* again: relax access boundry check for overridin...Lukas Rytz2010-12-0811-21/+26
| | | | | | again: relax access boundry check for overriding protected java members. review by eugenevigdorchik.
* I learned from #4067 that we don't have any kin...Paul Phillips2010-12-083-0/+52
| | | | | | | | | | | | | | | | | | | | I learned from #4067 that we don't have any kind of signature validation yet. I could swear someone did that. Anyway, I did. If you give option -Yverify-generics it will yell at you for any that are invalid. For instance, the one in the ticket. % scalac -Yverify-generics -d /tmp src/library/scala/collection/immutable/List.scala [BAD!] method orElse in scala.collection.immutable.List <A1:IB1:Ljava/lang/Object;>(Lscala/PartialFunction<TA1;TB1;>;)Lscala/Par tialFunction<TA1;TB1;>; [BAD!] method orElse in scala.collection.immutable.List <A1:IB1:Ljava/lang/Object;>(Lscala/PartialFunction<TA1;TB1;>;)Lscala/Par tialFunction<TA1;TB1;>; Of course you still have to figure out what is wrong and why. Review by moors, dragos.
* An unfortunate missing character in r22946 turn...Paul Phillips2010-12-081-1/+1
| | | | | | | An unfortunate missing character in r22946 turned listbuffer into an append-only machine. I was the reviewer on that commit and I am sad that I missed it. Closes #4066, no review.
* The lords of attrition informed me that "isType...Paul Phillips2010-12-081-11/+11
| | | | | | | The lords of attrition informed me that "isTypeApply" is passed all over the parser but nobody ever eats the hot potato. I put the potato on ice. No review.
* Test case closes #3642, no review.Paul Phillips2010-12-072-0/+10
|
* Made it possible to give :cp an argument with s...Paul Phillips2010-12-071-2/+5
| | | | | | Made it possible to give :cp an argument with spaces. Closes #3819, no review.
* Extricated Addable and AddingBuilder from the f...Paul Phillips2010-12-0712-38/+38
| | | | | | | Extricated Addable and AddingBuilder from the few classes still utilizing them and deprecated both. Closes #4059. Already endorsed by odersky, so no review.
* Reverts previous commit: needs a new starr..Lukas Rytz2010-12-075-16/+5
| | | | | This reverts commit 99652fe553a1a5b0c551950e8178e867943a088f.
* follow up on fix #3946.Lukas Rytz2010-12-075-5/+16
|
* Added uniqueness annotations. Review by rytz.Philipp Haller2010-12-077-9/+98
|
* Parse the contents of the package right away in...Eugene Vigdorchik2010-12-073-10/+14
| | | | | | Parse the contents of the package right away in the presentation compiler. No review.
* Making some unpickler logic more straightforward.Paul Phillips2010-12-072-122/+140
|
* Looks like I'd gotten a little quote-happy there.Paul Phillips2010-12-071-2/+2
|
* Oops, lost some java options for a moment there.Paul Phillips2010-12-071-1/+2
|
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-03 13:51:09 +0000