summaryrefslogtreecommitdiff
path: root/src/library
Commit message (Collapse)AuthorAgeFilesLines
...
* | 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
* Debugging output tweaks.Paul Phillips2012-05-141-1/+0
| | | | And undeprecated Positional.
* 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] ^
* 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 commit 'refs/pull/530/head'; commit 'refs/pull/531/head'; commit ↵Paul Phillips2012-05-116-42/+28
|\ | | | | | | 'refs/pull/532/head'; commit 'refs/pull/533/head'; commit 'refs/pull/534/head' into develop
| * Move resolver and resolveEither to impl.Promisephaller2012-05-106-42/+28
| |
| * Revert "Fix for ## inconsistency."Paul Phillips2012-05-101-7/+4
| | | | | | | | | | | | This reverts commit 58bb2d1bd2000ac3aa2c64b6c5dc56c91e911860. I guess this must be what's failing the nightly.
* | Fixes SI-5640Dominik Gruntz2012-05-102-8/+8
|/
* Custom hashCode methods for case classes.Paul Phillips2012-05-091-0/+89
| | | | No boxing, no MODULE$ indirection.
* Revert recent commits.Paul Phillips2012-05-092-28/+63
| | | | | This reverts commit 9b6f51d3ae6ddc6571d3101ea715e25a05aa8adb. This reverts commit b5919100e785df58bde35bb24abe9d60b4da08a2.
*---. Merge commit 'refs/pull/517/head'; commit 'refs/pull/518/head'; commit ↵Paul Phillips2012-05-093-68/+37
|\ \ \ | | | | | | | | | | | | 'refs/pull/519/head'; commit 'refs/pull/520/head' into develop
| | | * fixed typoDominik Gruntz2012-05-091-1/+1
| | | |
| | | * removes redundant hash implementation from BoxesRunTime.javaDominik Gruntz2012-05-092-63/+28
| | | |
| | | * Fix for ## inconsistency.Paul Phillips2012-05-081-4/+7
| | | | | | | | | | | | | | | | Hopefully without slowing things down overmuch. Closes SI-5640.
| * | | Fixes SI-5328. Iterator.patched failed when from=0.Josh Suereth2012-05-081-1/+2
| | |/ | |/| | | | | | | | | | | | | It turns out iterator.patched forgot to drop replacement values if they were at the beginning. This is because the index was advancing before checking to see if replaced elements should be dropped. Moved this behavior to the beginning of next.
* / | Fixes SI-5201. Adds flatten to TraversableViewLike.Josh Suereth2012-05-081-0/+2
|/ / | | | | | | | | | | | | | | | | Note: This commit exposes a pretty rich type on flatten in views. HOWEVER, because we don't capture the higher kinded type of the underlying collection, it makes returning a more minimal type pretty dang hard. I can imagine a very breaking and painful change of capturing the underling collection as a higher-kinded type as well as the current view type in a *ViewLike.scala. I hope this kind of issue, along with others, drives a rethink of our view API design.
| |
| \
| \
| \
*---. \ Merge commit 'refs/pull/504/head'; commit 'refs/pull/505/head'; commit ↵Paul Phillips2012-05-08155-815/+1073
|\ \ \ \ | | |_|/ | |/| | | | | | 'refs/pull/506/head'; commit 'refs/pull/507/head'; commit 'refs/pull/508/head' into develop
| | | * Changes scala.math.signum to return negative zeros.Dominik Gruntz2012-05-081-14/+3
| | |/ | |/|
| | * Part I of the Lazy Vals Saga: Bitmaps are no longer inherited.Hubert Plociniczak2012-05-032-2/+0
| | |
| | |
| | \
| *-. \ Merge commit 'refs/pull/486/head'; commit 'refs/pull/487/head'; commit ↵Paul Phillips2012-05-071-3/+29
| |\ \ \ | | | | | | | | | | | | | | | 'refs/pull/488/head'; commit 'refs/pull/489/head'; commit 'refs/pull/490/head' into develop
| | * | | Fixes SI-5199 through improved API docsHeather Miller2012-05-071-3/+29
| | |/ /
| * | | Merge commit 'refs/pull/479/head'; commit 'refs/pull/480/head'; commit ↵Paul Phillips2012-05-064-10/+60
| |\ \ \ | | |/ / | |/| | | | | | 'refs/pull/481/head'; commit 'refs/pull/482/head'; commit 'refs/pull/483/head'; commit 'refs/pull/484/head'; commit 'refs/pull/485/head' into develop
| | * | Add missing methods to GenTraversableLike.Aleksandar2012-05-064-10/+60
| | | |
| | | |
| | \ \
| *-. \ \ Merge commit 'refs/pull/477/head'; commit 'refs/pull/478/head' into developPaul Phillips2012-05-051-7/+9
| |\ \ \ \
| | | * | | Don't admit primitive arrays as a generic Java varargs param.Jason Zaugg2012-05-051-7/+9
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | They were sneaking through as polymorphic Arrays and avoiding boxing. Closes SI-4216
| * | | | Merge commit 'refs/pull/124/head' into developPaul Phillips2012-05-051-9/+10
| |\ \ \ \ | | |/ / / | |/| | | | | | | | | | | | | Conflicts: src/library/scala/util/parsing/combinator/Parsers.scala
| | * | | Fixes SI-4929, with a test to verify.Stephen Judkins2012-01-211-14/+15
| | | | | | | | | | | | | | | | | | | | Also fixes potential issue with Parsers.phrase not being reentrant; however, I was unable to actually reproduce this issue in practice. (The order in which lastNoSuccess was being set and compared seemed to guarantee that it would never actually be a problem).
| * | | | Working around the VerifyErrors.Paul Phillips2012-05-042-2/+6
| | | | | | | | | | | | | | | | | | | | Since the optimizer seems to have lost its mind.
| * | | | Put corresponds on Iterator.Paul Phillips2012-05-041-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I guess we can look forward to more and more duplication in the collections. I had to cut and paste the doc comment too, or at least I have no idea how to avoid that given the file I took it from was in parallel somewhere.
| * | | | Have ArrayCharSequence reuse its Array.Paul Phillips2012-05-041-5/+20
| | | | | | | | | | | | | | | | | | | | Your know, for performance. Closes SI-5641.
| * | | | Updated Symbol to record classfile origin.Paul Phillips2012-05-041-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change should be transparent to anything using sourceFile, unless it was drinking from the inheritance well too deeply. Rather than squander the already allocated field for every ClassSymbol not being compiled from source, I'm now populating it with the file representing the class. This will make a broad range of things easier, like debugging, issuing useful error messages, symbol invalidation, signature verification, you name it. def sourceFile - still returns only source code files def binaryFile - returns only class files def associatedFile - returns whatever is there, if anything Performance: I may be mistaken, but I believe this is a zero-impact change. No new fields are allocated; fields which were null now hold a useful reference. The reference is to a file instance which was already being allocated and already long-lived. Compare error messages: // Version 1 % scalac a.scala error: type _$1 is defined twice // Version 2 % scalac a.scala error: type _$1 is defined twice conflicting symbols both originated in file './foo/package.class' Note: this may be due to a bug in the compiler involving wildcards in package objects one error found Bonus for people who read commit logs. Try this in the repl after starting power mode. ListClass.info.members groupBy (_.associatedFile) foreach { case (k, vs) => println("%s\n %s\n".format(k, vs map (_.defString) mkString "\n ")) }
| | | | |
| | \ \ \
| | \ \ \
| | \ \ \
| | \ \ \
| | \ \ \
| | \ \ \
| | \ \ \
| *-------. \ \ \ Merge commit 'refs/pull/471/head'; commit 'refs/pull/473/head'; commit ↵Paul Phillips2012-05-047-11/+31
| |\ \ \ \ \ \ \ \ | | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | 'refs/pull/474/head'; commit 'refs/pull/475/head'; commit 'refs/pull/476/head' into develop
| | | | | | * | | Fixes SI-5514.Aleksandar Prokopec2012-05-041-2/+4
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The acceptIf and acceptMatch parsers now check for end of input. Review by moors.
| | | | | * | | Fixes si-5656.Aleksandar Prokopec2012-05-041-5/+13
| | |_|_|/ / / | |/| | | | |
| | | | * | | Fix for si-5577.Aleksandar Prokopec2012-05-041-0/+2
| | |_|/ / / | |/| | | |
| | | * | | Fix for si-5590.Aleksandar Prokopec2012-05-042-1/+9
| | |/ / / | |/| | |
| | * | | Fix for si-5018.Aleksandar Prokopec2012-05-042-3/+3
| |/ / / | | | | | | | | | | | | Methods keySet, values and withDefault now return serializable collections.
| * | | Removed BackquotedIdent.Paul Phillips2012-05-031-9/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In favor of a marker attachment using Attachment, as suggested in comments. (The Attachment interface needs work.) I did this bit trying to fix SI-5715, but it's still a bit elusive because the Ident node is thrown away as soon as there's a member definition. Also, as far as I can see this will if anything propagate the backquotedness of an identifier less effectively than before, because TreeCopiers don't copy attachments. Maybe that's on the "todo" list? The whole idea seems to depend on their being propagated to copies.
| | | |
| | \ \
| *-. \ \ Merge commit 'refs/pull/467/head'; commit 'refs/pull/468/head'; commit ↵Paul Phillips2012-05-03124-200/+272
| |\ \ \ \ | | | | | | | | | | | | | | | | | | 'refs/pull/469/head'; commit 'refs/pull/470/head' into develop
| | | * | | Eliminating reflective calls.Paul Phillips2012-05-023-2/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Frobbed knobs and made little traits until all relevant looking reflective calls were gone.
| | | * | | Unanchored regex extractors.Paul Phillips2012-05-021-7/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is really by Lanny Ripple <lanny@spotinfluence.com>, but I reworked it because I didn't want to put any more methods onto String. Instead, there is a method on Regex which removes the anchoring quality. """\d\d'"".r.unanchored
| | | | | |
| | | | \ \
| | | | \ \
| | | | \ \
| | | | \ \
| | | | \ \
| | | *-----. \ \ Merge commit 'refs/pull/457/head'; commit 'refs/pull/458/head'; commit ↵Paul Phillips2012-05-02119-184/+199
| | | |\ \ \ \ \ \ | | | |_|/ / / / / | | |/| | | | | | | | | | | | | | | 'refs/pull/459/head'; commit 'refs/pull/460/head'; commit 'refs/pull/461/head'; commit 'refs/pull/462/head'; commit 'refs/pull/463/head'; commit 'refs/pull/464/head'; commit 'refs/pull/465/head' into develop
| | | | | | * | | Fixes SI-4478.Simon Ochsenreither2012-05-02118-179/+181
| | | |_|_|/ / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Replaced/simplified usages of "wrt". - Added backticks to $Coll definitions, so stuff like "immutable.Stack" hopefully stops being interpreted as the end of a sentence and shown like that in the summary line of ScalaDoc's method description. See collection.immutable.Stack's sortBy. Additionally, it looks nicer this way. - Fixes the typo mentioned in SI-5666.
| | | | | * | | Changes scaladoc of trait DynamicDominik Gruntz2012-05-021-5/+18
| | | |_|/ / / | | |/| | | |
| | | * | | | Merge commit 'refs/pull/457/head'; commit 'refs/pull/458/head'; commit ↵Paul Phillips2012-05-022-7/+6
| | |/|\| | | | | | | | | | | | | | | | | | | | | | | | 'refs/pull/459/head'; commit 'refs/pull/460/head'; commit 'refs/pull/461/head'; commit 'refs/pull/462/head' into develop
| | | | * | | small tree attachment refactoring: firstAttachmentAdriaan Moors2012-05-021-6/+5
| | | |/ / / | | |/| | |
| | | * | | SI-5729: TypeVar experimentals iff -XexperimentalAdriaan Moors2012-05-021-1/+1
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | it used to also be enabled by -Yvirtpatmat, which is now on by default, but this type hackery is no longer necessary to bootstrap under the new pattern matching scheme, so let's only turn it on when people are feeling -Xexperimental
| * | | | Remove newPromise methodphaller2012-05-032-6/+0
| | | | |
| * | | | Resolve merge conflictsphaller2012-05-0312-143/+203
| |\| | |
| | * | | Workaround for scaladoc failure.Paul Phillips2012-05-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Addressing this little fellow: 21:40:22 [scaladoc] src/library/scala/collection/TraversableOnce.scala:382: error: missing parameter type for expanded function 21:40:22 [scaladoc] The argument types of an anonymous function must be fully known. (SLS 8.5) 21:40:22 [scaladoc] Expected type was: _[?] => Coll[A] 21:40:22 [scaladoc] case xs: generic.GenericTraversableTemplate[_, _] => xs.genericBuilder mapResult { 21:40:22 [scaladoc] ^