| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
* [JS] Fix casting to ints
* Simplify `genCast` by using `asUintN`/`asIntN`
|
|
|
|
|
|
|
|
|
|
|
| |
discriminator with defaults (#22303)
* fixes #22301; rejects branch initialization with a runtime discriminator with defaults
* undefault nimPreviewRangeDefault
* fixes tests
* use oldCheckDefault
|
|
|
|
|
|
|
| |
* Expands codegenDecl to work in function params. fixes #22306
* makes the test more concrete so T{lit} params dont match
* adds sfCodegenDecl
|
|
|
|
|
| |
Testcase
Co-authored-by: SirOlaf <>
|
| |
|
| |
|
|
|
| |
ref #22297; return in the finally in the closure iterators
|
|
|
|
|
| |
* test CI
* disable osx
|
|
|
|
|
|
|
|
|
|
|
| |
* wip; fixes #22210; transform return future in try/finally properly
* add a test case for #22210
* minor
* inserts a needsCompletion flag
* uses copyNimNode
|
| |
|
| |
|
|
|
|
|
| |
* Fixes Naive virtual crash the compiler fixes #22269
* adds type specific test
|
| |
|
|
|
|
|
| |
loop in ORC (#22240)
fixes #22237; fixes #21160; wrong cursor on unowned parameters
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add testcase
This tries to use a concept with a doc comment which currently leads to a segfault
* Ignore nil nodes which happen when there are doc comments in new style concept
This was done instead of semming the comments since `semConceptDecl` says it only supports lists of actual statements
* Go with alternative fix: Sem comments but ignore them
Since `nil` could mean anything it is best to not silently ignore it (In case another nil problem happens in future
Also fix test case so it isn't an infinite loop
|
| |
|
| |
|
| |
|
|
|
|
|
| |
(#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
|