summary refs log tree commit diff stats
Commit message (Collapse)AuthorAgeFilesLines
* fixes #19396; Nimdoc hide nonexported fields (#21305)ringabout2023-02-0113-24/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * suppresses non-exported fields of types and adds command-line option to re-enable this if desired * corrected the doctest that produced a CI error * an embarrassingly bad error in reasoning * modified a nimdoc test to reflect updated behavior * needed another change to bring utils.html doctest in sync with update * add info * fix nimdoc * lint * render postfix * fixes a problem * fixes nimdoc * fix nimdoc --------- Co-authored-by: johnperry-math <john.perry@usm.edu> Co-authored-by: johnperry-math <devotus@yahoo.com>
* fixes #21317; 1.6.4 regression; etyBaseIndex should return fat pointers ↵ringabout2023-02-012-3/+19
| | | | | [backport 1.6] (#21320) fixes #21317; regression; etyBaseIndex should return fat pointers
* fixes #16790; fixes #19075; put big arrays on the constant seqs; don't ↵ringabout2023-01-312-2/+24
| | | | | | | inline them in the VM; big performance boost (#21318) * don't inline arrays in VM * add a test for #19075
* remove md5_old (#21312)ringabout2023-01-313-299/+2
|
* remove `enumUtilsExist` defines (#21309)ringabout2023-01-291-17/+4
|
* Revert "Fix #13093 C++ Atomics: operator= is implicitly deleted because the ↵ringabout2023-01-275-54/+2
| | | | | | | default definition would be ill-formed " (#21307) Revert "Fix #13093 C++ Atomics: operator= is implicitly deleted because the default definition would be ill-formed (#21169)" This reverts commit a7bae919adb952362cb53206140872d2b7424b47.
* Fix #13093 C++ Atomics: operator= is implicitly deleted because the default ↵Bung2023-01-275-2/+54
| | | | | | | | | | | | | definition would be ill-formed (#21169) * add test * fix #17982 Invalid C++ code generation when returning discardable var T * fix #13093 * cpp atomic good example * clearify the condition
* Implemented basic macro expand functionality (#20579)Ivan Yonchovski2023-01-2713-60/+345
| | | | | | | | | | | | | | | | | | * Implemented level based macro expand functionality - it can handle single macro call or expand whole function/proc/etc and it - In addition, I have altered the parser to provide the endInfo for the node. The usefulness of the `endInfo` is not limited to the `expandMacro` functionality but also it is useful for `ideOutline` functionality and I have altered the ideOutline functionality to use `endInfo`. Note `endInfo` most of the time is lost during the AST transformation thus in `nimsuggest.nim` I am using freshly parsed tree to get the location information. * Make sure we stop expanding correctly * Test CI * Fix tv3_outline.nim
* fixes #14409; fixes #10674 VM callbacks switch to table-index seqs (#21297)ringabout2023-01-265-23/+37
| | | | | | | * fixes #14409; fixes#10674 VM callbacks switch to table-index seqs * fixes package name * reduce runtime cost
* fixes #18134; registers `formatBiggestFloat` in VM (#21299)ringabout2023-01-263-61/+73
| | | | | fixes #18134; registers formatBiggestFloat in vmops strformat supports float format in VM
* Update nims.md to use `nim e` instead of `nim` in advanced shebang (#21301)Heechul Ryu2023-01-261-1/+1
| | | | | | Update nims.md - suggest a better shebang that works with any filename - related to https://github.com/nim-lang/Nim/issues/17190
* remove dead code in VM (#21296)ringabout2023-01-254-11/+2
| | | remove deadcode in VM
* fixes #21273; fixes an io.readLine off by one bug [backport 1.0] (#21276)ringabout2023-01-252-3/+22
| | | fixes #21273; io.readLine off by one
* fixes #21261; always checking `nimTestErrorFlag` in the main module (#21288)ringabout2023-01-243-0/+13
| | | | | * fixes #21261; always checking nimTestErrorFlag in the main module * add a test
* fixes #21290; deindent if the last same level is a text node (#21293)ringabout2023-01-242-17/+38
| | | | | * fixes #21290; deindent if the last same level is a text node * add one more test
* add `md2html`, `md2tex` to cmd helpers (#21295)ringabout2023-01-241-0/+3
|
* Update the reference to parseSpec (#21289)ghais2023-01-211-1/+1
| | | The current reference pointed to the wrong location
* fixes #21278; `deques.shrink` off by one bug (#21284)ringabout2023-01-202-1/+7
| | | fixes #21278; deques.shrink off ny one bug
* switch to official chronos URL (#21283)ringabout2023-01-201-1/+1
| | | ref https://github.com/status-im/nim-chronos/pull/348
* Force lambda lifting for getImplTransformed. Hacky. Fixes #19818 (#21031)Yuriy Glukhov2023-01-193-6/+6
|
* close #21257 (#21275)metagn2023-01-182-1/+23
| | | | | * close #21257 * fix generics
* Implicitly set noNimblePath when nimble.lock is present (#21266)Ivan Yonchovski2023-01-181-0/+14
| | | Fixes https://github.com/nim-lang/nimble/issues/1004
* minor parseopt.nim improvements (#21256)Andreas Rumpf2023-01-182-39/+34
| | | | | | | * minor parseopt.nim improvements * attempt to make CI happy Co-authored-by: Clay Sweetser <Varriount@users.noreply.github.com>
* std/parsesql: Add OFFSET keyword support. (#21255)Phil Krylov2023-01-181-1/+10
| | | Co-authored-by: Clay Sweetser <Varriount@users.noreply.github.com>
* fixes #21243; ships `build_all.sh` on Unix [backport] (#21264)ringabout2023-01-181-0/+1
| | | fixes #21243; ships `build_all.sh` on Unix
* Add docs informing about gotcha of clients (#21262)PhilippMDoerner2023-01-181-0/+4
| | | | | | | | | | | | | | | * Add docs informing about gotcha of clients It is a hidden problem that AsyncHttpClient-instances *can not* deal with multiple requests at once. Similar to normal HttpClients they can only deal with sending one request at a time. This is not told anywhere in the documentation, but critical information that should be available to the user. * Update lib/pure/httpclient.nim fixes typo Co-authored-by: ringabout <43030857+ringabout@users.noreply.github.com> Co-authored-by: ringabout <43030857+ringabout@users.noreply.github.com>
* fixes #21260; add check for illegal recursion for defaults (#21270)ringabout2023-01-185-30/+38
| | | | | * fixes #21260; add check for illegal recursion for defaults * fixes differently
* fixes unittest warnings (#21271)ringabout2023-01-181-0/+4
|
* fixes #21263; consider all candidates for concept matches (#21265)ringabout2023-01-173-1/+43
|
* switch to official DelaunayNim URL (#21269)ringabout2023-01-171-1/+1
| | | follow up https://github.com/Nycto/DelaunayNim/pull/4#event-8240077930
* bump nimble (#21259)ringabout2023-01-141-1/+1
|
* fix #20253 (#21174)Bung2023-01-132-1/+14
| | | | | | | | | * 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-112-1/+19
| | | | | | | | | | | * Merge devel Add another test case * Fix test Use getCustomPragmaVal instead of hasCustomPragma Co-authored-by: Clay Sweetser <Varriount@users.noreply.github.com>
* testament: make valgrind optional (#21238)Anna2023-01-112-1/+12
| | | Add a simple switch to enable/disable valgrind support. On by default
* parseopt.nim documentation clarity - default values & cmdEnd in getopt (#21047)Russell Brinson2023-01-111-2/+27
| | | | | | parseopt.nim documentation clarity Added example for default values to cmd line parameters. Additionally, added lines in getopt documentation about case switching still requiring the `cmdEnd` kind. Hopefully this clears up any vagueness for those following along in the example but omitting the `cmdEnd` in the case because the documentation said it wasn't needed.
* fix typeRel of typeDesc (#21227)Bung2023-01-092-2/+10
|
* fixes #21232; highlite.getNextToken() returns gtOperator for '/' in C (#21234)ringabout2023-01-082-0/+13
|
* fixes #21209; exclude threads options for JS (#21233)ringabout2023-01-083-0/+9
|
* disable bump (#21241)ringabout2023-01-081-1/+1
|
* underscores for routine parameters (#21192)metagn2023-01-085-3/+131
| | | | | | | | | * underscores for routine parameters fixes #13443, fixes #13804, refs #21121 * add changelog + more tests * support generics and ensure inferred lambdas work
* adds changelog for more architectures support and fixes a link (#21229)ringabout2023-01-061-1/+2
|
* docgen: implement cross-document links (#20990)Andrey Makarov2023-01-0445-494/+1866
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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>
* make it printer friendly (#21218)Bung2023-01-042-0/+18
|
* fixes #21207; reports redefinition error in the definition of enums (#21217)ringabout2023-01-032-2/+11
| | | | | * fixes #21207; reports redefinition in the enums * add a test
* bump csource_v2 to include fix for linking parameter maximum length (#21216)ringabout2023-01-021-1/+1
| | | | | > If accepted, I believe it should be used in csources_v2 and also backport 2.0 ref https://github.com/nim-lang/Nim/pull/21186
* minor code cleanups (#21215)Andreas Rumpf2023-01-026-36/+32
|
* Update nim.cfg (#21204)Chavdar Ivanov2023-01-021-2/+1
| | | | | | | | | | | | | | | | tlsEmulation:on under NetBSD-10Beta and NetBSD-current produces an executable which crashes immediately as follows: Core was generated by `koch'. Program terminated with signal SIGSEGV, Segmentation fault. #0 0x000000000047b4c2 in nimZeroMem () (gdb) bt #0 0x000000000047b4c2 in nimZeroMem () #1 0x00000000004897b2 in threadVarAlloc__system_2162 () #2 0x000000000048980e in initThreadVarsEmulation () #3 0x0000000000489848 in PreMain () #4 0x000000000048986a in NimMain () #5 0x00000000004898a9 in main () I can't speak about the other BSDs.
* fixes pragmas reorder (#21205)ringabout2023-01-012-0/+20
|
* fixes #20906; update copyright year [backport 1.6] (#21210)ringabout2023-01-013-3/+3
|
* avoiding unnecessary allocation for dollar IpAddress (#21199)rockcavera2022-12-301-2/+2
| | | | | | | | | | | | | | | * avoiding allocating an unnecessary byte for IPv4 The inet.h file uses 16 as the string in C needs the last null byte https://github.com/torvalds/linux/blob/1b929c02afd37871d5afb9d498426f83432e71c2/include/linux/inet.h#L49 However, strings in Nim do not need this. So one byte is being allocated unnecessary and will never be used. * avoid unnecessary allocation in IPv6 dollar It is currently allocating 48 bytes. However, the Nim implementation for IPv6 will print a maximum of 39 characters. Nim does not implement IPv6 "0000:0000:0000:0000:0000:ffff:255.255.255.255" (45 characters) nor "0000:0000:0000:0000:0000:ffff:255.255.255.255%3" (47 characters). The indication in inet.h for 48 is due to the maximum use of 47 characters of a C string that needs a null byte at the end. So 48. https://github.com/torvalds/linux/blob/1b929c02afd37871d5afb9d498426f83432e71c2/include/linux/inet.h#L50