misc curated list of unfixed closed issues & PR's

[timotheecour]

issues

• compiler error message can improve: when forgetting = after proc signature · Issue #10229 · nim-lang/Nim : compiler error message can improve: when forgetting = after proc signature · Issue #10229 · nim-lang/Nim

• reorder:on fails with quoted proc · Issue #8801 · nim-lang/Nim : reorder:on fails with quoted proc · Issue #8801 · nim-lang/Nim

• deprecate macros.expectLen, expectKind, expectMinLen in favor of something generic or doAssert · Issue #185 · nim-lang/RFCs : deprecate macros.expectLen, expectKind, expectMinLen in favor of something generic or doAssert · Issue #185 · nim-lang/RFCs

• add more files to the ignore list, hopefully fixes nightlies on windows by narimiran · Pull Request #13474 · nim-lang/Nim : add more files to the ignore list, hopefully fixes nightlies on windows by narimiran · Pull Request #13474 · nim-lang/Nim
  see add more files to the ignore list, hopefully fixes nightlies on windows nim-lang/Nim#13474 (comment)

• [CI] Install node.js 8.x flaky step · Issue #13477 · nim-lang/Nim : [CI] Install node.js 8.x flaky step · Issue #13477 · nim-lang/Nim
  pending [WIP] fix #6999 (allow [nimDocOnly] in commit msg); fix #13477 (flaky nodejs install); log PR url (+other info) in runCI logs nim-lang/Nim#13556

• switch CI to azure-pipelines for much faster CI · Issue #167 · nimterop/nimterop : switch CI to azure-pipelines for much faster CI · Issue #167 · nimterop/nimterop

• see TODO in koch --nim:pathto/nim boot and koch boot --hint:cc:off now work  nim-lang/Nim#13516

Now, in combination with nim-lang#13488 and this PR, we can have a perfect workaround for nim-lang#13411 (isolate the build process from external configuration files) if we allow build_all.sh to forward arguments to the koch build commands:

• nim-lang@bcccb74
  of nkFloatLit..nkFloat64Lit: result = castuint64 == castuint64

• for a in MyEnum gives incorrect results with enum with holes · Issue #14001 · nim-lang/Nim
  => this should work but doesn't

• --gc:arc SIGSEGV with ptr aliasing · Issue #13137 · nim-lang/Nim

• re.split unexpected results with zero-width characters · Issue #14468 · nim-lang/Nim

• Finalizers are not called on thread exit · Issue #12192 · nim-lang/Nim

• TITLE: --gc:arc doesn't detect cycle, leaks memory · Issue #13080 · nim-lang/Nim

• [docgen] Show the commit used to generate the devel docs · Issue #9073 · nim-lang/Nim

• What are the compiler arguments used? · Issue #21 · lh3/biofast

• Writing to readonly data gives SIGBUS error; should be CT error instead · Issue #8463 · nim-lang/Nim

• Nimsuggest crashes when run without a backend specified · Issue #117 · nim-lang/nimsuggest

• sugar.collect doesn't work with string + other bugs · Issue #16078 · nim-lang/Nim

PR's

• list: https://github.com/pulls?q=is%3Apr+author%3Atimotheecour+sort%3Aupdated-desc+in%3Atitle+TODO+is%3Aclosed

• assertFalse: {.noreturn.} alternative to doAssert(false); safer than quit by timotheecour · Pull Request #13492 · nim-lang/Nim : assertFalse: {.noreturn.} alternative to doAssert(false); safer than quit by timotheecour · Pull Request #13492 · nim-lang/Nim

• docgen: don't create compiler's docs + cleanup by narimiran · Pull Request #13509 · nim-lang/Nim : docgen: don't create compiler's docs + cleanup by narimiran · Pull Request #13509 · nim-lang/Nim
  => need to re-enable docs for compiler

• [RFC] add isDefault; more general than isNil, isEmpty etc by timotheecour · Pull Request #13526 · nim-lang/Nim : [RFC] add isDefault; more general than isNil, isEmpty etc by timotheecour · Pull Request #13526 · nim-lang/Nim

• fix #13412 nim now recompiles for stdin input; SuccessX now configurable; can show whether it recompiled by timotheecour · Pull Request #13506 · nim-lang/Nim : fix #13412 nim now recompiles for stdin input; SuccessX now configurable; can show whether it recompiled by timotheecour · Pull Request #13506 · nim-lang/Nim
  => branch:pr_fix_13412_nim_stdin_rebuild_SuccessX_config
  D20200319T025934

