| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
| |
Fixed a bug where newly introduced type parameters didn't have all the
substitutions done correctly. Fixes #4266.
No review.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Moved unlift to the Function companion object, which might have been
better in the first place. Had to make a minor change to genprod, and
then I couldn't escape that unscathed. Finished the not very complete
undertaking I found there to update the scaladoc. Lots of little changes
to the generated text and code.
I changed genprod to only put a unique stamp on Function0 so we can stop
having a 100 file diff everytime an i is dotted somewhere.
Closes #3825, no review.
|
|
|
|
|
|
|
|
| |
Observed that some historical issues with package objects no longer seem
so issuey. In the interests of keeping the arbitrary limitations to a
minimum, re-enabled case classes in package objects (see #2130, #3437.)
Closes #3437, review by odersky.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Based on the frequency with which I hear questions about it and similar,
this error message assumes too much understanding.
scala> scala.collection.mutable.MultiMap(1, 2, 3) <console>:8: error:
value MultiMap is not a member of package scala.collection.mutable
Now it says:
scala> scala.collection.mutable.MultiMap(1, 2, 3) <console>:8: error:
object MultiMap is not a member of package scala.collection.mutable
Note: trait MultiMap exists, but it has no companion object.
No review.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rediscovering that transpose sometimes throws an exception on irregular
lists and sometimes returns an irregular result (depending on whether
the first collection is the longest or not) indicated that this needs a
better resolution than the status quo. Determination: as long as we're
throwing an exception on any invalid input, we should throw an exception
on all invalid input, so that's what it does now. I postpone any attempt
to offer a variation accepting a "hole value". Closes #3597, review by
community.
|
| |
|
| |
|
|
|
|
|
| |
No review.
|
|
|
|
|
|
|
|
| |
Revert "Added missing string prefixes for names of map and set
collection classes." and related commits.
No review.
|
|
|
|
|
|
|
|
| |
Another fix for a broken test case involving new set and map string
reps.
No review.
|
|
|
|
|
| |
Fixing tests depending on map and set string representations. No review.
|
|
|
|
|
| |
No review.
|
| |
|
|
|
|
|
|
|
|
| |
Added new setting -Ygen-javap, which takes a directory as an argument
and generates javap output for every generated classfile. There is still
a lot of noise in a diff: still todo is postprocessing the output to
remove diff-noise-generating numbers and such. No review.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I chased a lot of ghosts before finding the real culprit for why partest
failures have been unfailing.
Process(Seq("bash", "-c", "exit 42")) ! // 42 Process(Seq("bash",
"-c", "exit 42")) #> logFile ! // 0
That behavior is not yet fixed, but I altered partest not to use #> and
fixed the test which should have been failing but wasn't. Closes #4227,
no review.
|
|
|
|
|
|
|
|
| |
It seems to arise from the changes to inner objects which make them look
like lazy vals. I am not sure this is the right change but at least
it's only about a six character diff to review. Closes #4188, review by
odersky.
|
|
|
|
|
|
|
|
|
| |
Created invisible setting to expose empty package members outside of the
empty package, so the repl can use packages without going blind to the
empty package. This commit also eliminates a deadlock which hits when
the compiler starts logging before a lazy val has pulled its head from
the noose. Closes #4228, review by odersky.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
reinstate the disabled test.
COMMITTERS: the test is very fragile. If you don't run the whole test
suite (it's the very last one which runs!) then you broke the build.
Upon your toes you should be.
For anyone unaware, test/partest --update-check path/to/test will
magically pass any test and update the checkfile to resemble whatever
output it just produced. Incautious use of this feature may negatively
impact the mood of certain extempores.
No review.
|
|
|
|
|
|
|
|
|
|
| |
After seeing the 481 line diff I had to apply to bring the checkfile up
to date, I concluded this test cannot be in the general pool: this is
the third build break in the last few days, because it changes output
like some people change socks. (Hygenic people.) Sorry to move it to
disabled iulian, but I don't know what else to do with it on short
notice. Review by dragos.
|
|
|
|
|
| |
No review
|
|
|
|
|
|
| |
Applied a few more fixes for specialization and added a new instrumented
jar. Review by dragos.
|
| |
|
| |
|
|
|
|
|
|
| |
Improved an error message involving overload+default args. Closes #3909,
no review.
|
|
|
|
|
|
| |
Some cleanup from investigating #4041, with a comment instead of a fix
for the ticket. Review by rytz in case he sees a good way to fix it.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Added all the javap command line options to :javap. No review.
|
|
|
|
|
|
|
|
|
|
|
| |
Is this symbol defined in the same scope
and compilation unit as `that' symbol?
But "same scope" was never checked, only "same compilation unit."
Presumably other layers of logic kept this from being noticed until now,
but it has been crashing sbt. Added check to isCoDefinedWith. Closes
#4220, review by odersky.
|
|
|
|
|
|
| |
Fix for parser regression allowing 0-case match blocks. Closes #4217, no
review.
|
|
|
|
|
|
| |
anything fail. This test might be a bit overly output-sensitive to be in
the general pool... no review.
|
|
|
|
|
|
| |
Re-enabled test spec-traits (removed the swing dependent part, since all
it tested was call-by-name parameters). no review.
|
|
|
|
|
|
|
|
| |
After discovering #3376 was fixed I gave in and tried to write a test.
Now that the fiddling is over you can write repl tests without creating
5000 streams and settings. Look at test/files/run/bug3376.scala or
jvm/interpreter.scala to see. Test case closes #3376, no review.
|
|
|
|
|
|
|
|
| |
Switched back to StoreReporter for presentation compiler tests. This
eats up error messages, but if the output is correct we don't mind a
spurious error here and there. Renabled simple presentation compiler
tests. no review.
|
|
|
|
|
|
|
|
|
| |
I keep seeing huge simplifications available in the repl. Lot of
interesting features have come to town since it was first conceived. In
this commit I give the internal names some tips on aesthetics, and put
each line in its own package like civilized artificial constructs. No
review.
|
|
|
|
|
|
|
|
| |
I found more situations where primitive types were appearing in
signatures. I hacked in a flag which says boxing is or is not allowed.
The code is getting hideous: foortunately I have it mostly rewritten,
but I wanted to fix this bug now. No review.
|
|
|
|
|
|
|
|
|
|
|
|
| |
up the stray interpreter files and put them in the interpreter package.
Would really love to change the name of that package. Went looking for
some consistent divisions of responsibility and consistent naming.
Made some progress. There are deprecated versions of most everything I
changed so hopefully the carnage will be limited.
This isn't completely baked but I just realized I broke the build
earlier and this should fix it. I'll keep the oven on. No review.
|
|
|
|
|
|
| |
Fixed some generic signature bugs I found thanks to the compiler telling
me it was buggy. Thanks compiler. No review.
|
|
|
|
|
|
| |
A raw tree was making its way into an error message. Removed. Closes
#4196, no review.
|
|
|
|
|
| |
Forgot to commit the updated tests in my previous commit. no review.
|
| |
|
|
|
|
|
| |
Some overdue organization and cleanups in the repl. No review.
|
| |
|
|
|
|
|
|
| |
One last attempt at getting the presentation compiler pass some tests.
no review.
|