| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
(#22211)
fixes #22208; Ambiguous error when import modules with same names but different aliases
|
| |
|
| |
|
|
|
| |
the function actually returns
|
|
|
| |
fixes #22187
|
|
|
|
| |
(#22184)
|
|
|
|
|
|
|
| |
* zero initialization union casts
* cleans up and adds a test case for #19101
* uses nimZeroMem
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Don't throw errors on RST tables in Markdown and RstMarkdown modes
Additions to RST simple tables (#19859) made their parsing more
restrictive, which can introduce problems with of some old
nimforum posts, which have tables with sloppily aligned columns
(like this one:
https://github.com/nim-lang/nimforum/issues/330#issuecomment-1376039966).
Also this strictness contradicts to Markdown style of not getting
in the way (ignoring errors).
So this PR proposes a new strategy of dealing with errors:
* In Markdown and legacy (old default) RstMarkdown we try to
continue parsing, emitting only warnings
* And only in pure RST mode we throw a error
I expect that this strategy will be applied to more parts of markup code
in the future.
* Don't return anything in `checkColumns`
|
|
|
|
|
| |
fixes #22180
Backported since apparently the new type inference was backported
|
|
|
|
|
| |
* fixes #22001
* added test case
|
|
|
|
|
|
|
| |
* adds =destroy T support for strings and seqs
* fixes system
* fixes tests
|
|
|
|
|
|
|
|
|
| |
* adds T destructor for refs
* add `newRefdestructor`
* adds ref overload for destructors
* fixes config
|
|
|
|
|
|
|
| |
* Add simple test case
Just so the regression doesn't happen again
* Specify initDateTime is gcsafe in the forward declarations
|
|
|
|
|
|
|
| |
(#22154)
* fixes #22148; std/memfiles.memSlices nesting now fails with memory safety capture violation
* adds a test case
|
|
|
|
|
|
|
|
|
|
|
| |
* fix `var object` not matching better than `object`
fixes #13302
* remove comment for brevity
* try note
* try minimize breaks
|
|
|
|
|
|
|
|
|
|
|
| |
* fix VM uint conversion bug, stricter int gen on JS
fixes #19929
* fix float -> uint64 conversion too
* no need to mask to source type
* simpler diff with explanation, add test for described issue
|
|
|
| |
adds optonal --nimbasepattern
|
|
|
| |
fixes #21110; duplicate proc definitions for iters
|
|
|
|
|
|
|
| |
module name specifier (#22109)
* fixes #21231; template with module as parameter elides usage/checking of module name specifier
* add a test case
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* fix #20023 hash for generic tables
* use default computation
* Update lib/pure/collections/tables.nim
Co-authored-by: Dan Rose <dan@digilabs.io>
* Update lib/pure/collections/tables.nim
Co-authored-by: Dan Rose <dan@digilabs.io>
* Update lib/pure/collections/tables.nim
* Update lib/pure/collections/tables.nim
* Update t20023.nim
---------
Co-authored-by: Dan Rose <dan@digilabs.io>
Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a rebase of an earlier rejected PR. Following the discussion
around it, this commit provides a valid output for and edge case
of an empty separator for `split` and `rsplit` routines. The empty
separator is interpreted as "split by no separators" and the initial
string is returned. This is consistent with the behaviour of the
`set[char]` version of `split`/`rsplit` routines and unifies them all.
Compared to a commit merged earlier, this one has a benefit of
not using assertions that will be removed in release builds
and thus still not preventing possible infinite loops (which was the
earlier behaviour for this edge case for separator of type `string`).
Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
|
|
|
|
|
|
|
|
|
|
| |
var T)` (#22130)
* make destructors accept non var parameters
* define nimAllowNonVarDestructor
* add a test case and a changelog
* update documentation and error messages
* deprecate destructors taking 'var T'
|
|
|
|
|
|
|
| |
* Add Overflow checks & test adjust
* Avoiding nimvm differences in tests
* distinguish DivByZeroDefect
|
| |
|
| |
|
|
|
| |
refs #22029, refs https://github.com/status-im/nim-libp2p/actions/runs/5263850340/jobs/9514434659
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Adding divmod
* Adding support to VM
* Wrapped C structs and funcs
* Fix javascript impl
* Fixing struct compat
* Segregate tests, better compiletime defs
* Using `inline` and switch back to `func`
* Apply suggestions from code review
* Explicit structures
---------
Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
|
|
|
|
|
|
|
| |
(#22113)
* fixes #16331; aliasing of tuple construction within a single assignment, great coding style
* added test case
|
|
|
|
|
|
|
| |
* add test to close #7209
was fixed by #22029
* fix echo => doAssert
|
|
|
|
|
| |
add test to close #7223, close #11733
closes #7223, closes #11733, were fixed by #22076
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* fixes #22054; codegen for var tuples conv
* rethink fixes
* add test cases
* templates only
* fixes var tuples
* keep varness no matter what
* fixes typ.isNil
* make it work for generics
* restore isSubrange
* add a test case as requested
|
|
|
|
|
|
|
|
|
| |
* consider object types as declarative in templates
fixes #16005
* correct logic for nkRecList children, inject fields
* don't actually inject fields
|
| |
|
|
|
|
|
|
|
| |
* adds another pass for sets fixes #6259
* Update tsets.nim
removes extra `#`
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Allows for arbitrary ordering of inheritance in type section #6259
* prevents ilegal recursion
* fixes ilegal recursion. Test passes with a better message
* Apply suggestions from code review
---------
Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* sacrifice "tgenericshardcases" for working statics
* legacy switch for CI, maybe experimental later
* convert to experimental
* apparently untyped needs the experimental switch
* try special case call semcheck
* try fix
* fix compilation
* final cleanup, not experimental, make `when` work
* remove last needed use of untyped
* fix unused warning in test
* remove untyped feature
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* fixes #22065; do not search path for "./"
* simplify
* fixes
* fixes
* allow ".."
* cleanup
* add a test case
* slightly modify the import
* adds a changelog
|
|
|
|
|
| |
* strictly typecheck expressions in bracketed `emit`
* use nim check in test
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* fix dot calls with resolved symbols in templates
* make old code work
* fix custom number literals test
* remove leftover debug marker
* enable "bug 9" test too
* fix renderer, add test for #7085
|
|
|
|
|
|
|
| |
* binary `not` only parses simple expressions
fixes #16324
* switch to primary
|
|
|
| |
closes #9899, closes #14708, refs #21017
|
| |
|
|
|
|
|
|
|
| |
* add const RLIMIT_STACK
* generate by detect.nim
* add generated const for linux-amd64
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* make `move` use `=wasMoved` internally
* fixes tests
* fixes spawn finally
* fixes views
* rename to internalMove
* add a test case
|
| |
|
|
|
| |
allow addressing elements of openArray[char]
|
| |
|
|
|
|
|
| |
* Implemented with-nesting in underscoredCalls()
* Add tests for nested with
|