summaryrefslogtreecommitdiff
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
* SI-9104 Autodetect raw pastageSom Snytt2016-06-028-3/+77
| | | | | | | | | | | | | | If `-raw` is not supplied explicitly to REPL `:paste`, see if the code text starts with `package` keyword or else see if it parses to a named package (to cope with leading commentary). In that case, take it as raw. But parse only on suspect comment slash. It's only worth parsing for a package if there's a chance that package keyword is buried behind comments. Small refactors to the `paste` object.
* Merge pull request #5207 from lrytz/nan-compareAdriaan Moors2016-06-021-0/+38
|\ | | | | Fix comparisons involving NaN
| * Fix comparisons involving NaNLukas Rytz2016-06-021-0/+38
| | | | | | | | | | Floating point comparisons involving NaN should always return false, except for !=. Fixes a regression introduced by #4963.
* | scala version in benchmark projectLukas Rytz2016-06-021-1/+1
| |
* | Merge commit '90215ce' into merge-2.11-to-2.12-june-1Lukas Rytz2016-06-019-86/+286
|\ \
| * \ Merge pull request #4998 from som-snytt/issue/7898-iLukas Rytz2016-06-015-4/+35
| |\ \ | | | | | | | | SI-7898 Read user input during REPL warmup
| | * | SI-7898 Label for parsing -i sourcesLukas Rytz2016-05-243-0/+34
| | | | | | | | | | | | | | | | Text-based REPL pre-parses, so use the current label for errors.
| | * | SI-7898 Report paste errors improvedlySom Snytt2016-05-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use a "label" for errors, so that script names are shown. Position is still wrong for scripts in REPL. Initial scripts are run with `ILoop.echo` and results printing turned off, but reporter still enabled.
| | * | SI-7898 Read user input during REPL warmupSom Snytt2016-05-201-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The compiler is created on main thread and user input is read on an aux thread (opposite to currently). Fixes completion when `-i` is supplied. Now `-i` means pasted and new option `-I` means line-by-line. The temporary reader uses postInit to swap in the underlying reader. Completion is disabled for the temporary reader, rather than blocking while it waits for a compiler. But manically hitting tab is one way of knowing exactly when completion is live.
| * | | SI-9789 use quadratic probing in OpenHashMapPerformant Data LLC2016-05-264-82/+251
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original probe sequence, taken from Python's hash table code, is exponential, jumping around in the hash table with poor memory locality. This replaces the probe algorithm with the more conventional quadratic probing. This also adds tests to the benchmarking code using AnyRef keys, which have pseudorandom hash codes (unlike Ints, whose hash code is simply the Int itself). The intensity of the benchmarking is reduced to make the tests complete within 9 hours, by removing unnecessary sampling.
| * | | Merge pull request #4959 from rjolly/scripting15Stefan Zeiger2016-05-252-8/+4
| |\ \ \ | | | | | | | | | | Use jarlister in build
| | * | | Use jarlister in buildRaphael Jolly2016-05-212-8/+4
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | The goal of this change is to exercize the "manifest classpath" mechanism, meant to bring the compiler its needed classes as resources, listed in jar manifests, as opposed to files, thus enabling to use the compiler in sandboxed environments (and also the scripting engine for that matter).
* | | | Merge commit 'cba585d' into merge-2.11-to-2.12-june-1Lukas Rytz2016-06-0116-0/+503
|\| | |
| * | | Merge pull request #5061 from performantdata/benchmark-frameworkJason Zaugg2016-05-257-0/+457
| |\ \ \ | | | | | | | | | | JMH-based benchmark framework
| | * | | Enable full compiler optimizations in JMH benchmarking.Performant Data LLC2016-05-051-1/+1
| | | | |
| | * | | Address JMH benchmark reviewer's issues.Performant Data LLC2016-05-033-59/+61
| | | | | | | | | | | | | | | | | | | | | | | | | Besides tweaks to the documentation, this tests smaller (25-element) maps, and rewrites OpenHashMapRunner in more idiomatic Scala.
| | * | | Improve the OpenHashMapBenchmark run times.Performant Data LLC2016-05-031-7/+14
| | | | | | | | | | | | | | | | | | | | | | | | | For the warm-up invocations, suppress setup and teardown that is only needed for the measurement iterations. Reduce the number of forks.
| | * | | Add a JMH runner class to the library benchmark framework.Performant Data LLC2016-05-033-9/+167
| | | | |
| | * | | Benchmark the OpenHashMap memory usage.Performant Data LLC2016-05-034-11/+35
| | | | | | | | | | | | | | | | | | | | Also add sbteclipse to the benchmark project.
| | * | | Add get() tests to OpenHashMap, reduce timing artifacts.Performant Data LLC2016-05-031-35/+144
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to get a better exploration of the variance of tests in a limited time, I've reduced the number of measurement iterations and increased the number of forks. By sight, the measurement iterations seemed pretty consistent within a trial, whereas they would vary widely on occasional forks. I extended testing down to 50-entry maps, to explore the rise in service times that I was seeing at small scale. This is probably a timing artifact, from too-short invocations, since I'm using @Level.Invocation in the put() tests. To fix that, I enlarged the unit of testing, by creating multiple, sometimes thousands, of maps for the invocation to fill. This has also changed the test from filling a previously-filled map, to filling a new, but sufficiently sized map. The put()/remove() test now performs much worse (on a more realistic scenario). This also adds a couple tests for calling get() against a map that's been filled only with put()s, or with a mix of put() and remove().
| | * | | Add a reference to Doug Lea's benchmarks.Performant Data LLC2016-05-031-1/+2
| | | | |
| | * | | Add JMH to the benchmark framework.Performant Data LLC2016-05-035-0/+156
| | | | | | | | | | | | | | | | | | | | Add an example benchmark for OpenHashMap.
| * | | | Merge pull request #5169 from som-snytt/issue/4625Lukas Rytz2016-05-239-0/+46
| |\ \ \ \ | | | | | | | | | | | | SI-4625 Recognize App in script
| | * | | | SI-4625 Warn on first non-toplevel onlySom Snytt2016-05-171-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed the warning when main module is accompanied by snippets. Minor clean-up so even I can follow what is returned.
| | * | | | SI-4625 Warn when discarding script objectSom Snytt2016-05-173-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's pretty confusing when your script object becomes a local and then nothing happens. Such as when you're writing a test and it takes forever to figure out what's going on.
| | * | | | SI-4625 Permit arbitrary top-level in scriptSom Snytt2016-05-163-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In an unwrapped script, where a `main` entry point is discovered in a top-level object, retain all top-level classes. Everything winds up in the default package.
| | * | | | SI-4625 Recognize App in scriptSom Snytt2016-05-163-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cheap name test: if the script object extends "App", take it for a main-bearing parent. Note that if `-Xscript` is not `Main`, the default, then the source is taken as a snippet and there is no attempt to locate an existing `main` method.
| * | | | | Merge pull request #5167 from martijnhoekstra/SI-9766Lukas Rytz2016-05-231-0/+18
| |\ \ \ \ \ | | | | | | | | | | | | | | SI 9766 - allow ++ on empty ConcatIterator
| | * | | | | SI-9766 - allow ++ on empty ConcatIteratorMartijn Hoekstra2016-05-211-0/+18
| | |/ / / /
* | | | | | Merge commit '4196569' into merge-2.11-to-2.12-june-1Lukas Rytz2016-06-011-0/+48
|\| | | | |
| * | | | | SI-9688 Make merge in immutable HashMap1 work with null kv.Łukasz Gieroń2016-05-231-0/+48
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The kv field of scala.collection.immutable.HashMap.HashMap1 can be null. This commit corrects the behavior of updated0 (which is on call path for merged) to work in such cases, instead of throwing NPE. Commit contains regression test.
* | | | | Merge commit '90706b0' into merge-2.11-to-2.12-june-1Lukas Rytz2016-06-012-0/+0
|\| | | | | |_|_|/ |/| | |
| * | | Move t8449 to correct placeLukas Rytz2016-05-172-0/+0
| |/ / | | | | | | | | | Follow-up for https://github.com/scala/scala/pull/4117
* | | SI-9767 document and test behaviour of String->integer/float conversionsMike Pheasant2016-06-011-0/+31
| | | | | | | | | | | | | | | | | | | | | We delegate `String`'s extension methods `toInt`, `toFloat`, etc to corresponding methods in the Java standard library. These differ in the way they handle whitespace in the original string. This commit documents and tests the current behaviour.
* | | Merge pull request #5076 from soc/topic/deprecations-sinceLukas Rytz2016-05-3060-99/+104
|\ \ \ | | | | | | | | Improvements to deprecations related to `since` parameter
| * | | Add since arg to deprecationWarning and use itSimon Ochsenreither2016-05-2942-49/+55
| | | |
| * | | Lower-case spelling of @deprecated messagesSimon Ochsenreither2016-05-286-18/+17
| | | |
| * | | SI-9084 Add `since` (if available) to deprecation warningsSimon Ochsenreither2016-05-2821-59/+59
| | | |
* | | | Merge pull request #5191 from som-snytt/issue/9382Lukas Rytz2016-05-305-57/+84
|\ \ \ \ | | | | | | | | | | SI-9382 Privatize enhanced x in Tuple2Zipped.Ops
| * | | | SI-9382 Zippy clean-up in aisle 2 & 3Som Snytt2016-05-263-57/+68
| | | | | | | | | | | | | | | | | | | | | | | | | Consolated JUnit tests and heeded comment about private def and code beauty.
| * | | | SI-9382 Privatize enhanced x in Tuple2Zipped.OpsSom Snytt2016-05-252-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Probably there should be an Abide rule to avoid leaking the "underlying value" of a value class. The spec or SIP defines "underlying type" but doesn't mention the underlying value. The argument for concealing the member is that it is redundant and makes autocompletion results harder to read. Also, possibly an additional implicit might want to add a member so-named.
* | | | | Merge pull request #5193 from som-snytt/issue/9794Lukas Rytz2016-05-302-1/+9
|\ \ \ \ \ | |_|/ / / |/| | | | SI-9794 Error advice uses decoded method name
| * | | | SI-9794 Error advice uses decoded method nameSom Snytt2016-05-262-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So much work went into polishing this error message, it's worth buffing the method name when it's an operator. The message now says `+` instead of `$plus`.
* | | | | Merge pull request #5102 from milessabin/2.12.xJason Zaugg2016-05-2731-4/+470
|\ \ \ \ \ | | | | | | | | | | | | SI-2712 Add support for partial unification of type constructors
| * | | | | -Xexperimental mode now only includes -Ypartial-unificationMiles Sabin2016-05-242-4/+4
| | | | | |
| * | | | | SI-2712 Add support for higher order unificationMiles Sabin2016-05-2429-0/+466
| | | | | |
* | | | | | Merge pull request #5186 from lrytz/inlinerM5Jason Zaugg2016-05-271-5/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Debug flag to print a summary of the inliner's work
| * | | | | | Debug flag to print a summary of the inliner's workLukas Rytz2016-05-241-5/+5
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Example output below. Note that inlining List.map fails because the trait forwarder uses `INVOKESPECIAL` for now, will change with pr 5177. $ cat Test.scala class C { def foo = Map(1 -> 'a', 2 -> 'b') def bar(l: List[Int]) = l.map(_ + 1) } $ qsc -Yopt-log-inline _ -Yopt:l:classpath Test.scala Inlining into C.foo (initially 36 instructions, ultimately 72): - Inlined scala/Predef$ArrowAssoc$.$minus$greater$extension (8 instructions) 2 times: the callee is annotated `@inline` Inlining into C.bar (initially 12 instructions, ultimately 12): - Failed to inline scala/collection/immutable/List.map (the callee is a higher-order method, the argument for parameter (bf: Function1) is a function literal): The callee scala/collection/immutable/List::map(Lscala/Function1;Lscala/collection/generic/CanBuildFrom;)Ljava/lang/Object; contains the instruction INVOKESPECIAL scala/collection/TraversableLike.map (Lscala/Function1;Lscala/collection/generic/CanBuildFrom;)Ljava/lang/Object; that would cause an IllegalAccessError when inlined into class C.
* | | | | | Merge pull request #5192 from dwijnand/wip/scala-repl-no-importsJason Zaugg2016-05-2710-15/+586
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Fully qualify types in REPL generated code
| * | | | | Fully qualify types in REPL generated codeDale Wijnand2016-05-2610-15/+586
| | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-11 22:27:43 +0000