| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
SI-8817 Correct scaladoc for scala.sys.addShutdownHook
|
| |
| |
| |
| |
| | |
- Remove mention that shutdown hook is daemon thread
- Add link on scala.sys.ShutdownHookThread class
|
|\ \
| |/
|/| |
SI-8474 Inconsistent behavior of patch method
|
| |
| |
| |
| |
| |
| |
| |
| | |
Changed Iterator to be consistent with other collections.
Also fixed SeqViewLike to validate/constrain inputs.
No specific tests; quasi-comprehensive collection tests will cover this later.
|
|\ \ |
|
| | | |
|
| |\ \
| | | |
| | | | |
[backport] Backports library changes related to SI-6566 from a419799
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The idea of backporting this occured while developing the -source flag
for SI-8126 : withouth this the library breaks at refchecks for a
Scala compiler that checks type aliases variance.
This shoudl be BC and promote good hygiene.
|
| |/ / |
|
|\ \ \
| | | |
| | | | |
SI-8627 make Stream.filterNot non-eager
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The obvious fix, overriding `filterNot` in Stream, is not binary
compatible, see https://github.com/scala/scala/pull/3925
Instead, this makes `filterImpl` in TaversableLike private[scala],
which allows overriding it in Stream. The corresponding mima-failures
can be whitelisted, as the changes are only to private[scala].
In 2.12.x we can remove the override of `filter` in Stream, but in
2.11.x this is not binary compatible.
Eventually we'd also like to make filter / filterNot in
TraversableLike final, but that's not source compatible, so it cannot
be done in 2.12.x.
|
|\ \ \ \
| | |_|/
| |/| |
| | | |
| | | | |
Conflicts:
src/library/scala/util/matching/Regex.scala
|
| |\ \ \
| | |/ /
| |/| | |
SI-8787 Regextraction is null-proof
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Simplify the exposition and update the examples.
Emphasize pattern matching, especially using `r.unanchored`
instead of `for (r(x,y,z) <- r findFirstIn text)`.
Certain details are moved to method docs.
It would be nice to fix matching package doc,
but the doc must attach to the package object, it seems.
Introducing a package object is not binary-compatible.
Includes a doc line edit on 2.12, anticipating the merge.
|
| | | |
| | | |
| | | |
| | | | |
Regex is robust when unapplying null. A null never matches.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The f-interpolator gets a type param that better be Any to avoid
unfortunate widenings.
Hey, it worked!
Unfortunately, when `Any` is inferred, `-Xlint:infer-any` takes notice.
This is probably a greater problem for the f-interpolator than for
quasiquotes, which are a more specialized tool.
|
| |\ \ \
| | | | |
| | | | | |
a few missing deprecations in proxy collections.
|
| | |/ /
| | | |
| | | |
| | | |
| | | |
| | | | |
SeqProxy was properly deprecated, so were the CollProxyLike classes, and
the ones in collection.immutable, but these four somehow survived
the Big Proxy Deprecation (tm).
|
| |\ \ \
| | |/ /
| |/| | |
SI-6476 Improve error on escapement
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Behavior of escape processing under string interpolation can
be confusing. This commit improves the exception message so
you know at least what sort of escapes are handled.
This came up on SO in the form `s"\d".r`, where it may not
be obvious what is throwing and how to work around it.
```
scala> s"\d".r
scala.StringContext$InvalidEscapeException: invalid escape '\d' not one of [\b, \t, \n, \f, \r, \\, \", \'] at index 0 in "\d". Use \\ for literal \.
scala> s"a\"
scala.StringContext$InvalidEscapeException: invalid escape at terminal index 1 in "a\". Use \\ for literal \.
```
Referencing SI-6476 because that has become the magnet ticket
for "escape processing under string interpolation, huh?"
This doesn't address `$"` and doesn't handle the more interesting
parse error `s"a\"b"`.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This commit changes all first sentences of library functions which
contain `i.e.` or `e.g.` and adds a `,` to prevent that the scaladoc
summary sentence is cut after this abbreviation.
This is possible as pull/3824 fixed how Scaladoc parses the first
sentence of a method description into a sumary sentence(now the first
sentence has to end with a dot followed by whitespace).
Only docs in the core library are changed (src/library/**/*.scala)
and only if they occur in the first sentence.
Review by @heathermiller
(cherry picked from commit 72721ff5dd06dea1235ecb71acae0bd61aee4814)
|
| | | |
| | | |
| | | |
| | | | |
(cherry picked from commit 7cca7635b392cb533d0f8e26b74d7362c0dd3891)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
scala.math.PartialOrdering.
- Previous (misleading) documentation suggested that a partial ordering *is* an equivalence relation. This has now been clarified.
- Existing documentation on properties of partial orderings (reflexive, transitive, antisymmetric) had several formatting inconsistencies that have now been remove.
(cherry picked from commit 528df5becee03175e7462b64d7c22460cc5992f7)
|
| | | |
| | | |
| | | |
| | | | |
(cherry picked from commit 0a1694d8048f01b7f0812cedaa341813e9044e7f)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
given by som-snytt.
(cherry picked from commit ec4abf5483e890976ae460252e80a66a95674e61)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Scaladoc places the first sentence in the method summary table and index. The first sentence is defined as the sequence of chars up to the first period (not as in JavaDoc where the sentence ends with the first period *followed by a blank*). As a consequence, the clause starting with `i.e.` is interpreted as sentende delimiter. This request replaces `i.e.` with `i.e.`. Alghough a valid HTML code, I do not know whether this change is compatible with other tools. And I assume that this is not the only source file affected.
(cherry picked from commit c2e4da674d8c40e2d220854a966b0510fb6d459e)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Option.collect methods.
(cherry picked from commit e3311073bbad6f6f0dfdd3ea09628d324b4b3614)
|
| | | |
| | | |
| | | |
| | | | |
(cherry picked from commit 1e2a21681a4a55469faa59b07473a3b269e70258)
|
| | | |
| | | |
| | | |
| | | | |
(cherry picked from commit 6c698089aeb55e649a65dd7ae1bce2b4514ee865)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
See e2a3498 commit message for explanation
Explicit private declaration better than implicit
Thanks to @som-snytt
(cherry picked from commit d071abe8dd9035d8dd246ecb6b207e584b6437fa)
|
| | | |
| | | |
| | | |
| | | | |
(cherry picked from commit 80a9e908fd7b591a6fe452d908407d537675a999)
|
| | | |
| | | |
| | | |
| | | | |
(cherry picked from commit 71bc2e5f4c49463a754a6f23e3abd2d27467fca4)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
caller is responsible for checking 'isDefinedAt'
(cherry picked from commit 455bcd65c055119f928b3367839f0093cf0e251a)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
caller is responsible for checking 'isDefinedAt'
(cherry picked from commit f7d2cec3a0c31836747a25e3cc5949d1f3cbdff4)
|
|/ / /
| | |
| | |
| | | |
(cherry picked from commit fd001811c662f6daecf91319b36c1ad14a87ff72)
|
| | |
| | |
| | | |
As pointed out by @som-snytt, \u0023 is #, not \u0025.
|
| | |
| | |
| | | |
Since StringContext.raw doesn't escape its input, it does not call `treatEscapes` and cannot throw the InvalidEscapeException.
|
|/ /
| |
| |
| |
| |
| | |
Missed the case of comparing a non-empty range to an empty one. Fixed by checking nonEmpty/isEmpty on other collection.
Added a test to verify the behavior.
|
| |
| |
| |
| |
| |
| |
| | |
- link to Java API docs for ForkJoinPool-based default implementation
- add example for creating an execution context from a `java.util.concurrent.ExecutorService`
- add tags for parameters and return values
- expand doc comment for `prepare`
|
|\ \
| | |
| | | |
SI-8677 Duration: Zero - Inf should be MinusInf
|
| | |
| | |
| | |
| | |
| | | |
Fixes #8677. Add basic tests.
This is a backport from 2.12.x of dead39dc5f21c6eac41788e93426c50ddd398c24.
|
|\ \ \
| | | |
| | | | |
SI-8335 List.++ avoidably burns memory
|
| | | |
| | | |
| | | |
| | | | |
Changed to check the identity of the CanBuildFrom instead of the identity of the generated builder to shortcut building. Should reduce memory churn on ++ a little.
|
|\ \ \ \
| |_|/ /
|/| | | |
SI-8638 Empty UnrolledBuffer hangs on prepend.
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
Tracked down bug to incorrect recursion in insertAll. Fixed by adding a missing case (which incidentally will provide better performance when adding to the end of a block).
No specific tests, as this is caught by quasi-complete collections tests.
|
|\ \ \
| | | |
| | | | |
SI-5200 Incorrect advice for implementing mutable.Set in scaladoc
|
| |/ /
| | |
| | |
| | | |
Fixed advice; it was already there in mutable.SetLike but a case sensitivity error kept it from appearing.
|
|\ \ \
| | | |
| | | | |
SI-7115 JMapWrapper.get can incorrectly return Some(null)
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This isn't incorrect. Trying to use a single-threaded interface in a concurrent context is supposed to break in various unpleasant ways.
Documentation has been added to encourage one to avoid wrapping a concurrent map in the generic wrapper (which assumes a single thread), and pointing out that synchronized maps do not maintain synchronization for non-atomic operations (including get).
More docs.
|