summaryrefslogtreecommitdiff
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
* Large logging cleanup effort.Paul Phillips2012-09-011-6/+6
| | | | | | | | | Quieted down many logging statements which contribute disproportionate noise. Made others emit something more sensible. Spent lots of time on the inliner trying to find a regular format to make the logs more readable. Long way to go here but it'd be so worth it to have readable logs instead of mind-numbing indiscriminate text dumps.
* Merge pull request #1227 from paulp/issue/6034Paul Phillips2012-09-011-0/+1
|\ | | | | Fix for SI-6034, covariant value classes.
| * Fix for SI-6034, covariant value classes.Paul Phillips2012-08-311-0/+1
| | | | | | | | | | | | | | | | My summary in the ticket was incorrect. The problem was that the class type parameters were being cloned for the method and being allowed to keep their variance. I threw in an assertion for anyone attempting to create a method type with variant type parameters, because hey, why should we allow such madness.
* | Merge pull request #1201 from axel22/issue/4581Josh Suereth2012-09-015-4/+42
|\ \ | | | | | | Fix SI-4581.
| * | Fix SI-4581.Aleksandar Prokopec2012-08-275-4/+42
| | | | | | | | | | | | | | | | | | | | | Specifically, the final flag on the generated static field is no longer ommitted. Fix 2 failing test-cases.
* | | Merge pull request #1225 from jsuereth/fix/si-6290Josh Suereth2012-09-011-0/+4
|\ \ \ | | | | | | | | Fixes SI-6290 by creating real instnaces of language features.
| * | | Fixes SI-6290 by creating real instnaces of language features.Josh Suereth2012-08-311-0/+4
| | | |
* | | | More useful -Xlog-implicits output.Jason Zaugg2012-09-013-0/+25
| |_|/ |/| | | | | | | | | | | The test exercises the most important case, when implicits are invalidated through shadowing. (See SI-4270)
* | | Merge pull request #1213 from paulp/topic/anyvalJosh Suereth2012-08-305-17/+7
|\ \ \ | | | | | | | | Expanded the reach of value classes.
| * | | Expanded the reach of value classes.Paul Phillips2012-08-295-17/+7
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now extending AnyVal: - RichInt, RichDouble, etc. - ArrayOps.ofRef, ofBoolean, etc - StringAdd - StringFormat The rest of it is the changes necessary to enable those.
* | | Merge pull request #1160 from rklaehn/SI-6220Josh Suereth2012-08-301-0/+92
|\ \ \ | | | | | | | | Si 6220
| * | | Added test that should cover all code paths of the changes done in SI-6220Ruediger Klaehn2012-08-181-0/+92
| | | | | | | | | | | | | | | | This test will pass even with an older version of the scala library, since as mentioned this is just a performance improvement.
* | | | Merge pull request #1169 from rklaehn/SI-6261Josh Suereth2012-08-301-0/+130
|\ \ \ \ | | | | | | | | | | Si 6261
| * | | | Added test to ensure that ListMap.tail is O(1)Ruediger Klaehn2012-08-251-0/+7
| | | | |
| * | | | Extended test to also check proper handling of collisionsRuediger Klaehn2012-08-211-0/+91
| | | | |
| * | | | Added test that tests that correct classes are being created when removing ↵Ruediger Klaehn2012-08-211-0/+32
| | | | | | | | | | | | | | | | | | | | elements
* | | | | Merge pull request #1209 from odersky/ticket/6227Josh Suereth2012-08-303-2/+10
|\ \ \ \ \ | | | | | | | | | | | | Closes SI-6227
| * | | | | Added/fixed test files for SI-6227Martin Odersky2012-08-292-2/+4
| | | | | |
| * | | | | Closes SI-6227Martin Odersky2012-08-291-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | I added some general hook where one can add validation code before a name conflict involving at least one implicit symbol is reported.
* | | | | | Fixed cloning a double-linked list.Josh Suereth2012-08-291-0/+18
| |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Still need to determine if this is a systemic issue and needs to be addressed higher in the linked list hierarchy. For now, just fixing the reported bug, with a note here for other maintainers.
* | | | | Merge pull request #1197 from scalamacros/topic/kindEugene Burmako2012-08-293-10/+10
|\ \ \ \ \ | | | | | | | | | | | | removes Symbol.kind and Type.kind
| * | | | | removes Symbol.kind and Type.kindEugene Burmako2012-08-263-10/+10
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | use Universe.showRaw instead: scala> import scala.reflect.runtime.{universe => ru} import scala.reflect.runtime.{universe=>ru} scala> ru.showRaw(typeOf[Int]) res0: String = TypeRef(ThisType(scala), scala.Int, List()) scala> ru.showRaw(typeOf[Int].typeSymbol, printKinds = true) res1: String = scala.Int#CLS
* | | | | Merge pull request #1184 from pavelpavlov/partialfunGrzegorz Kossakowski2012-08-292-0/+92
|\ \ \ \ \ | | | | | | | | | | | | PartialFunction polishing
| * | | | | PartialFunction polishingPavel Pavlov2012-08-232-0/+92
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - ScalaDocs added - TODOs fixed - controversive method `run` deleted - not used class runtime.AbstractTotalFunction removed - small corrections & fixes - tests for `orElse` & `runWith`
* | | | | | Merge pull request #1206 from paulp/issue/6283Grzegorz Kossakowski2012-08-292-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix for SI-6283, no abstract value classes.
| * | | | | | Fix for SI-6283, no abstract value classes.Paul Phillips2012-08-282-0/+5
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The needless abstraction penalty was in full flower in Namers. I managed to find somewhere else to issue this error, where I can still just write an error message without tracking down an enumeration in a separate file and inventing an intermediate name for the enum member.
* | | | | | Merge pull request #1195 from Blaisorblade/topic/test-tryGrzegorz Kossakowski2012-08-291-1/+13
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Add missing tests for SI-6190
| * | | | | Add missing tests for SI-6190Paolo Giarrusso2012-08-251-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was silently fixed in commit 3cb0e784a05db7d0b542cec9bf4c5fbf3772a6cf but no test was added.
* | | | | | Merge pull request #1188 from retronym/ticket/6272Josh Suereth2012-08-282-0/+72
|\ \ \ \ \ \ | | | | | | | | | | | | | | SI-6272 Support lazy vals defined in try in template.
| * | | | | | SI-6272 Support lazy vals defined in try in template.Jason Zaugg2012-08-242-0/+72
| |/ / / / /
* | | | | | Merge pull request #1185 from paulp/210-updateConservePaul Phillips2012-08-281-12/+15
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Eliminated a metric ton of allocations.
| * | | | | Eliminated a metric ton of allocations.Paul Phillips2012-08-231-12/+15
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks to the hotspot DTrace probes. I'm not sure why this checkfile changed, but hopefully it's innocuous. Review by @VladUreche for the checkfile and @gkossakowski so he can tell me how much faster.
* | | | | SI-6281 macroArgs for defs with implicit argsEugene Burmako2012-08-254-0/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | macroArgs now correctly calculates the argss in case when a macro def has implicit parameters. Macro impls can never have implicit parameters other than type tag evidences, so tags will always come in a separate parameter list w.r.t other parameters of macro impls (which are in 1-to-1 correspondence with the parameters of macro defs). Example 1: def w/o implicits: (params1)...(paramsN) impl w/o tags: (params1')...(paramsN') Example 2: def w/o implicits: (params1)...(paramsN) impl w tags: (params1')...(paramsN')(implicit tags) Example 3: def w implicits: (params1)...(paramsN)(implicit paramsN+1) impl w/o tags: (params1')...(paramsN')(paramsN+1') Note: paramsN+1' is not an implicit parameter list because impls cannot have implicit parameters other than tags Example 4: def w implicits: (params1)...(paramsN)(implicit paramsN+1) impl w tags: (params1')...(paramsN')(paramsN+1')(implicit tags) Therefore we don't need to merge the argss.last (that correspond to implicit parameters of macro defs) with tags, as it was incorrectly coded before. We just need to append tags to argss.
* | | | | SI-6052 - fix groupBy on parallel collectionsAleksandar Prokopec2012-08-241-0/+21
| |/ / / |/| | |
* | | | Merge pull request #1179 from scalamacros/topic/classtag-of-nothingPaul Phillips2012-08-231-30/+30
|\ \ \ \ | |/ / / |/| | | reverts changes to ClassTag.Nothing
| * | | reverts changes to ClassTag.NothingEugene Burmako2012-08-221-30/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seemed to be a good idea to have ClassTag.Nothing crash in agreement with its incalculable nature. However this breaks CanBuildFrom for the situations when Nothing is inferred or provided explicitly. Hence the rollback.
* | | | Merge pull request #1182 from paulp/issue/1987Josh Suereth2012-08-236-0/+46
|\ \ \ \ | | | | | | | | | | Improved fix for SI-1987, overloading in package objects.
| * | | | Improved fix for SI-1987, overloading in package objects.Paul Phillips2012-08-226-0/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When reformulating an errant package object overload, don't forget to fully qualify it lest you trade one error for another.
* | | | | Merge pull request #1172 from Blaisorblade/topic/deprecated-conversionJosh Suereth2012-08-233-0/+33
|\ \ \ \ \ | | | | | | | | | | | | JavaConversions: Restore source compatibility with 2.9
| * | | | | Ensure implicit conversions to concurrent map are unambiguousPaolo Giarrusso2012-08-223-4/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Even after the parent change, implicit conversions should still convert ConcurrentHashMap to concurrent.Map and not to mutable.ConcurrentMap. I'll include the comment by axel22 which prompting this change (taken from https://github.com/scala/scala/pull/1172#issuecomment-7928285) since it is highly informative and links to pull request comments might not be very stable: << Instead of just testing an implicit conversion to ConcurrentMap, the test should call ConcurrentMap methods such as putIfAbsent (which returns an Option[T]) and replace, to see if the correct Scala concurrent map trait is being resolved. The problem is that putIfAbsent already exists on juc.ConcurrentMap so instead of triggering an implicit conversion, a type mismatch error is output anyway. And we cannot test that the correct concurrent map trait is returned using methods like map, flatMap and friends, because concurrent map traits extends mutable.Map anyway. For this reason, I recommend to add this to the test: scala> val a = new java.util.concurrent.ConcurrentHashMap[String, String]() += (("", "")) a: scala.collection.concurrent.Map[String,String] = Map("" -> "") and make sure that the returned map is concurrent.Map, not mutable.ConcurrentMap (the above += is one of the few methods in collection that has the return type this.type). >>
| * | | | | JavaConversions: Restore source compatibility with 2.9Paolo Giarrusso2012-08-201-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Readd an implicit conversion which was available in 2.9, the one from `java.util.concurrent.ConcurrentMap` (`juc.ConcurrentMap`) to the (now deprecated) type `scala.collection.mutable.ConcurrentMap`. This implicit conversion can also be used to convert from `juc.ConcurrentMap` to `collection.Map` and creates an ambiguity error in test/files/run/map_java_conversions.scala. To fix this, I have given lower priority to the new conversion. Moreover, update the documentation in `JavaConversions`: mark this conversion as deprecated and mention the new conversion which replaces it, converting to `scala.collection.concurrent.Map`. I discussed this issue previously with Paul Phillips on scala-language: https://groups.google.com/d/topic/scala-language/uXKRiGXb-44/discussion
* | | | | | Merge pull request #1183 from jsuereth/fix/try-catch-inversionJosh Suereth2012-08-232-0/+2
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Fix stupid logic inversion of try-catch
| * | | | | Fix stupid logic inversion of try-catchJosh Suereth2012-08-212-0/+2
| | | | | |
* | | | | | Merge pull request #1167 from Blaisorblade/topic/stream-const-spaceGrzegorz Kossakowski2012-08-221-0/+44
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Make Stream.withFilter.{map,flatMap} run in constant stack space
| * | | | | Also check that Stream.toSeq gives the right result.Paolo Giarrusso2012-08-221-1/+3
| | | | | |
| * | | | | Improve test for Stream.withFilter.{map,flatMap}Paolo Giarrusso2012-08-221-10/+40
| | | | | | | | | | | | | | | | | | | | | | | | Test a wider range of functionality.
| * | | | | Cleanup testcasePaolo Giarrusso2012-08-202-6/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | No need to check the output - checking programmatically that the produced streams are empty is enough.
| * | | | | Make Stream.withFilter.{map,flatMap} run in constant stack spacePaolo Giarrusso2012-08-202-0/+16
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The included test currently fails because `map` and `flatMap` do not run in constant stack space on a stream returned by `Stream.withFilter`, as I reported here: https://groups.google.com/d/msg/scala-language/WqJR38REXnk/saaSiDdmyqoJ Fix the problem and add a simple testcase. Note that the stack space consumed when producing an element of this stream is proportional to the number of elements failing the test before the next success. The stack space consumed to produce the stream itself is the space needed to produce the first element, that is, is proportional to the number of failures before the first success.
* | | | | Merge pull request #1176 from scalamacros/topic/nondeterministicPaul Phillips2012-08-214-36/+2
|\ \ \ \ \ | | | | | | | | | | | | fixes non-determinism in reflection-sorted-decls
| * | | | | fixes non-determinism in reflection-sorted-declsEugene Burmako2012-08-214-36/+2
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ordering of synthetic members on java7 with a lot of cores is non-deterministic that's not a problem, because the doc doesn't promise anything about synthetics hence I've adjusted the test, so that it no longer includes accessors discussion: http://groups.google.com/group/scala-internals/msg/087a7d4805313561
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-01 13:49:11 +0000