summaryrefslogtreecommitdiff
path: root/src/compiler
Commit message (Collapse)AuthorAgeFilesLines
* Commit r22003 changed the svn:ignoreAntonio Cunei2010-05-261-2/+2
| | | | | | | | | | | property of every directory in the source tree, probably unintentionally. http://lampsvn.epfl.ch/trac/scala/changeset/22003 Reverting those changes, leaving just the change to MutableList.
* also consider non-implicit locals when checking...Adriaan Moors2010-05-261-1/+19
| | | | | | | | | | | | | also consider non-implicit locals when checking shadowing of implicits: closes #3453 nonImplicitSynonymInScope implements the predicate that is used in tryImplicit's checks for shadowing of locally defined implicits benchmarking shows the predicate does not significantly affect quick.comp+quick.lib (goes from 11min to 11min2s on my machine -- no optimisations) review by odersky
* Closes #3452, inconsistency between method desc...Iulian Dragos2010-05-261-3/+6
| | | | | | Closes #3452, inconsistency between method descriptors and java signature for static forwarders. Review by extempore.
* Closes #3434.Iulian Dragos2010-05-261-4/+29
|
* Added a migration warning for matches and insta...Paul Phillips2010-05-262-3/+20
| | | | | | Added a migration warning for matches and instance tests when it might be an Array/Seq test whose answer has changed. Review by odersky.
* Made interpreter robust against a missing class...Paul Phillips2010-05-262-13/+31
| | | | | | | | | | | | | | | Made interpreter robust against a missing classpath and had it issue informative message in such cases, as per suggestion by Razvan Cojocaru. No review. % java -classpath build/pack/lib/'*' scala.tools.nsc.MainGenericRunner scala> Failed to initialize compiler: class scala.runtime.BooleanRef not found. ** Note that as of 2.8 scala does not assume use of the java classpath. ** For the old behavior pass -usejavacp to scala, or if using a Settings ** object programatically, settings.usejavacp.value = true.
* More thorough fix for #3454Martin Odersky2010-05-251-2/+4
|
* Fixe for botched Parser checking for #3454.Martin Odersky2010-05-251-1/+1
|
* Closes #3469. Review by rytz.Martin Odersky2010-05-251-3/+7
|
* Closes #3454, plus one syntax generalization wh...Martin Odersky2010-05-251-1/+2
| | | | | | | Closes #3454, plus one syntax generalization which lets us have mulitple if guards in for expressions without semicolons between them. Review by moors.
* Removed a few instances of redundant or dead co...Paul Phillips2010-05-243-11/+6
| | | | | | | Removed a few instances of redundant or dead code, most of which was marked as waiting for a new starr which has long since arrived. No review.
* Changed the script runner mechanism to alchemiz...Paul Phillips2010-05-2314-244/+176
| | | | | | | | 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.
* Adding some info to naked asserts so future cra...Paul Phillips2010-05-231-2/+2
| | | | | | Adding some info to naked asserts so future crashes of the same stripe won't be silent on what happened. No review.
* Pulled a function from the repl into the packag...Paul Phillips2010-05-222-11/+12
| | | | | | Pulled a function from the repl into the package object because I need it outside the repl. No review.
* It's clearly not a long term solution to go hun...Paul Phillips2010-05-221-3/+2
| | | | | | | | It's clearly not a long term solution to go hunting them one by one, but for the short term goal of shipping 2.8.0 without accidental java 6 dependencies, removed what should be all uses of String.isEmpty from the source. No review.
* A workaround for the crasher described in the c...Paul Phillips2010-05-221-1/+13
| | | | | | | A workaround for the crasher described in the comments of #3431 (not related to #3431 per se.) Thanks to iulian for pinpointing what I can't do during completion member discovery. No review.
* Discovered via irc one more assertion in the ma...Paul Phillips2010-05-222-5/+4
| | | | | | Discovered via irc one more assertion in the matcher which was asserting something too specific. Fixed, no review.
* Made workaround for #3430 noisier during its wo...Paul Phillips2010-05-221-1/+7
| | | | | | | Made workaround for #3430 noisier during its working around so we don't get complacent about whether there's still a bug in the optimizer. Conceived by dragos, ergo no review.
* Altered a bunch of places which call hashCode t...Paul Phillips2010-05-217-12/+11
| | | | | | Altered a bunch of places which call hashCode to call ## instead. No review.
* Added +=: prepend to MutableList.Martin Odersky2010-05-211-2/+2
|
* Workaround for #3462.Paul Phillips2010-05-211-2/+5
| | | | | | | | way it was, but this is the obvious fix for now. Review by odersky (perhaps you could point out where my logic fails, in that I expected the concrete return type of withHelpSyntax to contain a setter for value.)
* A more specifically targetted version of r21991...Paul Phillips2010-05-211-2/+2
| | | | | | | A more specifically targetted version of r21991 which ought to eliminate the pathological behavior displayed by files/run/Course-2002-07.scala under -optimise. No review.
* Make the inliner be more careful about where it...Paul Phillips2010-05-201-0/+3
| | | | | | Make the inliner be more careful about where it pokes around. Closes #3252, #3430. Review by dragos.
* Closes #3449. no review.Iulian Dragos2010-05-201-8/+14
|
* Misoptimization for list extractors caused non-...Paul Phillips2010-05-201-1/+8
| | | | | | Misoptimization for list extractors caused non-sequences to incorrectly match. Closes #3050, #2800. No review.
* Closes #3381.Iulian Dragos2010-05-203-44/+60
|
* Closed #3432.Iulian Dragos2010-05-201-0/+4
|
* Closes #3304. Review by extempore.Martin Odersky2010-05-191-2/+9
|
* tentative solution of typeParams problem discov...Martin Odersky2010-05-191-1/+14
| | | | | tentative solution of typeParams problem discovered by Iulian.
* Hacky but no-risk fix for #3189, which was caus...Paul Phillips2010-05-191-1/+3
| | | | | | | Hacky but no-risk fix for #3189, which was caused by the ghost of regular expression patterns rising from the grave to haunt the parser. No review.
* some more ad-hoc cookingAdriaan Moors2010-05-181-0/+1
| | | | | | closes #3429 review by odersky
* cook raw type when copying down result typeAdriaan Moors2010-05-181-0/+1
| | | | | | closes #3404 review by odersky
* Changed strategy to allow specialization on typ...Iulian Dragos2010-05-181-6/+14
| | | | | | Changed strategy to allow specialization on type params at @specialized positions. no review.
* Commented out unnecessarily verbose debug info.Iulian Dragos2010-05-181-8/+8
|
* Closes #3417. No review (already done by Martin)Hubert Plociniczak2010-05-183-5/+4
|
* [scaladoc] Closes #3428 (HTML markup supported ...Gilles Dubochet2010-05-154-73/+146
| | | | | | | [scaladoc] Closes #3428 (HTML markup supported in Scaladoc). Improves wiki-syntax parsing of lists. Improves stylesheet for headings and code blocks in comments. Review by malayeri.
* Swapped the order of the arguments to returning...Paul Phillips2010-05-153-8/+8
| | | | | | | | | | | Swapped the order of the arguments to returning to make it consistent with the way I've been using it for a long time. No review, how about a question for nobody in particular instead: can we put this in the library somewhere so I can stop looking around for it everywhere I go? It's really handy. def returning[T](x: T)(f: T => Unit): T = { f(x) ; x }
* Starting to look like the pattern matcher and I...Paul Phillips2010-05-143-50/+43
| | | | | | | Starting to look like the pattern matcher and I can meet in the middle. More distributing my acquired bottom-up knowledge among the current code. No review.
* Fixed a missed opportunity for code rewriting i...Iulian Dragos2010-05-141-16/+11
| | | | | | | | | | | Fixed a missed opportunity for code rewriting in specialize: a method that was specialized both on enclosing class types and its own was routed to the generic one. For instance, a map[@specialized B] would exhibit this bug. Restricted warnings to non-synthetic methods, removing warnings for Tuple default argument methods. No review.
* Took advantage of my new wealth of bottom-up pa...Paul Phillips2010-05-141-65/+38
| | | | | | Took advantage of my new wealth of bottom-up pattern matching knowledge to simplify MixTypes a bunch. No review.
* Realized that somehow the synthetic flag on pat...Paul Phillips2010-05-131-2/+2
| | | | | | | Realized that somehow the synthetic flag on pattern matcher temp vars was lost somewhere, and classfile debugging info was being generated for every one of them. Fixed. No review.
* Closes #3433.Iulian Dragos2010-05-131-1/+9
|
* Closes #3434.Iulian Dragos2010-05-131-44/+55
|
* Should fix the interpreter bug I just introduced.Paul Phillips2010-05-131-1/+1
| | | | | | | crashing-under-optimise test into pending since that looks like a genuine bug - I'll open it a ticket. No review.
* Removed an assertion which was asserting an unt...Paul Phillips2010-05-131-17/+6
| | | | | | Removed an assertion which was asserting an untrue statement thus causing chaos and strife. Closes #3411, no review.
* Fixed a long standing issue with interpreter co...Paul Phillips2010-05-121-5/+5
| | | | | | | | Fixed a long standing issue with interpreter commands containing more than a single statement. It was the classic "reversing a list and then flattening it is not the same as flattening and then reversing it." Closes #3116, no review.
* Notice and fail when a file given with -i is in...Paul Phillips2010-05-123-57/+66
| | | | | | | Notice and fail when a file given with -i is incomplete (which under other conditions in the repl is quite different from not compiling, thus the bug.) Closes #3011, no review.
* Overhauled sequence length logic in the pattern...Paul Phillips2010-05-123-87/+89
| | | | | | | | Overhauled sequence length logic in the pattern matcher. Removes unnecessary boxing and a few varieties of wrongness. Closes #3395, #3150, #2958, #2945, #2187. No review.
* Closes #3424, double definition when overriding...Iulian Dragos2010-05-121-10/+13
| | | | | | Closes #3424, double definition when overriding a generic method with specialized types, inherited from a specialized class. no review.
* [scaladoc] Closes #3351 (white space before fir...Gilles Dubochet2010-05-122-4/+5
| | | | | | [scaladoc] Closes #3351 (white space before first period in comment). No review.