summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* SI-5841 reification of renamed importsAndriy Polishchuk2012-12-0616-0/+141
| | | | | | Reification of renamed imports is done by catching Selects with name != their tree.symbol.name, replacing this name with tree.symbol.name, and then doing reifyProduct in case of renamed terms and reifyBoundType (inner) in case of renamed types.
* Merge pull request #1648 from paulp/unused-bonanza-2Adriaan Moors2012-11-21303-4427/+338
|\ | | | | Removing unused code, take 2.
| * Remove code from compiler central.Paul Phillips2012-11-2052-1146/+56
| | | | | | | | | | All those old-timey methods whose melodies have become unfashionable.
| * Remove code from misc bits of the compiler.Paul Phillips2012-11-209-47/+11
| | | | | | | | | | | | They are everywhere. They defy categorization. They are... M I S C
| * Removed code from scaladoc.Paul Phillips2012-11-209-131/+13
| | | | | | | | Nobody is immune!
| * Removed code from the typechecker.Paul Phillips2012-11-2012-251/+60
| | | | | | | | | | Removing code from this neighborhood is more difficult than elsewhere, making it all the more important that it be done.
| * Members removed in scala.reflect.Paul Phillips2012-11-2038-947/+39
| | | | | | | | | | Not a bad showing for a newcomer. Of course most of this code predates scala.reflect by a lot.
| * Members removed in the backend.Paul Phillips2012-11-1917-247/+4
| |
| * Members removed from partest.Paul Phillips2012-11-1911-67/+5
| | | | | | | | It has accreted its share through the bumpy years.
| * Members removed from scala.reflect.io.Paul Phillips2012-11-199-189/+20
| | | | | | | | | | Mostly hailing from a long-ago day when I imagined I was writing a general purpose library. We dodged that bullet.
| * Members removed from the repl.Paul Phillips2012-11-1932-512/+25
| | | | | | | | There were a whole lot of these.
| * Some files removed in their entirety.Paul Phillips2012-11-199-274/+8
| | | | | | | | And small associated changes.
| * Removing ancient comments and pointless comments.Paul Phillips2012-11-1933-356/+77
| | | | | | | | | | | | | | | | | | | | Translating <code></code> into backticks. Removed the "@param tree ..." blocks which have been taunting me for half a decade now. Removed commented-out blocks of code which had been sitting there for two years or more.
| * Revert "Commenting out unused members."Paul Phillips2012-11-19193-2579/+2660
| | | | | | | | This reverts commit 951fc3a486.
| * Commenting out unused members.Paul Phillips2012-11-19193-2662/+2581
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I want to get this commit into the history because the tests pass here, which demonstrates that every commented out method is not only unnecessary internally but has zero test coverage. Since I know (based on the occasional source code comment, or more often based on knowing something about other source bases) that some of these can't be removed without breaking other things, I want to at least record a snapshot of the identities of all these unused and untested methods. This commit will be reverted; then there will be another commit which removes the subset of these methods which I believe to be removable. The remainder are in great need of tests which exercise the interfaces upon which other repositories depend.
| * applyOrElse is a synthetic method.Paul Phillips2012-11-191-1/+1
| |
| * Deal with possibly spurious warning in Macros.Paul Phillips2012-11-191-4/+4
| |
| * Remove unused imports in library.Paul Phillips2012-11-1970-185/+13
| |
| * Remove unused imports in continuations.Paul Phillips2012-11-194-14/+0
| |
| * Remove unused imports in partest.Paul Phillips2012-11-196-9/+1
| |
| * Removed unused imports in swing.Paul Phillips2012-11-1914-47/+1
| |
* | Merge pull request #1640 from paulp/xdevAdriaan Moors2012-11-2023-42/+59
|\ \ | | | | | | Added -Xdev setting... you know, for devs
| * | Added -Xdev setting... you know, for devsPaul Phillips2012-11-1623-42/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A setting we developers can give all the time and expect to hear useful things without being buried in debugging output. As the comment says: This is for WARNINGS which should reach the ears of scala developers whenever they occur, but are not useful for normal users. They should be precise, explanatory, and infrequent. Please don't use this as a logging mechanism. !!! is prefixed to all messages issued via this route to make them visually distinct. This is what I always intended for "debugwarn", the method I have deprecated in favor of the more accurate: def devWarning(msg: => String): Unit In this VERY SAME COMMIT, I performed the CLOSELY RELATED task of quieting down an -Xlint warning which had become too noisy thanks to implicit classes tickling it. I tightened that warn condition to include both -Xlint and -Xdev.
* | | Merge pull request #1598 from dcsobral/mailmapJosh Suereth2012-11-201-0/+7
|\ \ \ | |_|/ |/| | Associate correct names to e-mails, correct or not
| * | Associate correct names to e-mails, correct or notDaniel C. Sobral2012-11-091-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is used by git log to produce names with certain formats, and is used by the release notes tool as well. Helpful if you have commits with Author not set correctly, as did I. I'm also including Aleksandar Prokopec and his friend Aleksandar Pokopec, as well as Pavel Pavlov, both of whom have commits appearing under more than one name. I'm not including others who have not set a full name, but don't have commits appearing under more than one name.
* | | Merge pull request #1626 from paulp/issue/6357Adriaan Moors2012-11-193-1/+12
|\ \ \ | | | | | | | | Fix for SI-6357, cycle with value classes.
| * | | Fix for SI-6357, cycle with value classes.Paul Phillips2012-11-143-1/+12
| | | | | | | | | | | | | | | | Don't force the owner info.
* | | | Merge pull request #1581 from lrytz/typerCommentAdriaan Moors2012-11-161-2/+3
|\ \ \ \ | | | | | | | | | | comment / question in typers
| * | | | comment / question in typersLukas Rytz2012-11-061-2/+3
| | | | |
* | | | | Merge pull request #1635 from paulp/slick-regression-2Adriaan Moors2012-11-162-11/+38
|\ \ \ \ \ | | | | | | | | | | | | Fix for overly eager package object initialization.
| * | | | | Fix for overly eager package object initialization.Paul Phillips2012-11-152-11/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A subtle change in the order in which symbol attributes were inspected (now you know why I avoid vals in the compiler) led to a cycle during initialization for slick. I'm afraid I don't know how to reproduce the issue outside of slick and sbt, so I added some logging instead. After some challenges juggling general correctness and cycle avoidance, I resorted to improving and documenting the logic as well. I predict reviewer will be pleased.
* | | | | | Merge pull request #1586 from retronym/ticket/6247Adriaan Moors2012-11-165-15/+105
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | Optimize primitive Array(e1, ..., en)
| * | | | | More principled tree copying.Jason Zaugg2012-11-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Canonical > home-spun.
| * | | | | Optimize primitive Array(e1, ..., en)Jason Zaugg2012-11-075-15/+105
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Expands an existing optimization for reference arrays to apply to primitives, as well. Fixes one aspect of SI-6247.
* | | | | | Merge pull request #1623 from paulp/merge-2.10.xAdriaan Moors2012-11-1625-39/+219
|\ \ \ \ \ \ | | | | | | | | | | | | | | Merge 2.10.0-wip/.x into master
| | \ \ \ \ \
| | \ \ \ \ \
| *-. \ \ \ \ \ Merge remote-tracking branches 'origin/2.10.x' and 'origin/2.10.0-wip' into ↵Paul Phillips2012-11-1425-39/+219
| |\ \ \ \ \ \ \ | | |_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | master Conflicts: src/compiler/scala/tools/nsc/transform/ExtensionMethods.scala
| | | * | | | | Merge pull request #1608 from retronym/ticket/6644Josh Suereth2012-11-142-3/+9
| | | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | SI-6644 Account for varargs in extmethod forwarder
| | | | * | | | | SI-6644 Account for varargs in extmethod forwarderJason Zaugg2012-11-102-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Which sounded difficult, so instead I offshored the work to the friendly republic of TreeGen.
| | | * | | | | | Merge pull request #1601 from cunei/patch-1Josh Suereth2012-11-141-1/+1
| | | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Update tools/epfl-publish
| | | | * | | | | | Update tools/epfl-publishcunei2012-11-091-1/+1
| | | | |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, during each Scala nightly build, the epfl-publish script rsyncs the resulting dists/archives to the repository machine (chara). Now, as a result of a change introduced a while back, dists/archives/ now also contains a symbolic link, created at the end of the build when the targets "pack-archives.latest.*" are invoked (in src/build/pack.xml). That was introduced in: scala/scala@506bcfe71c93160ebfa0ca9b8b170b4b54e844e9 scala/scala@cb99853c8655686dae1288cbcd44a42cf1ea6609 This link, created in dists/archives/, is copied over as-is at the end of the rsync to chara by epfl-publish. On chara, however, the link points to an invalid target (the path is absolute). Separately, the repository directory on chara is rsync'd every 30 minutes over to the machine that serves www.scala-lang.org, via cron. Rsynch finds that the link target does not exist, and generates an email with the crontab log, which is then duly sent to a system mailbox. Every 30 minutes. Since March. Needless to say, the mailbox is pretty large by now. The fix is trivial, but this needs to be included in all the branches that are tested during the nightly. Since RC2 has just been cut, and the code frozen, I'm not sure how this change can be included without disruption; therefore, I'll just send a pull request on 2.10.0-wip, and let Josh/Paul take the appropriate steps.
| | | * | | | | | Merge pull request #1606 from retronym/ticket/6646Adriaan Moors2012-11-134-19/+87
| | | |\ \ \ \ \ \ | | | | |/ / / / / | | | |/| | | | | SI-6646 Fix regression in for desugaring.
| | | | * | | | | Update comment.Jason Zaugg2012-11-131-1/+2
| | | | | | | | |
| | | | * | | | | SI-6646 `ident` or Ident is always new binding.Jason Zaugg2012-11-133-25/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous commit regressed in these cases: // no withFilter for (X <- List("A single ident is always a pattern")) println(X) for (`x` <- List("A single ident is always a pattern")) println(`x`) At the top level of the LHS of a <-, such identifiers represent new bindings, not stable identifier patterns.
| | | | * | | | | SI-6646 Fix regression in for desugaring.Jason Zaugg2012-11-104-19/+44
| | | |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The early check in the parser of pattern irrefutability, added in c82ecab, failed to consider InitCaps and `backquoted` identifiers.
| | * | | | | | Merge pull request #1585 from retronym/ticket/6539-2Eugene Burmako2012-11-147-0/+57
| | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | SI-6539 Annotation for methods unfit for post-typer ASTs
| | | * | | | | | Refine @compileTimeOnlyJason Zaugg2012-11-086-23/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Don't default the message, and show it exclusively. - Fix cut-and-pasto in the @since tag - Be tolerant if the annotaion class is missing, as seems to have been the case compiling the continuations plugin. - s/\t/ / in the test file to show the errors are positioned correctly. - Use defensive getOrElse
| | | * | | | | | SI-6539 Annotation for methods unfit for post-typer ASTsJason Zaugg2012-11-067-0/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Motivated by the `.value` method in the SBT task-syntax branch, which should only be called within the context of the argument to a setting initialization macro. The facility is akin to a fatal deprecation.
| | * | | | | | | Merge pull request #1618 from scalamacros/ticket/6662Eugene Burmako2012-11-144-1/+18
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix for SI-6662, macro failing too early.
| | | * | | | | | | Fix for SI-6662, macro failing too early.Paul Phillips2012-11-144-1/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't give up before you try tupling. Who knows what someone might be doing with a Unit.
| | * | | | | | | | Merge pull request #1583 from dragos/issue/fix-6616Adriaan Moors2012-11-134-9/+41
| | |\ \ \ \ \ \ \ \ | | | |/ / / / / / / | | |/| | | | | | | SI-6616 Check that unsafe operations are only called on the presentation...
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-02 18:14:25 +0000