summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | Fixed SI-5603. Early definitions now get transparent positions.Hubert Plociniczak2012-07-305-39/+107
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | This includes a fix for a minor problem described in #594 - ensureNonOverlapping still focuses on default position when outside of early defs. Review by @dragos, @odersky.
* | | | | | Merge pull request #998 from odersky/topic/bootpackagesJosh Suereth2012-07-301-1/+13
|\ \ \ \ \ \ | | | | | | | | | | | | | | Avoids loading scala.package and scala.reflect.package from source if a ...
| * | | | | | Avoids loading scala.package and scala.reflect.package from source if a ↵Martin Odersky2012-07-261-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | classfile exists. We know that loading these packages from source leads to compilation errors. To reproduce: Update scala.reflect.package, make sure it is on the source path and recompile anything using it. You will get a number of errors having to do with ClassTags and macro expansions. With the patch, these errors go away because the package is not loaded as long as a classfile exists.
* | | | | | | Merge pull request #1013 from odersky/ticket/5878odersky2012-07-303-2/+30
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Closes SI-5878
| * | | | | | | Closes SI-5878Martin Odersky2012-07-293-2/+30
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | We need to impose an additional rule on value classes: They may not unbox directly or indirectly to themselves.
* | | | | | | Merge pull request #1005 from odersky/topic/worksheetodersky2012-07-3012-72/+266
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | IDE improvements, with particular focus on making worksheets work.
| * | | | | | New Worksheet mixing schemeMartin Odersky2012-07-275-100/+196
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Debug changes
| * | | | | | Raw string interpolatorMartin Odersky2012-07-273-4/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a raw string interpolator raw"..." which does not do any escape sequence processing.
| * | | | | | Adds method askForResponseMartin Odersky2012-07-273-2/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds method askForResponse which returns a response immediately instead of waiting for a result. That way, one can wait for an ask's result using a timeout.
| * | | | | | Disable interrupts during code instrumentationMartin Odersky2012-07-271-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Code instrumentation needs to see a consistent snapshot of a source file. To achieve that we disable interrupts during getInstrumented.
| * | | | | | New Executor.Martin Odersky2012-07-273-4/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To try it out: run "instrument worksheet" as usual in the REPL. Worksheet needs to have enclosing object definition. Say it is named "Obj". This will generate a file Obj$instrumented. You can compile that file separately and run it using Java. It should reproduce itself.
* | | | | | | Merge pull request #997 from dragos/ide/range-positions-in-script-sourcesJosh Suereth2012-07-291-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | `ScriptSourceFile` should not hard-code `OffsetPosition`.
| * | | | | | | `ScriptSourceFile` should not hard-code `OffsetPosition`.Iulian Dragos2012-07-261-1/+1
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | The presentation compiler should support script source files as well, but it needs range positions. This commit fixes an oversight when `RangePosition` was introduced.
* | | | | | | Merge pull request #996 from gkossakowski/instrumentation-fixesJosh Suereth2012-07-295-3/+35
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Partest instrumentation fixes
| * | | | | | | Fix `Instrumentation.getStatistics` method in partest.Grzegorz Kossakowski2012-07-262-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous implementation was wrong because it would always enable profiling after call to `Instrumenation.getStatistics` method. Now we are checking the profiling status and enable it again only when it was enabled before. Review by @phaller.
| * | | | | | | Instrument all classes in `instrumented` package.Grzegorz Kossakowski2012-07-263-1/+23
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend instrumenting infrastructure to instrument classes in `instrumented` package. This is useful because very often you need to put your classes into non-empty package. E.g. inliner doesn't work properly with empty package at the moment so in order to test any behaviour we need to put classes in some other package that is instrumented. Added testing code for that to instrumentation test-case. Review by @phaller.
* | | | | | | Merge pull request #990 from VladUreche/issue/6140Josh Suereth2012-07-293-1/+20
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | SI-6140 Scaladoc infinite recursion in wiki parser
| * | | | | | | SI-6140 Scaladoc infinite recursion in wiki parserVlad Ureche2012-07-253-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | introduced by dc70d1b7.
* | | | | | | | Merge pull request #988 from paulp/issue/4560Josh Suereth2012-07-299-73/+115
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Issue/4560
| * | | | | | | | SI-4560 - improved testJohannes Rudolph2012-07-243-30/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added all the known cases of failing self-types. Added tests for symbol access (SI-4601) as well which should now be fixed for all of the same cases as well.
| * | | | | | | | Revert "tentative fix for RC5 lift build problem."Johannes Rudolph2012-07-242-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit cb4fd65825f3c88908103e48d0d7e89d70d26c22. Conflicts: src/compiler/scala/tools/nsc/symtab/Types.scala src/compiler/scala/tools/nsc/typechecker/Contexts.scala
| * | | | | | | | Revert "Closes #4560. Review by dragos." (introduction of safeREF)Johannes Rudolph2012-07-241-32/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 124cf3f9cbdc582a432c13edd229ba9b8726b99f. Conflicts: src/compiler/scala/tools/nsc/transform/CleanUp.scala
| * | | | | | | | Revert fix part of "Closes 4560. Review by dragos."Johannes Rudolph2012-07-241-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 7127d8293775905acd0d04e21d9e045b5c029261. Conflicts: test/files/run/t4560.scala
| * | | | | | | | Fix SI-4560, NoSuchMethodErrors involving self types.Paul Phillips2012-07-243-5/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Following this commit are reversions of three prior commits which introduced difficulties of their own, plus extra tests for this issue and SI-4601, all of which I pinched from jrudolph. Maybe there was a good reason for some of the complicated code related to this ticket. I took the naive position that if we avoided generating a method call to a particular receiver if the receiver has no such method, we would encounter fewer NoSuchMethodErrors. I would believe one can construct a scenario which this doesn't handle correctly, but it's hard to believe this isn't a big improvement. Review by @jrudolph, @odersky.
* | | | | | | | | Merge pull request #1002 from magarciaEPFL/fixes210Paul Phillips2012-07-285-5/+33
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | SI-6142: warn @inline-methods ending up not inlined (rightfully or not)
| * | | | | | | | | SI-6142: warn @inline-methods ending up not inlined (rightfully or not)Miguel Garcia2012-07-265-5/+33
| | |_|/ / / / / / | |/| | | | | | |
* | | | | | | | | Merge pull request #991 from adriaanm/ticket-5958Paul Phillips2012-07-282-0/+17
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | | SI-5958 This deserves a stable type
| * | | | | | | | SI-5958 This deserves a stable typeAdriaan Moors2012-07-262-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `this` (or the self variable) passed as an actual argument to a method should receive a singleton type when computing the method's resultType this is necessary if the method's type depends on that argument
* | | | | | | | | Merge pull request #1010 from odersky/ticket/6029Paul Phillips2012-07-282-24/+43
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | | Closed SI-6029 ...
| * | | | | | | | Closed 6029 ...Martin Odersky2012-07-282-24/+43
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... in a less nice way than I would like. Essentially, we mask type errors at later stages that arise from comparing existentials and skolems because we know that they are not tracked correctly through all tree transformations. Never mind that all these types are going erased anyway shortly afterwards. It does not smell nice. But as I write in the comment, maybe the best way out is to avoid skolems altogether. Such a change by far exceeds the scope of this pull request however.
* | | | | | | | Merge pull request #1004 from paulp/topic/warnings-2.10.xPaul Phillips2012-07-2766-148/+316
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Topic/warnings 2.10.x
| * | | | | | | Fix a few scaladoc tests I broke.Paul Phillips2012-07-271-4/+2
| | | | | | | |
| * | | | | | | Eliminated all the current feature warnings.Paul Phillips2012-07-2747-66/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This pretty much takes us down to deprecation and inliner warnings.
| * | | | | | | Changed isHidden condition back to isSynthetic.Paul Phillips2012-07-271-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we were blessed with spurious warnings. With this commit I have eliminated every warning in trunk except for deprecation, feature, and inliner warnings, plus one due to SI-6146. I'm not going to count "pattern matcher performance warnings" which are presently preventing me from turning on -unchecked even though there are no genuine -unchecked warnings remaining. Hey, get your own option! Conflicts: src/compiler/scala/tools/nsc/transform/SpecializeTypes.scala
| * | | | | | | Eliminated remainder of "catches Throwable" warnings.Paul Phillips2012-07-2710-13/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | And also non-exhaustive matches. And added -nowarn to the build options for scalacheck, we don't need to hear the warnings for code we aren't able to fix.
| * | | | | | | Cleared the remaining unchecked warnings in trunk.Paul Phillips2012-07-271-4/+4
| | | | | | | |
| * | | | | | | Improve unchecked warnings a lot.Paul Phillips2012-07-279-63/+198
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't warn on "uncheckable" type patterns if they can be statically guaranteed, regardless of their runtime checkability. This covers patterns like Seq[Any] and lots more. Review by @adriaanm.
* | | | | | | Merge pull request #993 from jsuereth/fix/typesafe-config-removalv2.10.0-M6Josh Suereth2012-07-252-3/+3
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Remove typesafe-config as dependency for library. Akka-actors will draw...
| * | | | | | Remove typesafe-config as dependency for library. Akka-actors will draw it in.Josh Suereth2012-07-252-3/+3
|/ / / / / /
* | | | | | Merge pull request #986 from scalamacros/ticket/6075Adriaan Moors2012-07-255-162/+32
|\ \ \ \ \ \ | | | | | | | | | | | | | | SI-6075 cleans up api.StandardNames
| * | | | | | SI-6075 cleans up api.StandardNamesEugene Burmako2012-07-245-162/+32
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I removed most of the stuff from api.StandardNames and reorganized all names. In the new scheme of things (as documented in the comments to StandardNames): 1) base.StandardNames contains names necessary for building trees. For example, tpnme.WILDCARD_STAR is necessary to build some invocations of varargs methods, and nme.CONSTRUCTOR is necessary to build invocations of constructors. 2) api.StandardNames hosts names that are core to doing reflection. E.g., to get to a package object, you need to find a member named nme.PACKAGE, to strip off the whitespace from the name of an underlying field of a val, you need nme.LOCAL_SUFFIX_STRING. Note that we don't need nme.MODULE_SUFFIX_STRING, because module name mangling is encapsulated in java mirrors.
* | | | | | Merge pull request #985 from scalamacros/ticket/6112Adriaan Moors2012-07-252-2/+10
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | SI-6112 exposes symbol mutators in macro universe
| * | | | | SI-6112 exposes symbol mutators in macro universeEugene Burmako2012-07-242-2/+10
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In makro.Universe we expose the most hardcore fragments of the compiler API that don't make sense / can't be made use of in runtime reflection API. makro.Universe is already refining Tree to include setters for symbols, types and positions. Now we bring Symbol up to speed.
* | | | | Merge pull request #976 from adriaanm/ticket-4440bAdriaan Moors2012-07-254-25/+58
|\ \ \ \ \ | | | | | | | | | | | | SI-4440 workaround: avoid outer accessor that'll vanish
| * | | | | SI-4440 workaround: avoid outer accessor that'll vanishAdriaan Moors2012-07-234-25/+58
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | outer accessors of final classes are dropped in constructors apply some foresight in explicit outer and replace those doomed outer checks by `true` (since this is unfortunate, we generate an unchecked warning)
* | | | | Merge pull request #989 from scalamacros/topic/hotfixPaul Phillips2012-07-241-3/+3
|\ \ \ \ \ | |_|_|/ / |/| | | | hotfix for TwoWayCache
| * | | | hotfix for TwoWayCacheEugene Burmako2012-07-251-3/+3
|/ / / /
* | | | Merge pull request #984 from scalamacros/ticket/6058Adriaan Moors2012-07-241-11/+25
|\ \ \ \ | | | | | | | | | | SI-6058 makes JavaMirror caches weak
| * | | | SI-6058 makes JavaMirror caches weakEugene Burmako2012-07-241-11/+25
| | |/ / | |/| | | | | | | | | | | | | | | | | | As discussed on reflection meeting on 2012/07/19, runtime reflection caches need to be weak. This way we prevent leaks in long-running reflection-based applications (the primary target here being SBT).
* | | | Merge pull request #966 from paulp/issue/6117Adriaan Moors2012-07-245-15/+52
|\ \ \ \ | |/ / / |/| | | SI-6117 regression involving import ambiguity.