summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Merge pull request #5754 from Philippus/issue/html-tag-in-hoverLukas Rytz2017-03-043-30/+25
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | fix regression in scaladoc
| * | | | | | pattern for entitylink was too narrow, cleaned up the testsPhilippus Baalman2017-03-033-30/+25
| | | | | | |
* | | | | | | Merge pull request #5671 from retronym/topic/stubby-2Lukas Rytz2017-03-035-7/+68
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Avoid compiler crash with missing transitive dependencies
| * | | | | | | Remove non-essential fix for stub symbol failureJason Zaugg2017-03-031-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Given that we correctly setup the flags on the stub symbol, we no longer trip an assertion in ModuleTypeRef's constructor.
| * | | | | | | Avoid forcing info transforms of primitive methodsJason Zaugg2017-02-192-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Although this is cheap, when debugging log output of info transformer activity this was a major source of noise. This commit avoids the info lookup for methods other than `+`, and then for `+` uses the typer phase info to distinguish concatentation from addition.
| * | | | | | | Avoid stub symbol related crash in backendJason Zaugg2017-02-194-2/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In this test case, the backend forces the specialization info transform of `Sub` during computation of its inner class metadata. This in turn runs the info transforms of the `Base`. This leads to the uncurry info tranform transforming a signature that has a type alias as a method parameter type. Subsequent substution of the new method symbol into the result type, which includes a stub symbol for an absent class, tripped an assertion: ``` requirement failed: package b java.lang.IllegalArgumentException: requirement failed: package b at scala.Predef$.require(Predef.scala:277) at scala.reflect.internal.Types$ModuleTypeRef.<init>(Types.scala:1879) at scala.reflect.internal.Types$PackageTypeRef.<init>(Types.scala:1897) at scala.reflect.internal.Types$TypeRef$.apply(Types.scala:2401) at scala.reflect.internal.Types.typeRef(Types.scala:3553) at scala.reflect.internal.Types.typeRef$(Types.scala:3536) at scala.reflect.internal.SymbolTable.typeRef(SymbolTable.scala:16) at scala.reflect.internal.Symbols$TypeSymbol.newTypeRef(Symbols.scala:3026) at scala.reflect.internal.Symbols$TypeSymbol.updateTypeCache(Symbols.scala:3079) at scala.reflect.internal.Symbols$TypeSymbol.maybeUpdateTypeCache(Symbols.scala:3065) at scala.reflect.internal.Symbols$TypeSymbol.tpe_$times(Symbols.scala:3043) at scala.reflect.internal.Symbols$Symbol.typeOfThis(Symbols.scala:2020) at scala.reflect.internal.Types$ThisType.underlying(Types.scala:1184) at scala.reflect.internal.Types$SimpleTypeProxy.boundSyms(Types.scala:150) at scala.reflect.internal.Types$SimpleTypeProxy.boundSyms$(Types.scala:150) at scala.reflect.internal.Types$SingletonType.boundSyms(Types.scala:1088) at scala.reflect.internal.tpe.TypeMaps$SubstMap.apply(TypeMaps.scala:726) at scala.reflect.internal.tpe.TypeMaps$SubstSymMap.apply(TypeMaps.scala:789) at scala.reflect.internal.tpe.TypeMaps$TypeMap.mapOver(TypeMaps.scala:102) at scala.reflect.internal.tpe.TypeMaps$SubstSymMap.apply(TypeMaps.scala:783) at scala.reflect.internal.tpe.TypeMaps$TypeMap.mapOver(TypeMaps.scala:102) at scala.reflect.internal.tpe.TypeMaps$SubstSymMap.apply(TypeMaps.scala:783) at scala.reflect.internal.Types$Type.substSym(Types.scala:727) at scala.reflect.internal.tpe.TypeMaps$TypeMap.mapOver(TypeMaps.scala:123) at scala.reflect.internal.transform.UnCurry$$anon$1.apply(UnCurry.scala:53) at scala.reflect.internal.transform.UnCurry.transformInfo(UnCurry.scala:154) ``` This commit address the direct failure above by setting coherent flags on the stub package class symbol (it also needs the MODULE flag).
* | | | | | | | Merge pull request #5622 from edmundnoble/extra-errsAdriaan Moors2017-03-027-39/+74
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Improved error messages for identically named, differently prefixed types
| * | | | | | | | Match error lengthsEdmund Noble2017-02-072-3/+6
| | | | | | | | |
| * | | | | | | | Improved error messages for identically named, differently prefixed typesEdmund Noble2016-12-317-39/+71
| | | | | | | | |
* | | | | | | | | Merge pull request #5728 from Philippus/issue/html-tag-in-hoverLukas Rytz2017-03-026-32/+90
|\ \ \ \ \ \ \ \ \ | | |_|/ / / / / / | |/| | | / / / / | |_|_|_|/ / / / |/| | | | | | | inlineToStr is not exhaustive and does not remove html tags inside HtmlTag [ci: last-only]
| * | | | | | | added missing Inline matches to inlineToStr so it is now exhaustivePhilippus Baalman2017-02-256-32/+90
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | scala.xml.XML.loadString(tag).text will remove all html tags inside the HtmlTag use a regex to remove html tags inside the tag added some tests for the inlineToStr-method moved inlineToStr to companion object of Page added test for nested html tags
* | | | | | | Merge pull request #5743 from som-snytt/issue/10207-bad-updateLukas Rytz2017-02-273-5/+26
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | SI-10207 Error before update conversion
| * | | | | | | SI-10207 Error before update conversionSom Snytt2017-02-263-5/+26
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Gaze deeper for errors before committing to conversion of assignment to update. The error buried in the transformed tree escapes notice of retypechecking and leaks to backend.
* | | | | | | Merge pull request #5746 from paulp/pr/partestLukas Rytz2017-02-272-0/+7
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Add partest paths to the list of watched sources.
| * | | | | | Add partest paths to the list of watched sources.Paul Phillips2017-02-262-0/+7
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | This allows running partest continuously (e.g. ~partest a/b/test) with it triggering on changes to the test source.
* | | | | | Merge pull request #5732 from retronym/topic/build-info-malarkeyAdriaan Moors2017-02-241-0/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | More predictable performance of SBT build startup, reload
| * | | | | | More predictable performance of SBT build startup, reloadJason Zaugg2017-02-231-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disable parallelism to avoid a nasty interaction between the SBT build info plugin, which internally uses `EvaluateTask`, and can get into a race condition with other concurrnently running tasks. This could be seen as frequent, unnecessary Ivy resolution during the `reload` command, even when nothing had changed. Gory details in https://github.com/sbt/sbt/issues/2970
* | | | | | | Merge pull request #5735 from SethTisue/sd-313Adriaan Moors2017-02-241-12/+18
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | increase timeouts on some sys.process tests
| * | | | | | increase timeouts on some sys.process testsSeth Tisue2017-02-231-12/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | these were occasionally timing out on our Windows CI reference: https://github.com/scala/scala-dev/issues/313
* | | | | | | Merge pull request #5723 from dragos/issue/regression-assert-ideLukas Rytz2017-02-241-5/+10
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Fix regression introduced by 5751763
| * | | | | | Fix regression in 5751763Iulian Dragos2017-02-221-5/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | enterClass/Module may return an existing symbol, but in 5751763 the return value was dropped leading to assertion failures. This may show up only in the presentation compiler, which explains why it went unnoticed. Here's what needs to happen: - a class with a companion is loaded by the IDE, but the class name is different than the file name. This is from source - the same class and companion object exist as binary, and are loaded from classfiles when the package is completed (since they have different names than the source file, the classpath abstraction will only "know" that there is a classfile, and no corresponding source file) It seems that companionClass always prefers to return the companion defined in a source file, but if this assertion is called from the code path that tries to load the binary version, the newly created module will not match.
* | | | | | | Merge pull request #5731 from janekdb/issue/GH-644/fix-spec-latex-renderingSeth Tisue2017-02-231-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | GH-644: Remove static html styling of spec code blocks
| * | | | | | | GH-644: Remove static html styling of spec code blocksJanek Bogucki2017-02-231-1/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to this Jekyll configuration change code blocks were being marked up in a ways that disrupted MathJax styling. Tested as far a possible locally. Confidence mainly drawn from html comparison to working 2.11.x version.
* | | | | | | Merge pull request #5729 from scala/revert-5658-topic/hashhashAdriaan Moors2017-02-223-24/+6
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Revert "Fix erasure of the qualifier of ##"
| * | | | | | Revert "Fix erasure of the qualifier of ##"Adriaan Moors2017-02-223-24/+6
|/ / / / / /
* | | | | | Merge pull request #5681 from Philippus/issue/9704Lukas Rytz2017-02-223-3/+33
|\ \ \ \ \ \ | | | | | | | | | | | | | | SI-9704 don't add a closing HtmlTag if it is already closed
| * | | | | | moved Pattern and TagsNotToClose to a HtmlTag companion objectPhilippus Baalman2017-02-211-4/+6
| | | | | | |
| * | | | | | SI-9704 don't add a closed HtmlTag if it is already closedPhilippus Baalman2017-02-083-1/+29
| | | | | | |
* | | | | | | Merge pull request #5726 from scala/revert-5629-issue/10120-quote-errAdriaan Moors2017-02-2116-84/+76
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Revert "SI-10133 Require escaped single quote char lit"
| * | | | | | | Revert "SI-10133 Require escaped single quote char lit"Adriaan Moors2017-02-2116-84/+76
|/ / / / / / /
* | | | | | | Merge pull request #5663 from ↵Adriaan Moors2017-02-214-2/+16
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gourlaysama/ticket/sd-256-enable-repl-colors-unix-2 Enable colored output by default on unix Fix scala/scala-dev#256
| * | | | | | | SD-256 enable colored output by default on unixAntoine Gourlay2017-02-214-2/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `scala.color` now has 3 states: `true`, `false` and `auto` (the default). `auto` allows colors if not on windows and if the shell is interactive (as in, both stdin and stdout are a tty). The autodetect works as expected when run via SBT too, and it can always be overriden on the CLI or via JAVA_OPTS.
* | | | | | | | Merge pull request #5658 from retronym/topic/hashhashLukas Rytz2017-02-213-6/+24
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Fix erasure of the qualifier of ##
| * | | | | | | Fix erasure of the qualifier of ##Jason Zaugg2017-01-243-6/+24
| | | | | | | |
* | | | | | | | Merge pull request #5708 from szeiger/issue/si10194Lukas Rytz2017-02-212-7/+22
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | SI-10194: Fix abstract type resolution for overloaded HOFs
| * | | | | | | | SI-10194: Fix abstract type resolution for overloaded HOFsStefan Zeiger2017-02-212-7/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Types in the applicable overload alternatives need to be seen from the respective owners of the individual alternative, not from the target’s owner (which can be a subtype of the types that define the methods).
* | | | | | | | | Merge pull request #5700 from retronym/ticket/10154-refactorLukas Rytz2017-02-213-39/+43
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Refactor lookupCompanion
| * | | | | | | | | Refactor implementation of lookupCompanionJason Zaugg2017-02-193-39/+43
| | |_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Check for module class up front to use sourceModule - Consolidate most of the logic in Contexts
* | | | | | | | | Merge pull request #5704 from som-snytt/issue/10190-elide-stringLukas Rytz2017-02-214-16/+38
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | SI-10190 Elide string to empty instead of null
| * | | | | | | | | SI-10190 Elide string to empty instead of nullSom Snytt2017-02-154-16/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid NPE when eliding string-valued functions. For example, `log(s"$cheap$expensive")` needn't print null. This is a natural and inexpensive way to elide strings.
* | | | | | | | | | Merge pull request #5640 from optimizely/repl-import-handlerAdriaan Moors2017-02-207-19/+131
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | SI-9881 Fix ImportHandler's reporting of importedNames and importedSymbols
| * | | | | | | | | | Fix ImportHandler's reporting of importedNames and importedSymbolsHao Xia2017-01-115-17/+102
| | | | | | | | | | |
| * | | | | | | | | | Fix SIOOBE in Name#pos for substrings of length 1Jason Zaugg2017-01-102-2/+29
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #5629 from som-snytt/issue/10120-quote-errAdriaan Moors2017-02-2016-76/+84
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | SI-10133 Require escaped single quote char lit
| * | | | | | | | | | | SI-10120 ReplReporter handles message indentSom Snytt2017-01-1812-68/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of indenting source code to make messages align on output, let the reporter add indentation, only if the source is the console (and not a pastie or a loaded file). Previously, syntax errors were not indented. ``` $ skala Welcome to Scala 2.12.2-20170108-010722-939abf1 (Java HotSpot(TM) 64-Bit Server VM, Java 1.8.0_111). Type in expressions for evaluation. Or try :help. scala> 'abc' <console>:1: error: unclosed character literal (or use " for string literal "abc") 'abc' ^ scala> :quit $ scala Welcome to Scala 2.12.1 (Java HotSpot(TM) 64-Bit Server VM, Java 1.8.0_111). Type in expressions for evaluation. Or try :help. scala> 'abc' <console>:1: error: unclosed character literal 'abc' ^ ```
| * | | | | | | | | | | SI-10120 Extra advice on unclosed char literalSom Snytt2017-01-084-8/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Folks from other languages might mistakenly enclose a string in single quotes. Since this presents as a symbol literal followed by the unpaired single quote, we can add a syntax reminder. Also polish the wording for bad string interpolation.
| * | | | | | | | | | | SI-10133 Require escaped single quote char litSom Snytt2017-01-083-1/+13
| | |_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The spec specifically requires `'\''` and not `'''`. The error consumes all consecutive single quotes.
* | | | | | | | | | | Merge pull request #5660 from som-snytt/issue/9464-specAdriaan Moors2017-02-201-4/+6
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | SI-9464 Clarify spec on no final trait
| * | | | | | | | | | | SI-9464 Clarify spec on no final traitSom Snytt2017-01-241-4/+6
| | |_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without being too finicky with syntax, say that `final abstract class` is OK (even as a nested class, where it would be an incomplete member) but not `final trait`. Such a class might be a standard lib primitive, or might be an implicit value that is only a ludicrous marker and, if present, need only have the null value.
* | | | | | | | | | | Merge pull request #5659 from retronym/ticket/10026Adriaan Moors2017-02-206-6/+38
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | SI-10026 Fix endless cycle in runtime reflection
generated by cgit v1.2.3 (git 2.39.1) at 2024-08-18 08:40:11 +0000