• [RFC] 'walkDir' now has a new 'checkDir' flag, to mimic behaviour of other languages by timotheecour · Pull Request #13642 · nim-lang/Nim : [RFC] 'walkDir' now has a new 'checkDir' flag, to mimic behaviour of other languages by timotheecour · Pull Request #13642 · nim-lang/Nim

  • need to make checkDir=true the new default, with some migration path, or add a new API eg dirs, dirsRec etc as suggested here [RFC] 'walkDir' now has a new 'checkDir' flag, to mimic behaviour of other languages nim-lang/Nim#13642 (comment)
  • nimble needs warning when nothing gets tested for a package see [RFC] 'walkDir' now has a new 'checkDir' flag, to mimic behaviour of other languages nim-lang/Nim#13642 (comment)

• Add module jsfetch by juancarlospaco · Pull Request #12531 · nim-lang/Nim : Add module jsfetch by juancarlospaco · Pull Request #12531 · nim-lang/Nim

• fix #13502 a and b now short-circuits semcheck in VM by timotheecour · Pull Request #13541 · nim-lang/Nim : fix #13502 a and b now short-circuits semcheck in VM by timotheecour · Pull Request #13541 · nim-lang/Nim
  => fix #13502 a and b now short-circuits semcheck in VM nim-lang/Nim#13541 (comment)

lazyAnd is a good name, but consider [...]

• Add JavaScript Exponentiation by juancarlospaco · Pull Request #12455 · nim-lang/Nim : Add JavaScript Exponentiation by juancarlospaco · Pull Request #12455 · nim-lang/Nim

• https://github.com/nim-lang/Nim/pull/11824/files
  => extract splitDefinition from that PR + re-introduce tests/stdlib/tsugar.nim

• revive [TODO] allow running cast in UFCS chains (everything else supports UFCS) by timotheecour · Pull Request #8528 · nim-lang/Nim : [TODO] allow running cast in UFCS chains (everything else supports UFCS) by timotheecour · Pull Request #8528 · nim-lang/Nim
  with same argument as here: distinctBase overload for values nim-lang/Nim#13746 (comment)

• enummaps: library based generalized typesafe enum with holes by timotheecour · Pull Request #14008 · nim-lang/Nim

• Allow underscored identifiers if within accented quotes by Vindaar · Pull Request #9014 · nim-lang/Nim

• experiment showing how iterators can (almost) be implemented with a library solution by timotheecour · Pull Request #15655 · nim-lang/Nim

• MultiTable: thin wrapper around Table with sane API for duplicate keys; deprecate Table.add · Issue #200 · nim-lang/RFCs

• add deferScoped, library defined scoped defer by timotheecour · Pull Request #16048 · nim-lang/Nim

PR's needing followup

Updated 'nim for embedded systems' section to use --os:any and --gc:arc by zevv · Pull Request #13237 · nim-lang/Nim

follow#16107 make heapqueue support user-defined cmp function by xflywind · Pull Request #16113 · nim-lang/Nim

• bring back nimbleutils and actionRetry from fix common source of flaky tests (443 / 503 github errors) by retrying with backoff nim-lang/Nim#14547 (was only partially re-integrated in Retry commands in testament again nim-lang/Nim#16262)

move to another repo / fusion

• glob: more flexible than walkDirRec by timotheecour · Pull Request #15598 · nim-lang/Nim

done

• echo 'echo 1' | nim c -r - silently gives wrong results (nimBetterRun not updated for stdin) · Issue #13412 · nim-lang/Nim : echo 'echo 1' | nim c -r - silently gives wrong results (nimBetterRun not updated for stdin) · Issue #13412 · nim-lang/Nim

• --gc:arc not self-consistent, not consistent with default gc regarding catching AssertionError · Issue #13585 · nim-lang/Nim : --gc:arc not self-consistent, not consistent with default gc regarding catching AssertionError · Issue #13585 · nim-lang/Nim

• Code behaves different with and without --gc:arc · Issue #13465 · nim-lang/Nim : Code behaves different with and without --gc:arc · Issue #13465 · nim-lang/Nim
  pending We're adding catchdefects:on/off but closing for now.
  => --panics:on|off

• Generate c code cannot compile with recent devel version · Issue #13645 · nim-lang/Nim : Generate c code cannot compile with recent devel version · Issue #13645 · nim-lang/Nim
  Generate c code cannot compile with recent devel version nim-lang/Nim#13645 (comment)

• remove deprecated things in macros except for callsite by dumjyl · Pull Request #13710 · nim-lang/Nim

• fix #12457 by dumjyl · Pull Request #13767 · nim-lang/Nim

• fix #1994 gorge, staticExec now error at CT on exitCode !=0 by timotheecour · Pull Request #10360 · nim-lang/Nim

[narimiran]

need to make checkDir=true the new default

Nope.