| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
consistent with sequtils.delete; follows #18487 (#18510)
|
|
|
|
| |
violated (#18512)
|
|
|
|
|
|
|
|
|
|
|
| |
* Revert "followup #17398: `getTempDir`, `getConfigDir` now do not have trailing DirSep (#17402)"
This reverts commit 2356d0603f70cad90f76fa57999054bf5c0a9157.
* Revert "fix #17393 getHomeDir and expandTilde should not include trailing `/` (#17398)"
This reverts commit bebf2ce24a43bef4cde5c90c4010631a1e4a7927.
* fix test
|
|
|
|
|
|
|
| |
* Allow empty new-styled concept
Slightly improve error messages
* Make empty new-styled concepts an error
|
| |
|
|
|
|
|
|
|
| |
* make dragonbox opt-in via -d:nimFpRoundtrips
* make tests green again
* make tests green again
|
| |
|
|
|
|
|
| |
* added Atlas helper tool
* further improvements
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implemented support for Nimble local cache with package directories with
a checksum of the package at the end of their names. Now the compiler
supports package paths in the form:
* /path_to_nimble_cache_dir/pkgs/package_name-1.2.3-
FEBADEAEA2345E777F0F6F8433F7F0A52EDD5D1B
* /path_to_nimble_cache_dir/pkgs/package_name-#head-
042D4BE2B90ED0672E717D71850ABDB0A2D19CD2
* /path_to_nimble_cache_dir/pkgs/package_name-#branch-name-
DBC1F902CB79946E990E38AF51F0BAD36ACFABD9
Related to nim-lang/nimble#127
|
|
|
|
|
|
|
|
| |
consistent with `system.delete` (#18487)
* deprecate sequtils.delete and add an overload with saner semantics
* AssertionDefect => IndexDefect
* improve tsequtils
* add tests; use splice in js for optimization
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Extended side effect error messages
* Applied feedback:
- refactored `markSideEffect`
- refactored string interpolations
- single message
- skip diagnostics in `system.compiles` context
Other:
- started a test of diagnostic messages
[ci skip] Tests aren't updated yet because messaging isn't nailed down.
* - Added hints of where for side effect call locations.
- Tried to clarify the reasons.
* fix tests
* Applied PR review feedback:
- moved collection of side effects from TSym to TContext
- used pragma shorthand form `.sideEffect` and `.noSideEffect` in messages
- added leading '>' to structured messages for readability
- changed `sempass2.markSideEffect` to a proc
- replaced `system.echo` in the test to make the test compatible with Windows
* Applied NEP1 formatting suggestion
Co-authored-by: quantimnot <quantimnot@users.noreply.github.com>
|
|
|
|
|
| |
* added switch
* alloc.nim needs page aligned memory blocks
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* [skip ci] thamming_orc test created/destroyed counts match
The thamming_orc.nim code now counts all created objects being tested, not just the ones following the "first 20" test, and the position of the `destroyed += 1` counter has been adjusted so it counts all the calls that are as a result of `=trace` tracing and not just the original destruction calls.
* Improve description of how `=trace` is used
The following nuances weren't previously fully explained:
1. That `=trace` is only used by `--gc:orc`.
2. That `=trace` is almost certainly used along with `=destroy` when manual resource allocation has been used, but it is only required if there is a possibility of cyclic references in the wrapped types (ie. generic types).
3. That, currently, a forward definition is required for the second of the pair to avoid an auto compiler generation conflict.
The pattern of the use of `=trace` has also been made more extensive, showing how both a custom `=destroy` and `=trace` are used for manual allocation of resources when there is any possibility of a cyclic reference in the resource-wrapped values.
* Update doc/destructors.rst
Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
|
|
|
|
|
| |
* fix https://github.com/dom96/choosenim/issues/256 WSL CRLF
* fixup
|
|
|
| |
* fixes #17893
|
|
|
|
|
| |
* beneficial refactoring; use system.Endianness
* closes #18433
|
| |
|
| |
|
|
|
|
|
| |
* workaround Nightlies bug
* Update tools/niminst/makefile.nimf
|
|
|
| |
The thamming_orc.nim code now counts all created objects being tested, not just the ones following the "first 20" test, and the position of the `destroyed += 1` counter has been adjusted so it counts all the calls that are as a result of `=trace` tracing and not just the original destruction calls.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
* ORC: support custom =trace procs (WIP)
* Update tests/arc/tcustomtrace.nim
Co-authored-by: Clyybber <darkmine956@gmail.com>
* =trace is now documented and seems to work
* make test green
Co-authored-by: Clyybber <darkmine956@gmail.com>
|
|
|
|
|
|
|
|
|
| |
(#18457)
* runnableExamples now show originating location in stacktraces on failure
* PRTEMP
* fix indentation inside multiline strings
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* [docs] improve std/encodings
* tiny
* shrink
* Apply suggestions from code review
Co-authored-by: konsumlamm <44230978+konsumlamm@users.noreply.github.com>
* Apply suggestions from code review
* Update lib/pure/encodings.nim
Co-authored-by: Varriount <Varriount@users.noreply.github.com>
Co-authored-by: konsumlamm <44230978+konsumlamm@users.noreply.github.com>
Co-authored-by: Varriount <Varriount@users.noreply.github.com>
|
|
|
|
|
|
|
| |
The asynchronous process completion handler callback should be called
only once. This is achieved by passing `WT_EXECUTEONLYONCE` flag to the
`registerWaitForSingleObject` Windows API procedure.
Related to cheatfate/asynctools#35
|
| |
|
|
|
| |
This reverts commit 23d23ecb081be6702d74024be8f96d92d9f88a59.
|
|
|
|
|
|
|
|
|
| |
(#18389)
* improve --declaredlocs to help disambiguate types (generics, aliases etc)
* avoid a cyclic deps
* fix test after rebase
|
| |
|
| |
|
|
|
|
|
| |
* improve rendering of newOSError.additionalInfo
* fixup
|
|
|
|
|
|
| |
* ORC refactoring in preparation for further changes (=dispose must die)
* ORC: embrace =destroy, avoid =dispose
* ORC: no need for =dispose
* closes #18421
|
|
|
|
|
|
|
| |
semantics (#18435)
* wrapnils now generates optimal code; also handles case objects
* changelog
* unsafeAddr => addr
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* fix compilation on Debian 7 (no git -C)
* address review
* allow specify branch for testing unmerged csources
* temporarily change csources checkout parameters for testing
* Update tools/deps.nim
Co-authored-by: Timothee Cour <timothee.cour2@gmail.com>
* fix failure
* Update config/build_config.txt
* set proper git branch/hash
Co-authored-by: Timothee Cour <timothee.cour2@gmail.com>
Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
mismatches (#18384)
* Added more concise calling convention/pragma mismatch messages
* Now only adds callConvMsg/lock message when sensible
* Fixed message formatting
* Added tests, and fixed some bugs
* Tests joined, and always indenting
* More tests and more bug fixes
* Fixed first test in tprocmismatch
* Using var param for writting mismatches
* Better logic for handling proc type rel and conv/pragma mismatch
* Refactored getProcConvMismatch
* Fixed callConv message formatting
* Fixed test for proper message
* Cleanup to address issues
* getProcConvMismatch now returns tuple, and reformatted code
|
| |
|
|
|
| |
clear/setTimeout procs were using ref ref Timeout. Added more interval overloads. Corrections.
|
|
|
|
|
| |
Co-authored-by: konsumlamm <44230978+konsumlamm@users.noreply.github.com>
Co-authored-by: Timothee Cour <timothee.cour2@gmail.com>
Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
|
| |
|
|
|
|
|
|
|
| |
runnableExamples (#18428)
* std/net: show addr+port on address already in use error; code-block => runnableExamples
* var=>let
|
|
|
|
|
|
|
| |
* Add lent annotation to avoid extra copies in sample
* Remove lent
* Update random.nim
|
| |
|
| |
|
|
|
|
| |
identifier` nimrtl error (#18406)
|
| |
|
|
|
|
|
|
| |
Co-authored-by: Dominik Picheta <dominikpicheta@googlemail.com>
Co-authored-by: Mark Pointing <mark@futurepoint.com.au>
Co-authored-by: Andreas Rumpf <rumpf_a@web.de>
Co-authored-by: Timothee Cour <timothee.cour2@gmail.com>
|