| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
| |
Realized how noisy the @deprecated-takes-two-arguments message was and
quieted it down. Oh, and documented the arguments. No review.
|
| |
|
|
|
|
|
|
| |
Since I don't want to commit anything "interesting" until we ship 2.9, a
few uninteresting cleanups involving how types are printed, getting some
debugging code in shape to prepare for the long winter ahead, etc. No
review.
|
| |
|
|
|
|
| |
Don't issue a deprecation warning if the deprecated method is being
called from a bridge method. No review.
|
| |
|
|
|
|
| |
A few minor cleanups involving logging messages, assertion messages,
compiler warnings, and other innocuous matters. No review.
|
| |
|
|
|
|
|
|
| |
Completely to my surprise, found that fixing all those sequence issues
revealed that the pattern matcher can catch a lot more inexhaustive
cases than it has been catching. Fixed most of the inexhaustive matches
in the compiler, which had become a bit warnier. No review.
|
| | |
|
| |
|
|
|
|
|
| |
After having to update the code for someone else, ran damarau
levenshtein on trunk again. Patchwise, I guess correcting spelling
errors in comments is about as safe as it gets. No review.
|
| |
|
|
|
| |
Making further unapply regressions less likely, no review.
|
| |
|
|
|
|
|
|
| |
Fixing the pattern matcher regression I introduced between rc1 and rc2.
Not done with this situation but at least I managed to boil out the big
problem and keep my five closed pattern matcher tickets to boot. Closes
#4523, no review.
|
| | |
|
| |
|
|
|
|
| |
Issuing warnings with detailed info when we encounter the lift crasher
case.
|
| |
|
|
|
| |
Second version of trying to avoid the lift crasher bug.
|
| |
|
|
|
|
|
| |
Ignore type errors raised in later phases that are due to mismatching
existentials. Quick fix to address lift build failures. Review by
dragos.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I wrote a warning when nullary methods return Unit. I wimped out of
including it in this patch because we had about 200 of them, and that's
what is fixed in this patch. I will add the warning to some kind of
"-Xlint" feature after 2.9.
This is motivated at least partly by the resolution of #4506, which
indicates the distinction between "def foo()" and "def foo" will
continue to jab its pointy stick into our eyes, so I believe we have a
minimal duty of at least following our own advice about what they mean
and not making a semirandom choice as to whether a method has parens or
not. Review by community.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Improved the error message for another of the most common situations
I hear about in newbieland. It could be taken further. If compilation
fails due to an unimplemented abstract method, and there is a concrete
method of the same name and arity, it will do a pairwise analysis of the
parameters and attempt to further explain where you went off the beam if
it feels it can do so sensibly. Such as in the test case:
% scalac S.scala
S.scala:1: error: class S needs to be abstract, since method g in class J of type (y: Int,z: java.util.List)Int is not defined
(Note that java.util.List does not match java.util.List[String]. To implement a raw type, use java.util.List[_])
class S extends J {
^
one error found
No review.
|
| |
|
|
|
|
|
|
|
| |
Improvements to the AST browser contributed by Yuvi Masory. Uses Nimbus
LAF when available and improves readability by padding components. Adds
menu items and key bindings for expanding and contracting nodes, closing
the browser and continuing compilation, and closing the browser and
aborting. No review.
|
| |
|
|
|
|
|
|
| |
Upgraded -d so you can output classes directly to a jar. Very (very)
loosely based on a patch from dmharrah. Like dmharrah before me, I
see little if any change in compile times, which I find difficult to
explain. Closes #27, review by dmharrah.
|
| |
|
|
|
| |
Cleaned up some hopelessly atrophied documentation, no review.
|
| |
|
|
|
|
|
|
| |
Trying out a different strategy for restoring terminal settings so we
don't have a list of hardcoded terminal types. Now it saves the terminal
settings on script start and restores those on exit. Closes #4170,
review by rytz.
|
| |
|
|
|
|
|
|
|
| |
Trying to fix the pattern matcher took me into the lambda lifter, and
I made some changes which seemed sensible to me. I'm going to be a
stickler about eliminating mutable maps which hold mutable listbuffers.
I could use some confirmation that I didn't somehow break the world:
review by dragos.
|
| | |
|
| |
|
|
|
| |
Closes #4498, no review.
|
| |
|
|
|
| |
Fixed inversion of fsc's exit code, closes #4519 no review.
|
| | |
|
| |
|
|
|
| |
Some solid progress on the pattern matcher, no review.
|
| |
|
|
|
|
| |
Added a bunch of bridges to make ameliorate binary compatibility of new
collections. Review by prokopec. Review by extempore.
|
| |
|
|
|
|
| |
Added warning when someone tries to return a non-Unit value from a Unit
method, no review.
|
| |
|
|
|
|
| |
Removed restriction on case classes having only two parameter lists.
Closes #1333, no review.
|
| |
|
|
|
| |
Enclosed the out of bounds index with some IOOB exceptions, no review.
|
| |
|
|
|
| |
Updated a bunch of @deprecated annotations to have a version, no review.
|
| | |
|
| |
|
|
|
|
|
|
|
| |
Working my way through pattern matcher sequence issues mostly caused by
the special handling of Lists. Also deleting all kinds of useless or
almost useless code which is presently only clutter.
Closes #2756, #2800, #3050, #3530, #3972, no review.
|
| | |
|
| |
|
|
|
| |
Reverted a jline commit which had broken history search, no review.
|
| |
|
|
|
|
| |
Strip unused pattern variable bindings out before performing match
translation. Closes #4269, no review.
|
| |
|
|
|
|
| |
Introduce new request for informing the presentation compiler of files
deletion. Review by odersky.
|
| |
|
|
|
|
| |
Fixing a BaseBerrySethi bug I introduced in r19842. Patch by buraq, no
review.
|
| |
|
|
|
|
| |
Fixed spurious qualification of types that are locally quantified.
Review by extempore.
|
| | |
|
| | |
|
| |
|
|
|
|
| |
Made FreshRunReq a traceable exception, so that we can better spot
problems in the IDE.
|
| |
|
|
|
|
|
|
|
|
|
| |
There is still all kinds of code in the pattern matcher which I've
been dragging forward (for years now) unclear on the need for it but
having been burned too many times by its undocumented ways. Returned to
the fray once again and encountered shocking success levels with the
abatement stick. Dumped all kinds of unnecessary code (or so we can
hope) and seem close to fixing some fundamental issues. But we've heard
that before. No review.
|
| |
|
|
|
|
|
|
|
|
| |
One of the blips in the performance charts seems to implicate some
changes I made with slice to reduce the number of implementations and
surface area for inconsistencies and bugs. Altering those changes in a
more performance-mindful way, although I don't see anything here which
is likely to help much. Also fixing some wrong documentation about
copyToArray. No review.
|
| |
|
|
|
| |
Some tweaks to slice to make it run faster, no review.
|
| |
|
|
|
|
| |
[scaladoc] kindFilter should switch show/hide anchor of package. Closes
#4463. Review by malayeri.
|
| |
|
|
|
| |
[MSIL] finer-grain debugging (steps one sub-expression at time).
|
| | |
|
