summary refs log tree commit diff stats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* fixes #21263; consider all candidates for concept matches (#21265)ringabout2023-01-171-0/+25
|
* fix #20253 (#21174)Bung2023-01-131-0/+10
| | | | | | | | | * fix #20253 * change NimbleStableCommit * Update koch.nim Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
* Fix getting custom pragma from generic object (#20481)Jake Leahy2023-01-111-0/+18
| | | | | | | | | | | * Merge devel Add another test case * Fix test Use getCustomPragmaVal instead of hasCustomPragma Co-authored-by: Clay Sweetser <Varriount@users.noreply.github.com>
* fix typeRel of typeDesc (#21227)Bung2023-01-091-0/+4
|
* fixes #21232; highlite.getNextToken() returns gtOperator for '/' in C (#21234)ringabout2023-01-081-0/+10
|
* fixes #21209; exclude threads options for JS (#21233)ringabout2023-01-082-0/+7
|
* underscores for routine parameters (#21192)metagn2023-01-081-0/+109
| | | | | | | | | * underscores for routine parameters fixes #13443, fixes #13804, refs #21121 * add changelog + more tests * support generics and ensure inferred lambdas work
* docgen: implement cross-document links (#20990)Andrey Makarov2023-01-042-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * docgen: implement cross-document links Fully implements https://github.com/nim-lang/RFCs/issues/125 Follow-up of: https://github.com/nim-lang/Nim/pull/18642 (for internal links) and https://github.com/nim-lang/Nim/issues/20127. Overview -------- Explicit import-like directive is required, called `.. importdoc::`. (the syntax is % RST, Markdown will use it for a while). Then one can reference any symbols/headings/anchors, as if they were in the local file (but they will be prefixed with a module name or markup document in link text). It's possible to reference anything from anywhere (any direction in `.nim`/`.md`/`.rst` files). See `doc/docgen.md` for full description. Working is based on `.idx` files, hence one needs to generate all `.idx` beforehand. A dedicated option `--index:only` is introduced (and a separate stage for `--index:only` is added to `kochdocs.nim`). Performance note ---------------- Full run for `./koch docs` now takes 185% of the time before this PR. (After: 315 s, before: 170 s on my PC). All the time seems to be spent on `--index:only` run, which takes almost as much (85%) of normal doc run -- it seems that most time is spent on file parsing, turning off HTML generation phase has not helped much. (One could avoid it by specifying list of files that can be referenced and pre-processing only them. But it can become error-prone and I assume that these linke will be **everywhere** in the repository anyway, especially considering https://github.com/nim-lang/RFCs/issues/478. So every `.nim`/`.md` file is processed for `.idx` first). But that's all without significant part of repository converted to cross-module auto links. To estimate impact I checked the time for `doc`ing a few files (after all indexes have been generated), and everywhere difference was **negligible**. E.g. for `lib/std/private/osfiles.nim` that `importdoc`s large `os.idx` and hence should have been a case with relatively large performance impact, but: * After: 0.59 s. * Before: 0.59 s. So Nim compiler works so slow that doc part basically does not matter :-) Testing ------- 1) added `extlinks` test to `nimdoc/` 2) checked that `theindex.html` is still correct 2) fixed broken auto-links for modules that were derived from `os.nim` by adding appropriate ``importdoc`` Implementation note ------------------- Parsing and formating of `.idx` entries is moved into a dedicated `rstidx.nim` module from `rstgen.nim`. `.idx` file format changed: * fields are not escaped in most cases because we need original strings for referencing, not HTML ones (the exception is linkTitle for titles and headings). Escaping happens later -- on the stage of `rstgen` buildIndex, etc. * all lines have fixed number of columns 6 * added discriminator tag as a first column, it always allows distinguish Nim/markup entries, titles/headings, etc. `rstgen` does not rely any more (in most cases) on ad-hoc logic to determine what type each entry is. * there is now always a title entry added at the first line. * add a line number as 6th column * linkTitle (4th) column has a different format: before it was like `module: funcName()`, now it's `proc funcName()`. (This format is also propagated to `theindex.html` and search results, I kept it that way since I like it more though it's discussible.) This column is what used for Nim symbols resolution. * also changed details on column format for headings and titles: "keyword" is original, "linkTitle" is HTML one * fix paths on Windows + more clear code * Update compiler/docgen.nim Co-authored-by: Andreas Rumpf <rumpf_a@web.de> * Handle .md and .nim paths uniformly in findRefFile * handle titles better + more comments * don't allow markup overwrite index title for .nim files Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
* fixes #21207; reports redefinition error in the definition of enums (#21217)ringabout2023-01-031-0/+9
| | | | | * fixes #21207; reports redefinition in the enums * add a test
* fixes pragmas reorder (#21205)ringabout2023-01-011-0/+19
|
* fix #15117 zero size array cause incorrect codegen for VCC compiler (#21197)Bung2022-12-301-0/+27
| | | fix #15117
* fixes #21195; `std/assertions` continue to use `sysFatal` when ↵ringabout2022-12-294-4/+25
| | | | | | | | | | | | | `nimPreviewSlimSystem` is not defined (#21196) * fixes #21195; `std/assertions` continue to use `sysFatal` * try includes * make `std/assertions` self-contained * fixes tests * fixes tests
* fixes #14444; add `genLineDir` before assignment (#21201)ringabout2022-12-291-0/+14
| | | | | | | * fixes #14444; add `genLineDir` before raises * add a test case * fixes differently
* less verbose type mismatch messages (#21191)ringabout2022-12-283-0/+45
| | | | | | | | | * less verbose type mismatch messages * Update compiler/types.nim * fixes i386 * fixes i386
* fixes #21171; dynamic acyclic refs need to use dyn decRef (#21184)ringabout2022-12-281-0/+77
| | | | | | | * fixes #21171; dyn destructors for acyclic inherited refs * add a test * Update compiler/liftdestructors.nim
* fix #14667 (#21190)Bung2022-12-281-0/+12
|
* fix for bad error message with const in case statement (#21182)metagn2022-12-271-0/+7
| | | | | | | * preliminary fix for bad error message with const * add test case * fix tmatrixconcept and tmatrixlib
* add test case for #20289 (#21180)Bung2022-12-271-0/+15
|
* enforce void for nkWhileStmt [backport: 2.0] (#21170)ringabout2022-12-241-0/+11
| | | enforce void for nkWhileStmt
* add the cpp target (#21164)ringabout2022-12-231-0/+4
| | | The issue is related to cpp codegen, the previous test doesn't test cpp backend, which will join into the megatest.
* fix #20997 calling system.card[T](x: set[T]) with T of int8 or uint8 … ↵Bung2022-12-231-0/+18
| | | | | | | (#21010) * fix #20997 calling system.card[T](x: set[T]) with T of int8 or uint8 uses mismatched C array sizes * fullfil set variant
* fix #11634 (#21146)Bung2022-12-231-0/+20
|
* fix #12946 Bad C++ codegen on distinct generics C++ types (#21157)Bung2022-12-231-0/+4
|
* Don't repeat suggestions for same symbol (#21140)Jake Leahy2022-12-223-27/+48
| | | | | | | | | | * Track seen module graphs so symbols from the same module aren't repeated Add test case * Track symbols instead of modules * Don't show duplicate symbols in spell checker Removes the declared location from the message. Since we don't show duplicates anymore it would be a bit misleading if we only show the location for the first declaration of the symbol
* Feature/xmltree additions (#20988)Michael Voronin2022-12-228-0/+402
| | | | | | | * [change] add/insert/delete family of xmltree expanded with several variations. Added replace methods family * [change] Lifted child limitations on insert methods (consulted with @araq) * [tests] add/insert/replace/delete of xmltree XmlNodes tests added
* fix #16541 (#21148)Bung2022-12-221-0/+12
|
* Implement setLineInfo (#21153)Peter Munch-Ellingsen2022-12-221-1/+22
| | | | | * Implement setLineInfo * Add tests
* fixes #21144; try expression will not match the less indentation except (#21152)ringabout2022-12-211-0/+27
| | | fixes #21144; try expression will not match the less indent except
* fixes #19292; fixes #21122; fixes putEnv and setEnv with vcc (#21143)ringabout2022-12-201-0/+6
| | | | | * fixes #19292; fixes 21122; fixes putEnv and setEnv with vcc * add a test
* fix #20248;fix #6215;turns into simple CT error (#21141)Bung2022-12-201-0/+14
|
* fix #21109 (#21127)Bung2022-12-191-0/+13
|
* close #11705; add a testcase (#21128)ringabout2022-12-181-0/+17
|
* fixes #21116; always mangles the param (#21121)ringabout2022-12-172-1/+11
| | | | | | | | | * fixes #21116; always mangles the param * idOrSig * fixes tests * Apply suggestions from code review
* fix #12122 (#21096)Bung2022-12-161-1/+1
|
* Check file exists in `{.compile.}` pragma (#21105)Jake Leahy2022-12-151-0/+5
| | | | | | | | | * Add test * Check file exists before adding it into compilation * Make error message look like other error messages i.e. following the format `error msg: file`
* fix #20588 (#21104)Bung2022-12-151-0/+22
|
* Make async stacktraces less verbose (#21091)Jake Leahy2022-12-152-39/+10
| | | | | | | | | | | | | | | | | | | | | | | * Name iterators something human readable Remove intermediate async procs from stacktraces Clean async traceback message from reraises message * Remove unused import/variable * Fix failing tests Don't add {.stackTrace: off.} to anonymous procs (They already don't appear in stacktrace) * Fix failing tests in pragma category Now check that the nim is a routine type first so we don't run into any assertion defects * Hide stack trace pragma in docs and update doc tests User doesn't need to know if something won't appear so this more becomes verbose noise If this is a bad idea we can always add a `when defined(nimdoc)` switch so we don't add {.stackTrace: off.} to the Future[T] returning proc for docs
* fix #19580; add warning for bare except: clause (#21099)ringabout2022-12-151-1/+1
| | | | | | | | | | | | | | | | | | | | | * fix #19580; add warning for bare except: clause * fixes some easy ones * Update doc/manual.md * fixes docs * Update changelog.md * addition * Apply suggestions from code review Co-authored-by: Jacek Sieka <arnetheduck@gmail.com> * Update doc/tut2.md Co-authored-by: Jacek Sieka <arnetheduck@gmail.com>
* generic `define` pragma + string alias (#20979)metagn2022-12-131-16/+48
| | | | | | | | | * generic `define` pragma + string alias * clean * add tests and document * remove char/float, minimize changelog
* Fix #20416. Enable the recursion limit for ref/ptr types. (#21092)Aditya Siram2022-12-131-0/+20
|
* JS backend properly extends string with `setLen` (#21087)Jake Leahy2022-12-131-0/+11
| | | | | * Add test case * Extend string with '0' when setting length to be longer
* Refactor JS sourcemap generator (#21053)Jake Leahy2022-12-131-0/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Parse the new line format * Fix pattern Didn't have space after the path * Remove duplicate sources Sources would sometimes double up with a new line at the end * Remove unused variable * Refactor sourcemap.nim Removes the multiple translations needed, now goes from single high level type to the final SourceMap Adds documentation for procs * Line numbers line up properly now Files aren't linking correctly though * Files now link up correctly Lines are sometimes off but overall seems pretty good Just need to implement parser * Add column info to output Add sourceMappingURL to rope directly to prevent copy * Properly handle columns * Remove debug lines * Add testcase * Finish testcase * Use the outdir folder instead of the folder the test is in to find the sourcemap Co-authored-by: ringabout <43030857+ringabout@users.noreply.github.com>
* fixes #21043; fixes a named exception in the infixAs expression which ↵ringabout2022-12-122-1/+23
| | | | | | | | | | | generate an implicit uninitialized let statement (#21081) * fixes #21043; fixes a named exception in the infixAs expression which generate an implicit uninitialized let statement * Update compiler/sempass2.nim Co-authored-by: Andreas Rumpf <rumpf_a@web.de> Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
* closes #20808 (#21077)Andreas Rumpf2022-12-121-0/+35
|
* ship a modern nimble with lock files support (#21061)ringabout2022-12-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | * change `include genode/env` to an import ref https://github.com/nim-lang/Nim/commit/0b262e9496387d5e8adc0c5f6020b3f3300e8f79#diff-8718bd20d8f61d6638d3d64b19efc31bcd40a6d5be8215b2a1f0b75ed93e8d56 * fixes comments * ship a modern nimble with lock files support * not sure whether the latest nimble has a regression now I'm trying 0.14.0 * change `pkgs` to `pkgs2` (#21073) * overwrite problematic packages * Apply suggestions from code review * Apply suggestions from code review * Apply suggestions from code review
* fix #15836 proc arg return type auto unexpectly match proc with concr… ↵Bung2022-12-125-1/+61
| | | | | | | | | (#21065) * fix #15836 proc arg return type auto unexpectly match proc with concrete type * fix #16244 * add test case for #12869
* alternative, much simpler algorithm for strict func checking (#21066)Andreas Rumpf2022-12-114-15/+72
| | | | | | | | | | | | | * alternative, much simpler algorithm for strict func checking * forgot to git add new compiler module * new spec is incredibly simple to describe * fixes bigints regression * typos * closes #16305; closes #17387; closes #20863
* `multisync` now allows tuples in return type (#21074)Jake Leahy2022-12-111-0/+19
| | | | | | | * Add test case * Use .toStrLit() on param node first This means that more complex types are fully rendered
* fix #16758 Nim crashes in fixAbstractType (#20855)Bung2022-12-111-0/+38
| | | | | | | | | * fix #16758 Nim crashes in fixAbstractType * Update compiler/semexprs.nim Co-authored-by: Andreas Rumpf <rumpf_a@web.de> Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
* fix #21045; getTime with vmopsDanger is broken; alternative to #21054 (#21056)ringabout2022-12-101-0/+10
| | | | | * fix #21045 getTime with vmopsDanger is broken; alternative to #21054 * typo