| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Created infrastructure for testing icode + settings/partest yak shaving.
See enclosed test files/run/inline-ex-handlers.scala. To compare
optimized and unoptimized icode after a given phase, all you need in a
partest source file is this:
import scala.tools.partest.IcodeTest
object Test extends IcodeTest {
override def printIcodeAfterPhase = "inlineExceptionHandlers"
}
Other things can be done, see IcodeTest.scala. Review by ureche.
|
| |
|
|
|
|
|
|
| |
Bug in optimizer eliminated potentially exceptional not-dead code.
Streamlined isSideEffecting logic. Review by ureche.
|
|
|
|
|
| |
0-arg getClass reach bytecode. References SI-4931, no review.
|
|
|
|
|
| |
a hurry, no review.
|
|
|
|
|
| |
References SI-4931, no review.
|
| |
|
|
|
|
|
| |
Moved files to welcome our ant overlords, no review.
|
|
|
|
|
|
| |
[recommit] Backend optimization: Inline exception handlers. Review by
dragos.
|
|
|
|
|
|
| |
Literals now take Any as argument (used to represent free values in
reified trees).
|
| |
|
|
|
|
|
| |
New utilitly method to create scope with some elements.
|
|
|
|
|
|
|
|
|
| |
Major rewrite of the testing infrastructure for the presentation
compiler. Added several new tests that will be part of the nightly
build. Once the move to SBT is completed I will look into how to extract
the test infrastructure (as it should really not be living in the
compiler codebase). Review by dragos
|
| |
|
|
|
|
|
|
| |
Improved the error message given when a concrete method implementation
doesn't match the abstract one. No review.
|
|
|
|
|
|
|
| |
Removing the code which has been deprecated since 2.8.0. Contributed by
Simon Ochsenreither, although deleting code is such fun one hesitates to
call it a contribution. Still, we will. Closes SI-4860, no review.
|
|
|
|
|
|
|
| |
Fixed a method which got hosed in the reflection refactor because the
meaning of "this" changed. This is a perfect opportunity to yell "What
is the meaning of this!" No review.
|
|
|
|
|
|
| |
Split up adapt in some smaller methods; added code lifting for
non-function types to it.
|
|
|
|
|
| |
Added debug output to track down Fregor's "failing to lift" problem.
|
| |
|
| |
|
|
|
|
|
|
|
| |
Turned up the defense against compiler exceptions in the repl. And gave
in and disabled an assertion I should have disabled two years ago.
Closes SI-4874, no review.
|
|
|
|
|
|
|
|
|
|
|
| |
Expanded the range of a warning, and made suppressed warnings visible.
Modified the positioning of "permanently hidden" errors so that when
there is more than one, the later ones are not ignored. Also changed the
error suppression code to emit the error anyway if -Ydebug was given (it
is prefixed with "[suppressed] ".) Since I can't be the only one who
wondered where his errors were vanishing to. No review.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reversed the values of "is" and "is not" in recent for comprehension
deprecation.
DO NOT BLOW HATCH
REPEAT
DO NOT BLOW HATCH
"Roger! Hatch blown."
Events reveal it was all baby, no bathwater. It turns out that the
specification is merely a document, not infallible holy writ as we
had all previously believed. So it is not the ABSENCE of val in a for
comprehension assignment which is deprecated, it is the PRESENCE of val.
Summarizing again, more accurately perhaps:
for (x <- 1 to 5 ; y = x) yield x+y // THAT's the one
for (val x <- 1 to 5 ; y = x) yield x+y // fail
for (val x <- 1 to 5 ; val y = x) yield x+y // fail
for (x <- 1 to 5 ; val y = x) yield x+y // deprecated
No review.
|
|
|
|
|
|
| |
Fix java signature generation for traits: no classes as parents. Closes
SI-4891, review by grek.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Don't discard deprecation/unchecked warnings regardless of settings.
Changed warnings code to accumulate them rather than thoughtlessly
discarding them and issuing its well-known taunt. In the repl you can
take advantage of this with the :warnings command, which will show the
suppressed warnings from the last line which had any. Be advised that at
the moment it has some issues: unchecked warnings aren't making it out,
and near repl startup neither are deprecation warnings, so don't open a
bunch of tickets please.
References SI-4594, no review.
|
|
|
|
|
|
| |
Big cleanups in the tracing utility and its usage for signature tracing.
To assist in my current java signature thrill-o-rama. No review.
|
|
|
|
|
|
|
|
|
| |
Looks like there was a little baby in the parser bathwater, or at least
some bathwater still in the general shape of a baby. Apparently both
old forms weren't deprecated. The spec does not suggest that the 'val'
in for comprehension assignment is optional, but clearly it can't go
anywhere for a while yet regardless. Will seek clarification. No review.
|
|
|
|
|
|
|
|
|
|
|
| |
Made the thread behavior of the repl a little bit configurable. Every
line will run in the same thread:
scala -Dscala.repl.no-threads
This mechanism is likely to change. Repl is obscenely overdue for a
config file. No review.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When I tried to remove the old for comprehension syntax in r24958, I
managed to instead only remove the deprecation warnings. No wonder it
went so smoothly. Sorry to get your hopes up only to execute you, old
syntax. Now the for comprehensions have to get it right:
for (x <- 1 to 5 ; y = x) yield x+y // nope
for (val x <- 1 to 5 ; y = x) yield x+y // nope
for (val x <- 1 to 5 ; val y = x) yield x+y // nope
for (x <- 1 to 5 ; val y = x) yield x+y // that's the one
No review.
|
|
|
|
|
|
|
| |
Fixing all the tests and source which still use the old for
comprehension syntax with vals where there are no vals and no vals where
there are vals. No review.
|
|
|
|
|
|
|
|
|
|
| |
Fix for a bug in CharArrayReader which made triple quoted strings
fail to parse sometimes. Note: when the temptation strikes to adjust
for special cases by letting the regular case happen and subsequently
attempting to fix the ball of mutation by selectively applying what
seems like the inverse operation, please consider the possibility that
this is not the optimal approach. Closes SI-4785, no review.
|
|
|
|
|
|
| |
Better error message for case class/object match confusion. Closes
SI-4879, no review.
|
|
|
|
|
|
| |
Avoid some spurious errors after a cyclical reference error. Closes
SI-2388, no review.
|
|
|
|
|
|
| |
Improved structural type error messages, and other error message related
boosts. Closes SI-4877, review by odersky.
|
|
|
|
|
|
|
|
| |
Fixed bug in the disambiguation of f(foo='bar') style method calls in
the presence of overloading, parameterization, and by-name arguments.
Took the opportunity to clean things up a little bit. Closes SI-4592,
review by rytz.
|
|
|
|
|
|
|
| |
Disallowed implicit modifier on auxiliary constructors, since it either
silently accomplishes nothing or crashes the compiler. If it should do
something useful let me know. Closes SI-4882, review by odersky.
|
|
|
|
|
|
|
|
| |
Don't want to chase NPEs around for the rest of my life. Created
"NoCompilationUnit" and "NoSourceFile" objects to represent not-present
versions of these items. Seems a lot better than null. References
SI-4859, got past NPE only to uncover the actual problem. No review.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Attacked classpaths to get "." off of it when it's not actually
specified. The commit makes me nervous, but there's no invisible way to
fix something like this.
** Attention, this commit changes classpath handling **
We desperately need some way of testing that the classpath has certain
qualities and does not have others; partest is not that way. Closes
SI-4857, no review.
|
| |
|
| |
|
|
|
|
|
| |
Oops, let unnecessary boxing creep into that last commit, no review.
|
|
|
|
|
|
|
| |
Rewrote the case class synthetic equals method to be more efficient and
to cause fewer problems for compiler hackers who are always saying stuff
like "the only place this comes up is case class equals..." No review.
|
|
|
|
|
|
|
|
|
| |
Reverting the inline exception handlers while I figure out what is
breaking the bytecode. (doesn't seem obvious, might take some time to
figure it out)
No review.
|
| |
|
|
|
|
|
| |
Cleanups in synthetic method generation, no review.
|
|
|
|
|
| |
Issue a warning about classes/objects inside package objects, no review.
|
|
|
|
|
|
|
|
| |
Cleanups in Namers and AddInterfaces emerging from bugfixing attempts
and comprehension pursuits. I appear to have accidentally fixed at least
one bug, as there are new (correct) warnings when building the compiler
involving permanently hidden imports. No review.
|
|
|
|
|
|
|
| |
Adds version information from @migration and @deprecated to ScalaDoc.
Adds support for @migration in Symbols and MemberEntity. Contributed by
Simon Ochsenreither. Review by heathermiller.
|