summaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Migrate build to @odersky's suggestion of convertTo.Josh Suereth2012-06-185-16/+19
| | | | | | * Move method into TraversableOnce from Iterator and Traversable to make the build pass. * Udpate IDE tests with new collection methods. * Rewire default toXYZ methods to use convertTo.
* Rename copyTo to build based on consensus of 3Josh Suereth2012-06-184-7/+7
|
* Fixes from review.Josh Suereth2012-06-184-9/+9
| | | | | | * Fixed typo * Renamed copyInto to copyTo * Added tparam doc.
* Adding copyInto and toVector methods to collections.Josh Suereth2012-06-186-0/+45
| | | | | * Added generic copyInto method for collections. For any collection with a CanBuildFrom, can convert a generic collection into it using the builder. * Added specifici toVector method for collections. This is more efficient than copyInto if the collection is a Vector.
* Merge pull request #736 from damienobrist/feature/diagrams-dev-revertJosh Suereth2012-06-1833-2486/+190
|\ | | | | Reverting the diagrams
| * Revert "Scaladoc class diagrams part 1"Vlad Ureche2012-06-1820-1173/+162
| | | | | | | | This reverts commit 831f09bb6d00c152bd8aef3ce8bf9e200080fe36.
| * Revert "Scaladoc class diagrams part 2"Vlad Ureche2012-06-1814-1301/+22
| | | | | | | | This reverts commit fba4098ca592bdd7d6609017daa4a08f274f35ec.
| * Revert "Documented SyncVar"Vlad Ureche2012-06-181-13/+7
| | | | | | | | This reverts commit 556065151c10685240dd907691dab5e094c87efc.
* | Merge pull request #718 from damienobrist/feature/diagrams-dev-finalJosh Suereth2012-06-1733-190/+2486
|\| | | | | Diagrams in Scaladoc
| * Documented SyncVarVlad Ureche2012-06-141-7/+13
| | | | | | | | | | | | | | Since we used it in the DocRunner and noticed it could have better documentation. Review by @heathermiller.
| * Scaladoc class diagrams part 2Damien2012-06-1414-22/+1301
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit contains the svg diagram generation using the graphviz package, the template changes, the css styling and javascript code that enables displaying and interacting with the diagrams. The full history is located at: https://github.com/damienobrist/scala/tree/feature/diagrams-dev The diagrams are included as svg markup inside the html code. This enables interacting with the image beyond what would be possible with a static image (highlighting, scaling, tooltips, links to nodes, etc). The svg generation has four main phases: model => dot, dot => svg (using the graphviz package), svg postprocessing, inclusion in the html page. This commit also fixes SI-5212 - links to individual pages automatically load the left navigation panel of the website. Commit summary: - diagram generation - model => dot (DotDiagramGenerator.scala, DiagramGenerator.scala) - dot => svg (DotRunner.scala) - svg post-processing (DotDiagramGenerator.scala) - svg inclusion in the html (Template.scala) - diagram interaction - css, js and image files Review by @heathermiller, @kzys.
| * Scaladoc class diagrams part 1Vlad Ureche2012-06-1420-162/+1173
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit contains model changes required for adding class diagrams to scaladoc. It also contains an improved implicit shadowing computation, which hides the shadowed implicitly inherited members from the main view and gives instructions on how to access them. This is joint work with Damien Obrist (@damienobrist) on supporting diagram generation in scaladoc, as part of Damien's semester project in the LAMP laborarory at EPFL. The full history is located at: https://github.com/damienobrist/scala/tree/feature/diagrams-dev Commit summary: - diagrams model - diagram settings (Settings.scala, ScalaDoc.scala) - diagram model object (Entity.scala, Diagram.scala) - model: tracking direct superclasses and subclasses, implicit conversions from and to (ModelFactory.scala) - diagram object computation (DiagramFactory.scala, DocFactory.scala) - capacity to filter diagrams (CommentFactory.scala, DiagramDirectiveParser.scala) - diagram statistics object (DiagramStats.scala) - delayed link evaluation (Body.scala, Comment.scala) - tests - improved implicits shadowing information - model shadowing computation (ModelFactoryImplicitSupport.scala, Entity.scala) - html generation for shadowing information (Template.scala) - tests Also fixes an issue reported by @dragos, where single-line comment expansion would lead to the comment disappearing. Review by @kzys, @pedrofurla.
* | Merge pull request #727 from scalamacros/topic/classtag-to-classmanifestJosh Suereth2012-06-151-1/+1
|\ \ | | | | | | classtag => classmanifest conversion no longer requires runtime universe
| * | classtag => classmanifest conversion no longer requires runtime universeEugene Burmako2012-06-151-1/+1
| | |
* | | Merge pull request #721 from viktorklang/wip-sensible-deprecation-msgs-√Josh Suereth2012-06-151-26/+26
|\ \ \ | | | | | | | | Wip sensible deprecation msgs √
| * | | Removing erronous quoting of implicitNotFoundViktor Klang2012-06-141-2/+2
| | | |
| * | | Adding backticks to code in deprecation messages in PredefViktor Klang2012-06-141-28/+28
| | | |
| * | | Clarifying deprecation for implicit and explicit usage of the x2y's in PredefViktor Klang2012-06-131-24/+24
| | | |
| * | | Clarifying deprecation messages for numeric conversions in PredefViktor Klang2012-06-131-24/+24
| | |/ | |/|
* | | fixes SI-5912Eugene Burmako2012-06-151-1/+4
| |/ |/|
* | Fix for broken forwarder.Paul Phillips2012-06-141-1/+1
| | | | | | | | | | | | | | Wow, I wonder what the upper bound is on how long I might have chased my tail before finding this. I know what the lower bound is, unfortunately. It's just that usually the answer to a mystery is not "the empty package class has magically become the root class, but only sometimes."
* | Merge pull request #713 from havocp/havocp-task-stackAdriaan Moors2012-06-142-49/+82
|\ \ | | | | | | Fixups to Future task stack
| * | rework Future.dispatchFuture a bit to fix bugs / optimizeHavoc Pennington2012-06-132-49/+82
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a bug where _taskStack could batch a task into the wrong executor, as previously commented in the code. It now uses the _taskStack machinery for the Future.apply dispatch in addition to callback dispatch, so we can batch Future(body) as well. Less significantly, it micro-optimizes by combining some different closures and Runnable into a Task object, so there aren't as many objects created when storing and dispatching a callback. So it saves a bit of memory and runtime perhaps.
* | Merge pull request #706 from scalamacros/topic/fixcheckinitJosh Suereth2012-06-131-0/+2
|\ \ | |/ |/| fixes a checkinit problem
| * fixes a checkinit problemEugene Burmako2012-06-131-0/+2
| |
* | Merge pull request #705 from paulp/topic/uncheckedAdriaan Moors2012-06-131-1/+2
|\ \ | | | | | | Suppress non-local return unchecked warnings.
| * | Suppress non-local return unchecked warnings.Paul Phillips2012-06-121-1/+2
| | | | | | | | | | | | | | | There doesn't seem to be any way to do that by adding a synthetic annotation.
* | | Breaks.break should return Nothing, not UnitDan Brown2012-06-121-1/+1
|/ /
* | Forkjoin and fjbg are now always compiled in the build.Josh Suereth2012-06-111-0/+0
| | | | | | | | | | * forkjoin.done/forkjoine.clean can test forkjoin source * fjbg.done/fjbg.clean can test fjbg source.
* | Merge pull request #698 from retronym/ticket/5696Adriaan Moors2012-06-112-0/+7
|\ \ | | | | | | SI-5696 Detect excess constructor argument lists.
| * | SI-5696 Detect excess constructor argument lists.Jason Zaugg2012-06-102-0/+7
| | | | | | | | | | | | An apply method fooled the usual mechanism.
* | | SI-5162 Exclude super.foo from the erasure cast of SI-4283Jason Zaugg2012-06-111-9/+18
|/ / | | | | | | | | | | | | | | | | If the target method is defined in Java, treat the super reference as an error, otherwise allow it in the knowledge that Scala loosens the access restrictions on its generated classes. Moves the test for that bug out of pending-ville. It's sufficient to place Test in the empty package to exercise the right code paths.
* | Merge pull request #694 from retronym/ticket/4831-2Adriaan Moors2012-06-091-3/+3
|\ \ | | | | | | SI-4831 Fix ambiguous import detection for renamed imports.
| * | SI-4831 Fix ambiguous import detection for renamed imports.Jason Zaugg2012-06-091-3/+3
| |/
* | Merge pull request #696 from retronym/topic/intellij-reflect-module3Adriaan Moors2012-06-093-1/+27
|\ \ | | | | | | An IntelliJ module for reflect.
| * | An IntelliJ module for reflect.Jason Zaugg2012-06-093-1/+27
| |/
* / better unreachability for selectionsAdriaan Moors2012-06-091-12/+34
|/ | | | | | | | | | | | | | | Consts are hashconsed modulo static-approximation-for-dynamic-value-equality thus, two value-equality tests in patterns should reuse the same ValueConst if and only if the tested values are guaranteed to be equal in all possible executions the implementation uses unique types to track unique consts for an Ident with a stable symbol, we simply use the corresponding singleton type for a Select, we have to indirect some more: we store all the unique trees we've encountered and a unique type for each of them this unique type is then used to find the uniqut const that approximates the run-time value this may seem roundabout, but we need to standardize on types for representing "value" tests, as a type test against a singleton type must give rise to the same ValueConst as a value test using a tree that refers to the same symbol as the singleton type test
* A quickfix for 09bf95675bEugene Burmako2012-06-091-1/+1
|
* Merge pull request #660 from retronym/ticket/5167-3Josh Suereth2012-06-081-5/+15
|\ | | | | SI-5167 An impl class method should refer to its own parameter symbols.
| * SI-5167 An impl class method should refer to its own parameter symbols.Jason Zaugg2012-06-031-5/+15
| | | | | | | | | | | | | | | | Rather than those of the original method in the trait. If they are shared, parameter renaming in the implementaion class is visible in the original method. This led to a crash in the resident compiler when looking up the default argument getter.
* | Merge pull request #631 from retronym/ticket/5652-3Adriaan Moors2012-06-081-12/+21
|\ \ | | | | | | SI-5652 Mangle names of potentially public lambda lifted methods.
| * | SI-5652 Mangle names of potentially public lambda lifted methods.Jason Zaugg2012-05-261-12/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This can happen if they are accessed from an inner class. If a subclass is happens to lift a public method to the same name, a VerifyError ensues. The enclosed tests: - demonstrate the absense of the VerifyError - show the names generated for the lifted methods (which are unchanged if not called from an inner class, or if lifted into a trait implementation class.) - ensure that the callers are rewritten to call the correct method when multiple with the same name are lifted. It's not ideal that this phase needs a priori knowledge of the later phases to perform this mangling. A better fix would defer this until the point when the methods are publicised, and leave the unmangled private method in place and install an public, mangled forwarder.
* | | Merge pull request #679 from axel22/issue/5853Adriaan Moors2012-06-082-1/+17
|\ \ \ | | | | | | | | Fix SI-5853.
| * | | Fix SI-5853.Aleksandar Prokopec2012-06-072-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This solves two issues. First, up to now the newly generated symbols for normalized members were not being added to the declaration list of the owner during `specialize`. Now they are. Second, during `extmethods`, the extension methods generated get an additional curried parameter list for `$this`. Trouble was, after that, during `uncurry` and before `specialize`, these curried parameter lists were merged into one list. Specialization afterwards treats extension methods just like normal methods and generates new symbols without the curried parameter list. The `extensionMethod` now takes this into account by checking if the first parameter of a potential extension method has the name `$this`. Review by @dragos. Review by @odersky.
* | | | Avoids cyclic reference error in AddInterfaces.Martin Odersky2012-06-082-68/+70
| | | | | | | | | | | | | | | | | | | | | | | | Compile Global or Typers twice in resident mode yielded a CyclicReference error, in phase erasure. The error was due to a cyclic call of isDerivedValue class. Checking I noted that isDerivedValue class is called very often, and mostly unneccessesarily. I tightened teh condition to not force the type of the class if it is a trait or package. This made the CyclicReference go away. Widened criterion that enables validation of derived value classes. Previous version made two neg tests slip through.
* | | | Reduce time spent in lubsMartin Odersky2012-06-087-78/+114
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce time spent in lubs by making depth more adaptive. It now takes into account separately the depth of the lub types and the maximal depth of theior base type sequences. It cuts depth more aggressively if it is the base types instead of the types themselves that grow deep. The old truncation behavior is retained under option -Xfull-lubs Another change is that we now track depth more precisely, which should also help or at least allow better statistics. Also added statistics that measure #lubs and time spent in them.
* | | | tests and fixes for the mirror APIEugene Burmako2012-06-083-75/+99
| | | |
* | | | don't warn about abstract types that are checkableAdriaan Moors2012-06-082-47/+58
| | | |
* | | | turn unchecked type patterns into checked onesAdriaan Moors2012-06-082-6/+73
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the pattern `(_: T)` is made checkable using (ct: ClassTag[T]).unapply by rewriting it to `ct(_: T)` (if there's a ClassTag[T] available) similarly for extractors: if the formal type of the unapply method is an uncheckable type, wrap in the corresponding classtag extractor (if available) don't trigger rewrite on non-toplevel unchecked types (i.e., only look at type constructor part of T when looking for unchecked types) TODO: find outer match to figure out if we're supposed to be unchecked would like to give users a chance to opt-out from the wrapping, but finding the match to which this pattern belongs turned out to be tricky...
* | | | de-duplicate isUncheckedAnnotation/isSwitchAnnotationAdriaan Moors2012-06-083-10/+10
| | | |