summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #558 from lrytz/wip/t5259Josh Suereth2012-05-162-6/+27
|\ | | | | Fix SI-5259
| * Fix SI-5259Lukas Rytz2012-05-152-6/+27
| | | | | | | | Calling the type checker on an Ident tree instead of using gen.mkAttributedRef assigns a SingleType to the tree.
* | Merge pull request #557 from lrytz/wip/t5610Josh Suereth2012-05-163-5/+47
|\ \ | | | | | | Fix for SI-5610
| * | Fix for SI-5610Lukas Rytz2012-05-153-5/+47
| |/
* | Merge pull request #555 from srp/masterJosh Suereth2012-05-161-0/+22
|\ \ | | | | | | mutable.MapLike: override $mapNote to reflect actual require mutable api
| * | mutable.MapLike: override $mapNote to reflect actual require mutable apiScott R. Parish2012-05-151-0/+22
| | | | | | | | | | | | | | | | | | | | | The api needed to define a mutable.Map is different then the one needed to define an immutable.Map. Prior to this patch the mutable one reflected the api needed for the immutable one causing confusion about what really needed to be defined.
* | | Merge pull request #553 from heathermiller/doc/linksJosh Suereth2012-05-1616-30/+27
|\ \ \ | | | | | | | | Small documentation fixes & small fix to Scaladoc @see formatting
| * | | Corrects links in API documentationHeather Miller2012-05-1611-18/+16
| | | |
| * | | Missed a straggling doc comment.Heather Miller2012-05-151-1/+1
| | | |
| * | | Small documentation fixes & small fix to Scaladoc @see formattingHeather Miller2012-05-155-11/+10
| | | |
* | | | Putting back things sbt noticed were gone.Paul Phillips2012-05-153-0/+30
| | | |
* | | | Removing more unneeded code.Paul Phillips2012-05-1544-1432/+128
| |/ / |/| |
* | | Test adjustments.Paul Phillips2012-05-152-1/+4
| | |
* | | Removing extraneous files.Paul Phillips2012-05-1593-6478/+56
| | | | | | | | | | | | Culling accumulated unnecessary code.
* | | Merge pull request #554 from srp/masterJosh Suereth2012-05-151-2/+2
|\ \ \ | | | | | | | | Enumeration#maxId: fix documentation to reflect reality
| * | | Enumeration#maxId: fix documentation to reflect realityScott R. Parish2012-05-151-2/+2
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | maxId is not really the "maximum id", but rather one past the max: scala> object Foo extends Enumeration { | val x = Value | val y = Value | } defined module Foo scala> Foo.maxId res0: Int = 2 scala> Foo(2) java.util.NoSuchElementException: key not found: 2 at scala.collection.MapLike$class.default(MapLike.scala:225) ... scala> Foo(1) res2: Foo.Value = y
* / / Fixed positions for `AnnotationInfo.original`.Iulian Dragos2012-05-151-1/+5
|/ /
* | Tweak to avoid value classes in modifier check.Paul Phillips2012-05-141-1/+1
| |
* | New starr with adriaan's fix from 1b198fadd1 .Paul Phillips2012-05-143-3/+3
| | | | | | | | Since we still fail in locker.comp.
* | Debugging output tweaks.Paul Phillips2012-05-146-27/+36
|/ | | | And undeprecated Positional.
* No-op changes.Paul Phillips2012-05-143-6/+3
| | | | | Fixed a comment and simplified a mysterious "if (x) y else y" method thanks to the helpful jibe received in comments.
*-------. Merge commit 'refs/pull/547/head'; commit 'refs/pull/548/head'; commit ↵Paul Phillips2012-05-1415-49/+122
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 'refs/pull/549/head'; commit 'refs/pull/550/head'; commit 'refs/pull/551/head' into develop
| | | | | * Better fix for SI-5676. Review by @paulpHubert Plociniczak2012-05-146-30/+35
| | | | | |
| | | | * | Closes SI-5796.Hubert Plociniczak2012-05-142-2/+11
| | | | |/
| | | * | fix SI-5384Lukas Rytz2012-05-142-2/+29
| | | | | | | | | | | | | | | | | | | | make TreeInfo recognize constructor calls after named arguments transformation.
| | * | | suspend type vars in SubTypePair's equalsAdriaan Moors2012-05-141-4/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SubTypePair's equals method calls =:= on the involved types, which mutates the TypeVars contained in them this is undesirable since we're simply checking whether a subtype test is pending in addition to making subtyping "more correct" for type vars, it should avoid the stackoverflow that's been plaguing us (https://groups.google.com/d/topic/scala-internals/2gHzNjtB4xA/discussion) SubTypePair's equals method method is only called when subtype checking hits a recursion threshold (subsametypeRecursions >= LogPendingSubTypesThreshold)
| * | | | Added test case for commit f7d5f45 (re SI-5552)Erik Osheim2012-05-142-0/+12
| | | | |
| * | | | Specialize lazy vals (closes SI-5552)Erik Osheim2012-05-142-11/+14
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, specialized lazy vals would not work at all when used in specialized classes, and would just return an uninitialized value. After this patch, they work in the same way as other specialized fields do (i.e. a new specialized field is created, and the specialized class uses that instead of the base class' field). Note that there are still known problems with specialized lazy vals (for instance SI-4717) but it seemed to me that this was better than nothing.
* / / / Removed an unnecessary self type.Paul Phillips2012-05-141-2/+0
|/ / /
| | |
| \ \
*-. \ \ Merge commit 'refs/pull/543/head'; commit 'refs/pull/544/head'; commit ↵Paul Phillips2012-05-13135-514/+640
|\ \ \ \ | | | | | | | | | | | | | | | 'refs/pull/546/head' into develop
| | * | | Test case closes SI-5137.Jason Zaugg2012-05-131-0/+17
| | | | | | | | | | | | | | | | | | | | virtpatmat strikes again.
| * | | | Address doc comment rot in the standard library.Jason Zaugg2012-05-1394-370/+204
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Match @param/@tparam names to the actual parameter name - Use @tparam for type parameters - Whitespace is required between `*` and `@` - Fix incorrect references to @define macros. - Use of monospace `` and {{{}}} (much more needed) - Remove `@param p1 ...` stubs, which appear in the generated docss. - But, retainsed `@param p1` stubs, assuming they will be filtered from the generated docs by SI-5795. - Avoid use of the shorthand `@param doc for the solitary param` (which works, but isn't recognized by the code inspection in IntelliJ I used to sweep through the problems) The remaining warnings from `ant docs` seem spurious, I suspect they are an unintended consequence of documenting extension methods. [scaladoc] /Users/jason/code/scala/src/library/scala/collection/TraversableOnce.scala:181: warning: Variable coll undefined in comment for method reduceOption in class Tuple2Zipped [scaladoc] def reduceOption[A1 >: A](op: (A1, A1) => A1): Option[A1] = reduceLeftOption(op) [scaladoc] ^
| * | | Tweak to the runner to deal with the big bad world.Paul Phillips2012-05-131-2/+11
| | | | | | | | | | | | | | | | | | | | Relative symlinks which start with .. are then run with an absolute path. It's robustness city.
| * | | Deprecated some classes.Paul Phillips2012-05-1218-2/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All this stuff. I believe most of this is uncontroversial. scala/util/Marshal.scala scala/util/automata/BaseBerrySethi.scala scala/util/automata/DetWordAutom.scala scala/util/automata/Inclusion.scala scala/util/automata/NondetWordAutom.scala scala/util/automata/SubsetConstruction.scala scala/util/automata/WordBerrySethi.scala scala/util/grammar/HedgeRHS.scala scala/util/grammar/TreeRHS.scala scala/util/parsing/ast/AbstractSyntax.scala scala/util/parsing/ast/Binders.scala scala/util/parsing/combinator/testing/RegexTest.scala scala/util/parsing/combinator/testing/Tester.scala scala/util/parsing/input/Positional.scala scala/util/regexp/Base.scala scala/util/regexp/PointedHedgeExp.scala scala/util/regexp/SyntaxError.scala scala/util/regexp/WordExp.scala I'd have deprecated much of it long ago if the compiler didn't still depend on it due to xml/dtd/something. And it still does, but it's time to deprecate them anyway.
| * | | Merge branch 'master' of https://github.com/scala/scalaPaul Phillips2012-05-125-4/+47
| |\| |
| | * | Test case closes SI-5037.Paul Phillips2012-05-122-0/+20
| | | |
| | * | Clutch modification to tree printing.Paul Phillips2012-05-123-4/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't print trees under -Xprint:all if they're identical to the tree printed at the previous phase. It only works for a single compilation unit but that is a huge step forward for us debuggers. For instance this file: trait Foo { def f = 5 } used to produce 332 lines of output and now produces 92, with zero loss of information. It ends with: [[syntax trees at end of cleanup]] // a.scala: tree is unchanged since mixin [[syntax trees at end of icode]] // a.scala: tree is unchanged since mixin [[syntax trees at end of inliner]] // a.scala: tree is unchanged since mixin [[syntax trees at end of inlineExceptionHandlers]] // a.scala: tree is unchanged since mixin [[syntax trees at end of closelim]] // a.scala: tree is unchanged since mixin [[syntax trees at end of dce]] // a.scala: tree is unchanged since mixin [[syntax trees at end of jvm]] // a.scala: tree is unchanged since mixin
| | | |
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| | \ \
| *-------------. \ \ Merge remote-tracking branches 'retronym/ticket/5407', ↵Paul Phillips2012-05-1220-136/+333
| |\ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | 'scalamacros/topic/macrocherrypick', 'lrytz/t5626', 'lrytz/t5009', 'retronym/ticket/5029', 'retronym/ticket/4025', 'retronym/topic/quieter-nsdhnao' and 'retronym/ticket/1133' into develop
| | | | | | | | | * | Confirm extractor based pattern matches don't explode exponentially.Jason Zaugg2012-05-121-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes SI-1133, thanks virtpatmat.
| | | | | | | | * | | NSDNHAO need not tell us in which version it resides.Jason Zaugg2012-05-121-1/+1
| | | | | | | | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is done for all crashers now. Furthermore, that error report makes it pretty clear that it's a compiler bug, not user error, so "this is a bug" seem like needless words. Resisted the nostalgic temptation to remove the "an".
| | | | | | | * / / A REPL pattern matching crasher that crashes no more.Jason Zaugg2012-05-122-0/+31
| | | | | | | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Not due to virtpatmat, mind you; it passes with -Xoldpatmat. Closes SI-4025.
| | | | | | * / / Another bug bites virtpatmat's dust.Jason Zaugg2012-05-122-0/+4
| | | | | | |/ / | | | | | | | | | | | | | | | | | | | | | | | | Test case closes SI-5029.
| | | | | * | | Fix SI-5009: case-class copy method now eta-expands over higher parameter lists.Lukas Rytz2012-05-116-10/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Example: Given case class C(a: Int)(b: Int) if you call (new C(1)(2)).copy(a = 10)), you get a function (f: Int => C) such that (f.apply(20)) yields a (new C(10)(20)).
| | | | * | | | Fix SI-5626.Lukas Rytz2012-05-112-2/+29
| | | | |/ / / | | | | | | | | | | | | | | | | | | | | | By not replacing 'CaseClass.apply()' factor by 'new CaseClass()' when the class type 'CaseClass' is not accessible.
| | | * | | | cherry-picking important macro fixes from topic/reflectionEugene Burmako2012-05-124-123/+174
| | | | | | |
| | * | | | | Test case closes SI-5407Jason Zaugg2012-05-122-0/+19
| | | |_|/ / | | |/| | | | | | | | | | | | | | | It still breaks as described in the ticket under -Xoldpatmat.
* | / | | | Test for SI-5394.Simon Ochsenreither2012-05-121-0/+4
|/ / / / /
| | | | |
| \ \ \ \
| \ \ \ \
| \ \ \ \
| \ \ \ \
| \ \ \ \
*-----. \ \ \ \ Merge commit 'refs/pull/530/head'; commit 'refs/pull/531/head'; commit ↵Paul Phillips2012-05-1171-343/+632
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 'refs/pull/532/head'; commit 'refs/pull/533/head'; commit 'refs/pull/534/head' into develop
| | | | * | | | | Test case closes SI-4124.Jason Zaugg2012-05-122-0/+28
| | | | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | This looks like a job for... virtpatmat!
| | | | * | | | Test case closes SI-5165.Paul Phillips2012-05-113-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also fixed by annotation/enum commit.