| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
| |
Thrusting spear into darkened alcove attempting to slay java5 build
failure. No review.
|
| |
|
|
|
| |
truth of theory. No review.
|
| |
|
|
|
|
|
|
|
| |
Everything builds for me, but apparently not for jenkins. First attempt
to solve mystery: explicitly set return type of Any#getClass() to
Class[_ <: Any] rather than allowing java's to be used. I'm guessing
that somehow it materializes as Class[_ <: Any] sometimes and Class[_ <:
AnyRef] other times. Review by moors.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A total rewrite of "runtimeClass", discarding the user-space approach in
favor of simply fixing getClass.
def f1 = 5.getClass // Class[Int]
def f2 = (5: AnyVal).getClass // Class[_ <: AnyVal]
def f3 = (5: java.lang.Integer).getClass // Class[_ <: java.lang.Integer]
class A
class B extends A
def f1 = (new B: Any).getClass().newInstance() // Any
def f2 = (new B: AnyRef).getClass().newInstance() // AnyRef
def f3 = (new B: A).getClass().newInstance() // A
def f4 = (new B: B).getClass().newInstance() // B
But that's not all!
def f0[T >: B] = (new B: T).getClass().newInstance()
def f5 = f0[Any] // Any
def f6 = f0[AnyRef] // AnyRef
def f7 = f0[A] // A
def f8 = f0[B] // B
Closes #490, #896, #4696. Review by moors. (Note: I think this is pretty
good, but picky review requested.)
|
| |
|
|
|
|
|
| |
Removing the package-private modifiers on Gen*Like traits.
No review.
|
| |
|
|
|
|
|
|
| |
Removed "isInterpreterWrapper" from the conditions which imply an
effective root. This was an old hack from before my time which is no
longer necessary and then recently became actively hostile. Closes
#4710, no review.
|
| |
|
|
|
|
|
|
| |
Relaxes a typer check which fails valid code with NoCommonType. If the
instantiated types or type bounds do not conform, it tries normalizing
the type before throwing the exception. Closes #4553. I wrote this patch
with adriaan already, but bonus review by moors.
|
| |
|
|
|
|
| |
Documented what happened with the implementation of narrow for future
performance detectives. No review.
|
| |
|
|
|
|
| |
Routed Stream.+: through cons to avoid some truly pathological
performance. Closes #4697, no review.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
Performance-motivated work: going through SpecializeTypes and
eliminating unnecessary intermediate data structures and multiple
traversals of lists. Also trying to get that code under control: it
dwarfs all other phases in terms of debugging output, and a too-large
percentage of the source is commented out debugging code which looks
past its sell-by date. I realize the patch is a little big for a very
thorough review, but review by dragos, prokopec.
|
| |
|
|
|
|
| |
Quoted the windows runner path so "c:\program files (x86)" has some
chance of working. Closes #4636, no review.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
| |
Reverts r25085 ("Performance tweaks for AbstractFile, no review.") which
apparently is making the IDE refuse to overwrite classfiles. For the
life of me I can't find why there should be any difference. Bowing to
expedience, no review.
|
| |
|
|
|
|
|
| |
When TypeVars were given higher-order abilities, so too should have been
WildcardType, which acts as a plceholder for typevars. Always inflicting
arguments upon it was the cause of #2308. Closes #2308, review by moors.
|
| | |
|
| |
|
|
|
|
|
|
| |
Fixed an overflow which occurs in hashtable size computations. Fixes
#4678.
No review.
|
| | |
|
| |
|
|
|
|
|
|
| |
Fixed an issue with higher kinded type inference on constructors without
any help from adriaan. Provisionally notched belt. Wrapped up as many
tickets as I added characters of code. Closes SI-3343, SI-4018. Review
by moors.
|
| |
|
|
|
|
| |
Formatting fixes for scala.collection.JavaConverters. Contributed by
Daniel Capo Sobral. Review by dubochet.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
Fixes typo in ScalaDoc: IndexOutofBoundsException =>
IndexOutOfBoundsException. Contributed by Simon Ochsenreither. Review by
dubochet.
|
| |
|
|
|
|
| |
Took a hint from -Xcheckinit to make some pattern matcher code
init-order consistent. No review.
|
| |
|
|
|
|
| |
Go back to printing the message that transcript pastes can be finished
with ctrl-D. No review.
|
| |
|
|
|
|
|
|
| |
Deprecated scala.reflect.generic, except for ByteCodecs. After we move
some non-internal things out of scala.reflect.internal we should go back
and improve some deprecation messages (i.e. point somewhere useful.) No
review.
|
| |
|
|
|
|
| |
Suppressing the ability to suppress stack trace suppression in
NonLocalReturnControl for performance reasons, no review.
|
| |
|
|
|
| |
Fix for that slightly not compiling previous patch, no review.
|
| | |
|
| |
|
|
|
| |
Eliminating accumulated dead ends from the pattern matcher. No review.
|
| |
|
|
|
|
|
| |
Triumph over various unsolved mysteries of the pattern matcher. More
elimination of redundant ways of doing things. More allowing the typer
to do its typing thing. No review.
|
| |
|
|
|
|
|
| |
Bytecode inspection reveals the pattern matcher leaving a lot more
redundancy in generated trees than is necessary. Added more inteligence
to the process. Review by dragos.
|
| |
|
|
|
|
|
|
|
|
|
| |
closes #4692: unification in type constructor inference now widens *and*
dealiases when necessary
in 2.8.1 implicit conversion search started with a widened type, so that
combo never came up
no review
|
| |
|
|
|
|
| |
got rid of grotesque isCompat closure in exprTypeArgs and friends. no
review
|
| | |
|
| |
|
|
|
|
|
|
| |
Update manually created tree which was calling Predef.error to call
scala.sys.error instead. Created convenience functions in Definitions
for getting package objects akin to those for getting classes and
objects. No review.
|
| |
|
|
|
|
|
|
|
| |
More batched performance improvements for io.{ File, Classpath } and
others in the neighborhood. Avoids calling the expensive
getCanonicalPath in favor of getAbsolutePath: I note that because
it has the potential to change compiler behavior at the borders.
No review.
|
| |
|
|
|
| |
review by rompf -- odersky may want to take a quick look and update the spec
|
| |
|
|
|
|
|
| |
Documented what I could figure out about late flags and anti-flags.
Eliminated some indirection in flag usage. Made -Xshow-phases print out
more info about flags if -Ydebug is also given. No review.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Views using methods implemented in terms of isEmpty (in particular,
headOption and lastOption) were traversing the collection twice up
to the nonEmpty element, because "if (isEmpty) None else Some(head)"
means calling isEmpty separately from head. I overrode those methods in
TraversableViewLike to avoid the second traversal.
This leaves at least init and tail still in that boat, but they were
getting too involved.
How do I say "review by pool of reviewers", who can help set that up? In
the meantime no review.
|
| |
|
|
|
|
|
| |
Marked some more by-name-argument-using methods with final @inline to
make possible and then encourage their inlining. Knocked almost 300 more
classfiles and 200K off the compiler jar. No review.
|
